Closed
Bug 1399005
Opened 7 years ago
Closed 7 years ago
Fix strict ordering of data and stop request events with cross-thread stream filters
Categories
(WebExtensions :: Request Handling, defect, P1)
WebExtensions
Request Handling
Tracking
(firefox57 fixed)
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: kmag, Assigned: kmag)
References
Details
Attachments
(1 file)
When data delivery is not on the same thread as the StreamFilterParent actor, it's sometimes possible for the OnStopRequest to be dispatched on the main thread before the last OnDataAvailable event has been processed on the IO thread, which causes issues for listeners that expect it to be otherwise.
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8907197 [details] Bug 1399005: Always dispatch OnStop from the IO thread. https://reviewboard.mozilla.org/r/178870/#review183986
Attachment #8907197 -
Flags: review?(mixedpuppy) → review+
Updated•7 years ago
|
status-firefox57:
--- → affected
Priority: -- → P1
Assignee | ||
Comment 4•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a676cb6f09a4d0b29cc8b0072acc035c2082224 Bug 1399005: Always dispatch OnStop from the IO thread. r=mixedpuppy
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9a676cb6f09a
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Comment 6•6 years ago
|
||
Is manual testing required on this bug? If Yes, please provide some STR and the proper webextension(if required), if No set the “qe-verify-“ flag.
Flags: needinfo?(kmaglione+bmo)
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(kmaglione+bmo) → qe-verify-
Updated•6 years ago
|
Product: Toolkit → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•