Implement interface to send data stop notification to stream listeners
Categories
(Core :: Networking: HTTP, enhancement, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox121 | --- | fixed |
People
(Reporter: smayya, Assigned: smayya)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
Attachments
(7 files, 4 obsolete files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review |
We would like to send a OMT message to stream listners to notify them that there wont be any more ODA notifications sent to them and hence, they can start processing the data.
To do this we will need a new interface. This will be implemented by this bug.
For start, we will send these notifications to css and html5 listeners.
Assignee | ||
Updated•9 months ago
|
Updated•9 months ago
|
Assignee | ||
Comment 1•9 months ago
|
||
Comment 2•9 months ago
|
||
Assignee | ||
Comment 3•9 months ago
|
||
Assignee | ||
Comment 4•9 months ago
|
||
Depends on D187832
Updated•9 months ago
|
Updated•9 months ago
|
Assignee | ||
Comment 5•9 months ago
|
||
Updated•9 months ago
|
Assignee | ||
Comment 6•9 months ago
|
||
Depends on D187833
Assignee | ||
Comment 7•8 months ago
|
||
Depends on D188757
Updated•8 months ago
|
Updated•8 months ago
|
Updated•8 months ago
|
Updated•8 months ago
|
Updated•8 months ago
|
Updated•8 months ago
|
Updated•8 months ago
|
Assignee | ||
Comment 8•8 months ago
|
||
Updated•8 months ago
|
Updated•8 months ago
|
Pushed by smayya@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/03c17255f1af implement nsIThreadRetargetableStreamListener::OnDataFinished to send OMT data stop notification to thread retargetable stream listeners. r=necko-reviewers,jesup,valentin https://hg.mozilla.org/integration/autoland/rev/79969d9eb5cd moving html5 stream parser's OnStopRequest off main thread. r=necko-reviewers,edgul,jesup,valentin,hsivonen https://hg.mozilla.org/integration/autoland/rev/fa61d4e76827 implement nsIThreadRetargetableStreamListener::OnDataFinished to multiple listeners. r=necko-reviewers,extension-reviewers,jesup,rpl https://hg.mozilla.org/integration/autoland/rev/95e4df69007a send onDataFinished notifications from nsInputStreamPump. r=necko-reviewers,jesup https://hg.mozilla.org/integration/autoland/rev/d28399b3d947 implement empty nsIThreadRetargetableStreamListener::OnDataFinished for stream loader. r=emilio
Comment 10•8 months ago
|
||
Backout by chorotan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d354d93ae06b Backed out 5 changesets for causing multiple failures
Comment 11•8 months ago
|
||
Backed out 5 changesets (Bug 1851992) for causing multiple failures
Backout: https://hg.mozilla.org/integration/autoland/rev/d354d93ae06b28127112934f9ed4cb28415fa182
Failure logs:
https://treeherder.mozilla.org/logviewer?job_id=432788946&repo=autoland&lineNumber=9479
https://treeherder.mozilla.org/logviewer?job_id=432787146&repo=autoland&lineNumber=9125
Assignee | ||
Updated•8 months ago
|
Assignee | ||
Comment 12•8 months ago
|
||
Depends on D191097
Assignee | ||
Comment 13•8 months ago
|
||
Depends on D191290
Updated•8 months ago
|
Comment 14•7 months ago
|
||
Pushed by smayya@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/38c40d735ab7 implement nsIThreadRetargetableStreamListener::OnDataFinished to send OMT data stop notification to thread retargetable stream listeners. r=necko-reviewers,jesup,valentin https://hg.mozilla.org/integration/autoland/rev/f9f9143ac713 moving html5 stream parser's OnStopRequest off main thread. r=necko-reviewers,edgul,jesup,valentin,hsivonen https://hg.mozilla.org/integration/autoland/rev/46e18c56dd39 implement nsIThreadRetargetableStreamListener::OnDataFinished to multiple listeners. r=necko-reviewers,extension-reviewers,jesup,rpl https://hg.mozilla.org/integration/autoland/rev/673df3f83249 send onDataFinished notifications from nsInputStreamPump. r=necko-reviewers,jesup https://hg.mozilla.org/integration/autoland/rev/c9d322362e22 implement empty nsIThreadRetargetableStreamListener::OnDataFinished for stream loader. r=emilio https://hg.mozilla.org/integration/autoland/rev/3ceaf46f8f55 move assignment of mTransferSize,mEncodedBodySize from HttpChannelChild::OnStopRequest to HttpChannelChild::OnStopRequest::ProcessOnStopRequest. r=necko-reviewers,jesup
Comment 15•7 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/38c40d735ab7
https://hg.mozilla.org/mozilla-central/rev/f9f9143ac713
https://hg.mozilla.org/mozilla-central/rev/46e18c56dd39
https://hg.mozilla.org/mozilla-central/rev/673df3f83249
https://hg.mozilla.org/mozilla-central/rev/c9d322362e22
https://hg.mozilla.org/mozilla-central/rev/3ceaf46f8f55
Comment 16•7 months ago
|
||
Backed out for DT failure on browser_net_image_cache.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/2d73b37b48b3c0d49e800d71641f3283ae9ae9b6
Log link: https://treeherder.mozilla.org/logviewer?job_id=433555456&repo=autoland&lineNumber=7649
Assignee | ||
Updated•7 months ago
|
Assignee | ||
Comment 17•7 months ago
|
||
Depends on D191291
Comment 18•7 months ago
|
||
Pushed by smayya@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4517b343b411 implement nsIThreadRetargetableStreamListener::OnDataFinished to send OMT data stop notification to thread retargetable stream listeners. r=necko-reviewers,jesup,valentin https://hg.mozilla.org/integration/autoland/rev/f7fe117ca37d moving html5 stream parser's OnStopRequest off main thread. r=necko-reviewers,edgul,jesup,valentin,hsivonen https://hg.mozilla.org/integration/autoland/rev/fcb29a7d62eb implement nsIThreadRetargetableStreamListener::OnDataFinished to multiple listeners. r=necko-reviewers,extension-reviewers,jesup,rpl https://hg.mozilla.org/integration/autoland/rev/4471b2edc794 send onDataFinished notifications from nsInputStreamPump. r=necko-reviewers,jesup https://hg.mozilla.org/integration/autoland/rev/d2e0fddb380e implement empty nsIThreadRetargetableStreamListener::OnDataFinished for stream loader. r=emilio https://hg.mozilla.org/integration/autoland/rev/f21cf0e9ff8e move assignment of mTransferSize,mEncodedBodySize from HttpChannelChild::OnStopRequest to HttpChannelChild::OnStopRequest::ProcessOnStopRequest. r=necko-reviewers,jesup https://hg.mozilla.org/integration/autoland/rev/e480891bbd3b ensure all of the OnProgress updates are sent before triggering OnDataFinished in HttpChannelChild. r=necko-reviewers,jesup
Comment 19•7 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4517b343b411
https://hg.mozilla.org/mozilla-central/rev/f7fe117ca37d
https://hg.mozilla.org/mozilla-central/rev/fcb29a7d62eb
https://hg.mozilla.org/mozilla-central/rev/4471b2edc794
https://hg.mozilla.org/mozilla-central/rev/d2e0fddb380e
https://hg.mozilla.org/mozilla-central/rev/f21cf0e9ff8e
https://hg.mozilla.org/mozilla-central/rev/e480891bbd3b
Comment 20•7 months ago
|
||
Comment on attachment 9355381 [details]
Bug 1851992 - implement OnDataFinished for CSSLoader. r=#necko,jesup,emilio
Revision D189403 was moved to bug 1864817. Setting attachment 9355381 [details] to obsolete.
Description
•