Closed Bug 1393909 Opened 3 years ago Closed 3 years ago
Remove sync messaging in Web
59 bytes, text/x-review-board-request
WebRequestContent uses sync messaging if we have blocking listeners. With legacy addons removed in 57, this is no longer necessary since WebExtensions cannot make use of it. Lets remove those bits of code so the messaging is all async.
Comment on attachment 8901308 [details] Bug 1393909 remove sync messaging from WebRequest, https://reviewboard.mozilla.org/r/172754/#review178184 r=me if tests pass. There might still be some legacy WebRequest.jsm tests that rely on this. If so, we should just disable them.
Attachment #8901308 - Flags: review?(kmaglione+bmo) → review+
(In reply to Kris Maglione [:kmag] from comment #2) > There might still be some legacy WebRequest.jsm tests that rely on this. If > so, we should just disable them. I ran what I could find local and they passed. should hopefully catch anything else on try.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/e1d63de12f2c remove sync messaging from WebRequest, r=kmag
You need to log in before you can comment on or make changes to this bug.