Closed Bug 76866 Opened 24 years ago Closed 24 years ago

http spews many "private" events before any real data events

Categories

(Core :: Networking: HTTP, defect)

defect
Not set
major

Tracking

()

VERIFIED FIXED
mozilla0.9.1

People

(Reporter: darin.moz, Assigned: darin.moz)

References

Details

(Keywords: perf)

http spews many "private" events before any real data events. because layout can occupy the UI thread for relatively long periods of time, processing the top level document, this can result in large delays before actually downloading any sub documents such as inline images. by pushing more of http onto the socket thread, we can ensure that the first plevent posted via http corresponds to real data for http's listener.
targetting 0.9.1
Status: NEW → ASSIGNED
Keywords: nsbeta1
Target Milestone: --- → mozilla0.9.1
this work is happening on the branch: HttpConnectionMagic_20010415_BRANCH and currently includes: netwerk/protocol/http, netwerk/build, and netwerk/test
actually, it was necessary to branch all of netwerk/
Keywords: perf
Blocks: 77075
Blocks: 71668
Blocks: 44536
Blocks: 35956
Blocks: 47352
Blocks: 48202
Blocks: 60811
Blocks: 61229
Blocks: 66694
Blocks: 70968
Blocks: 71211
Blocks: 71333
Blocks: 71334
Blocks: 71645
Blocks: 72383
Blocks: 70208
the following files are also on the branch: dom/src/base/nsGlobalWindow.cpp content/html/document/src/nsHTMLContentSink.cpp content/html/document/src/nsHTMLDocument.cpp content/xml/document/src/nsXMLDocument.cpp content/xml/document/src/nsXMLDocument.h content/xbl/src/nsXBLService.cpp content/build/nsContentHTTPStartup.cpp content/build/gbdate.pl layout/html/forms/src/nsFormFrame.cpp xpfe/bootstrap/nsAppRunner.cpp xpfe/browser/src/nsBrowserInstance.cpp xpfe/components/bookmarks/src/nsBookmarksService.cpp xpfe/components/search/src/nsInternetSearchService.cpp xpfe/components/xfer/src/nsStreamTransfer.cpp xpfe/components/ucth/src/nsUnknownContentTypeHandler.cpp docshell/base/nsDocShell.cpp docshell/base/nsWebShell.cpp gfx/src/nsImageNetContextAsync.cpp uriloader/base/nsDocLoader.h uriloader/base/nsDocLoader.cpp uriloader/base/nsURILoader.cpp uriloader/exthandler/nsExternalHelperAppService.cpp modules/plugin/nglsrc/nsPluginHostImpl.cpp intl/chardet/src/nsObserverBase.cpp embedding/browser/webBrowser/nsWebBrowserPersist.cpp extensions/cookie/nsCookieHTTPNotify.h extensions/cookie/nsCookieHTTPNotify.cpp mailnews/compose/src/nsMsgCompUtils.cpp mailnews/compose/src/nsURLFetcher.cpp mailnews/compose/src/nsURLFetcher.h extensions/xmlextras/base/src/nsXMLHttpRequest.h extensions/xmlextras/base/src/nsXMLHttpRequest.cpp extensions/xmlextras/base/public/nsIXMLHttpRequest.idl mailnews/absync/src/nsAbSyncPostEngine.cpp
Keywords: nsbeta1nsbeta1+
Blocks: 71740
Blocks: 70919
Blocks: 75783
Blocks: 63745
Blocks: 60588
Depends on: 79585
Blocks: 59016
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
the branch has landed.
qa to me. Anything I can do to verify this?
QA Contact: tever → benc
prolly not.. jrgm confirmed the 10% speed up.. so that should be sufficient.
Yeah, indeed, I hath seen this effect, so vrfy.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.