Closed Bug 1199504 Opened 5 years ago Closed 2 years ago

Get webRequest to work in b2g WebExtensions

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(Not tracked)

RESOLVED WONTFIX
FxOS-S7 (18Sep)

People

(Reporter: fabrice, Assigned: fabrice)

References

Details

Attachments

(1 file, 1 obsolete file)

No description provided.
Summary: Get webRequest to work on b2g WebExtensions → Get webRequest to work in b2g WebExtensions
Target Milestone: --- → FxOS-S7 (18Sep)
Attached patch wip (obsolete) — Splinter Review
This patch works for me on devices, but tests are still failing:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=f50a43eb051e

The reason of the failures is that in these tests moz-extension: is an alias for a file: uri, but content processes don't have the rights to open file: content for security reasons.

Bill, should I port all the tests to be installed from app: urls, or should we use the JSON support that I saw you added recently?
Flags: needinfo?(wmccloskey)
I filed bug 1202897 to get WebExtension tests converted to using generated zip files from JSON. Then Fabrice will work on using app: URIs for these zip files. For reference, here's the code where the zip is created:

http://mxr.mozilla.org/mozilla-central/source/toolkit/components/extensions/Extension.jsm#457
Flags: needinfo?(wmccloskey)
Attached patch webrequest.patchSplinter Review
Attachment #8657956 - Attachment is obsolete: true
Blocks: 1225463
No longer blocks: 1225463
Yo Fabrice, is the work needed to get this landed busy work (like fixing up some tests formats) that I can do? If there is something I can do to help get this landed would be happy to
Flags: needinfo?(fabrice)
We are basically blocked by bug 1224703 where we need to figure out why the message manager behaves differently on b2g and desktop e10s. Bill is on it.
Flags: needinfo?(fabrice)
Blocks: 1233402
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.