Closed
Bug 1357487
Opened 8 years ago
Closed 6 years ago
Turn on OOP extensions by default on Linux
Categories
(WebExtensions :: General, defect, P2)
WebExtensions
General
Tracking
(Performance Impact:low, relnote-firefox 63+, firefox63+ fixed)
webextensions | ? |
People
(Reporter: kmag, Assigned: kmag)
References
(Depends on 1 open bug, Blocks 3 open bugs, Regressed 1 open bug)
Details
(Keywords: perf, Whiteboard: triaged)
Attachments
(1 file)
No description provided.
Assignee | ||
Updated•8 years ago
|
webextensions: --- → ?
Comment 1•8 years ago
|
||
waiting if jim is getting to popover stuff
Priority: -- → P2
Whiteboard: triaged
Comment 2•8 years ago
|
||
Marking as quantum flow p1 because this is about the ability to run webextensions in a seperate process.
Whiteboard: triaged → triaged[qf:p1]
Comment 3•8 years ago
|
||
I went through the Quantum Flow release criteria and I don't believe Linux is a blocker for that. So taking off the :p1 designation.
Whiteboard: triaged[qf:p1] → triaged[qf]
Updated•7 years ago
|
Whiteboard: triaged[qf] → triaged[qf:p3]
Comment 4•7 years ago
|
||
Nightly 56 x64 20170706100310 @ Debian Testing (Linux 4.9.0-3-amd64, Radeon RX480)
extensions.webextensions.remote;true works now (I have stylo enabled), but panels have fat black (webrender:red) borders, which is okay for testing
dom.ipc.processCount.extension;2 does not work, no addons are loaded then, I will wait patiently
Updated•7 years ago
|
status-firefox57:
--- → wontfix
Comment 6•7 years ago
|
||
There is a problem with OOP Webextensions inside locationbar if the otherwise perfectly working GPU Process is enabled (for example for having some more stability while testing WebRender).
This might not be a blocker for this bug because the GPU Process is currently disabled by default on Linux.
Updated•6 years ago
|
Product: Toolkit → WebExtensions
Updated•6 years ago
|
status-firefox57:
wontfix → ---
Comment hidden (mozreview-request) |
Comment 11•6 years ago
|
||
mozreview-review |
Comment on attachment 8994917 [details]
Bug 1357487: Enable OOP extensions by default on all platforms.
https://reviewboard.mozilla.org/r/259428/#review266550
hooray!
Attachment #8994917 -
Flags: review?(aswan) → review+
Assignee | ||
Comment 12•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/e5423d29aaf0b711e02b68b7340a2297dd6bfe16
Bug 1357487: Enable OOP extensions by default on all platforms. r=aswan
Comment 13•6 years ago
|
||
Backed out changeset e5423d29aaf0 (Bug 1357487) for reftest failures in file:///Z:/task_1532567375/build/tests/reftest/tests/layout/reftests/css-blending/blend-difference-stacking.html on a CLOSED TREE
Failure: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-classifiedState=unclassified&fromchange=4e08eb5f94e73e34f331f8dce08f77e9953d525a&selectedJob=190152821
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=190152821&repo=mozilla-inbound&lineNumber=38618
01:37:09 ERROR - REFTEST TEST-UNEXPECTED-FAIL | file:///Z:/task_1532567375/build/tests/reftest/tests/layout/reftests/css-blending/blend-difference-stacking.html == file:///Z:/task_1532567375/build/tests/reftest/tests/layout/reftests/css-blending/blend-difference-stacking-ref.html | image comparison, max difference: 221, number of differing pixels: 2
Flags: needinfo?(kmaglione+bmo)
Comment 14•6 years ago
|
||
Backout by shindli@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/95039d4a54ce
Backed out changeset e5423d29aaf0 for reftest failures in file:///Z:/task_1532567375/build/tests/reftest/tests/layout/reftests/css-blending/blend-difference-stacking.html on a CLOSED TREE
Comment 15•6 years ago
|
||
Sorry,
This is the correct failure for which this bug was backed out - https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&fromchange=4e08eb5f94e73e34f331f8dce08f77e9953d525a&selectedJob=190146187
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=190146187&repo=mozilla-inbound&lineNumber=2200
17:41:59 INFO - TEST-UNEXPECTED-FAIL | browser/components/extensions/test/browser/browser_ext_browserAction_popup_preload.js | Test timed out -
17:41:59 INFO - Not taking screenshot here: see the one that was previously logged
17:41:59 INFO - TEST-UNEXPECTED-FAIL | browser/components/extensions/test/browser/browser_ext_browserAction_popup_preload.js | no tasks awaiting on messages - Got ["tabTitle"], expected []
17:41:59 INFO - Stack trace:
17:41:59 INFO - chrome://mochikit/content/browser-test.js:test_is:1305
17:41:59 INFO - chrome://mochikit/content/tests/SimpleTest/ExtensionTestUtils.js:ExtensionTestUtils.loadExtension/<:31
17:41:59 INFO - chrome://mochikit/content/browser-test.js:nextTest:705
17:41:59 INFO - chrome://mochikit/content/browser-test.js:timeoutFn:1200
17:41:59 INFO - setTimeout handler*chrome://mochikit/content/browser-test.js:Tester_execTest:1162
17:41:59 INFO - chrome://mochikit/content/browser-test.js:nextTest/<:996
17:41:59 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:795
17:41:59 INFO - Not taking screenshot here: see the one that was previously logged
17:41:59 INFO - TEST-UNEXPECTED-FAIL | browser/components/extensions/test/browser/browser_ext_browserAction_popup_preload.js | Extension left running at test shutdown -
17:41:59 INFO - Stack trace:
17:41:59 INFO - chrome://mochikit/content/tests/SimpleTest/ExtensionTestUtils.js:ExtensionTestUtils.loadExtension/<:109
17:41:59 INFO - chrome://mochikit/content/browser-test.js:nextTest:705
17:41:59 INFO - chrome://mochikit/content/browser-test.js:timeoutFn:1200
17:41:59 INFO - setTimeout handler*chrome://mochikit/content/browser-test.js:Tester_execTest:1162
17:41:59 INFO - chrome://mochikit/content/browser-test.js:nextTest/<:996
17:41:59 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:795
17:41:59 INFO - GECKO(1969) | MEMORY STAT | vsize 4559MB | residentFast 514MB | heapAllocated 116MB
17:41:59 INFO - TEST-OK | browser/components/extensions/test/browser/browser_ext_browserAction_popup_preload.js | took 45073ms
17:41:59 INFO - checking window state
17:41:59 INFO - Not taking screenshot here: see the one that was previously logged
17:41:59 INFO - TEST-UNEXPECTED-FAIL | browser/components/extensions/test/browser/browser_ext_browserAction_popup_preload.js | Found a browser window after previous test timed out -
Assignee | ||
Comment 16•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ef1550969466fbe9049ee922735f020af82f66d6
Bug 1357487: Enable OOP extensions by default on all platforms. r=aswan
Comment 17•6 years ago
|
||
awesome bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Updated•6 years ago
|
Comment 18•6 years ago
|
||
We mentioned that change in 61 release notes for macOS, therefore I'll add a note in 63 nightly notes with the same wording:
WebExtensions now run in their own process on Linux
tracking-firefox63:
--- → +
relnote-firefox:
--- → 63+
Comment 19•6 years ago
|
||
Backed out changeset ef1550969466 (Bug 1357487) caused perf regressions & author didn't justified them.
Push with issues: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=ef1550969466fbe9049ee922735f020af82f66d6
Backout link: https://hg.mozilla.org/integration/mozilla-inbound/rev/ed6567a175dd6a9b9de58b8867f03db7a17ee330
Updated•6 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Updated•6 years ago
|
Comment 20•6 years ago
|
||
Note about this feature disabled from nightly 63 release notes following the backout.
Assignee | ||
Comment 21•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/cb97b513e80eacbf60518002acd95f1591023bc7
Bug 1357487: Enable OOP extensions by default on all platforms. r=aswan
Comment 22•6 years ago
|
||
bugherder |
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(kmaglione+bmo)
Comment 23•6 years ago
|
||
BTW enabling extensions.webextensions.remote also disabled Firefox' JSON viewer on the manifest.json. I doubt this is intended.
See bug 1481012 for a related issue/problem with the JSON viewer for WebExtensions.
Comment 24•6 years ago
|
||
Verified using Ubuntu 18.04 and FF Nightly running on latest build as of (04/08/2018). As the feature has also been validated previously, will be closing out this bug as well.
Updated•6 years ago
|
Status: RESOLVED → VERIFIED
Updated•3 years ago
|
Performance Impact: --- → P3
Whiteboard: triaged[qf:p3] → triaged
You need to log in
before you can comment on or make changes to this bug.
Description
•