|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
We currently spend several milliseconds at startup creating these bindings, despite the fact that they're never used.
Is this going to break anything in test pilot?
Thanks for the ping. This doesn't affect Test Pilot or any active experiments right now. :jkt was saying it will affect the new lightbeam add-on though (not a test pilot experiment)
If any extensions need an indexedDB constructor from a bootstrap sandbox, they can import it explicitly using Cu.importGlobalProperties. I'd be surprised if they actually used it directly from a bootstrap sandbox rather than a JSM, though.
The new Lightbeam is a web extension, the old isn't I think we are fine here. Thanks :clouserw
Comment on attachment 8896672 [details] Bug 1389868: Create bootstrap Sandboxes without indexedDB constructor by default. https://reviewboard.mozilla.org/r/167982/#review173606
Attachment #8896672 - Flags: review?(aswan) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/8cde80c96f44 Create bootstrap Sandboxes without indexedDB constructor by default. r=aswan
Status: NEW → RESOLVED
Last Resolved: 6 months ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.