1398642 - More WebRequest.jsm optimizations

Status: RESOLVED FIXED

(WebExtensions :: Request Handling, defect)

Comment 1

[Kris Maglione [:kmag]]

Attachment: Bug 1398642: Several minor WebRequest optimizations.

Review commit: https://reviewboard.mozilla.org/r/178136/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/178136/

Comment 2

[Shane Caraveo (:mixedpuppy)]

Comment on attachment 8906423 [details]
Bug 1398642: Several minor WebRequest optimizations.

https://reviewboard.mozilla.org/r/178136/#review183352

Comment 3

[Kris Maglione [:kmag]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1b595b630d490254b859c2100c8808aad544666a
Bug 1398642: Several minor WebRequest optimizations. r=mixedpuppy

Comment 4

[Kris Maglione [:kmag]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/fbc99e4af34facef5b4a5fee6a34a3f99f8d814e
Bug 1398642: Follow-up: Fix terrible error checking code. r=me CLOSED TREE

Comment 5

[Kris Maglione [:kmag]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/33cfd48a65ac744b5a9be36ef35c290581558c9e
Bug 1398642: Bustage fix bustage fix. r=me CLOSED TREE

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/1b595b630d49
https://hg.mozilla.org/mozilla-central/rev/fbc99e4af34f
https://hg.mozilla.org/mozilla-central/rev/33cfd48a65ac

Comment 7

[marius.santa]

Is manual testing required on this bug? If yes, please provide some STR and the proper webextension(if required) or set the “qe-verify-“ flag.