Closed
Bug 1685424
Opened 4 years ago
Closed 4 years ago
Re-enable toolkit/components/extensions/test/mochitest/test_ext_contentscript_activeTab.html test for Fission
Categories
(WebExtensions :: General, task, P2)
WebExtensions
General
Tracking
(Fission Milestone:M7a, firefox-esr78 disabled, firefox86 disabled, firefox87 disabled, firefox88 disabled, firefox89 fixed)
People
(Reporter: cpeterson, Assigned: zombie)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Tom, is this test still failing with Fission? Does fixing and re-enabling this test need to block shipping Fission MVP?
This toolkit/components/extensions/test/mochitest/test_ext_contentscript_activeTab.html test is currently skipped for Fission (by bug 1646573):
[test_ext_contentscript_activeTab.html]
skip-if = os == 'android' || fission
Flags: needinfo?(tomica)
Assignee | ||
Comment 1•4 years ago
|
||
Yes, this functionality (not the test) should probably block MVP.
Looking into.
Assignee: nobody → tomica
Severity: -- → N/A
Flags: needinfo?(tomica)
Priority: -- → P2
Reporter | ||
Updated•4 years ago
|
Blocks: fission-tests
Reporter | ||
Comment 2•4 years ago
|
||
zombie recommends that this test bug still block Fission M7.
status-firefox86:
--- → disabled
status-firefox87:
--- → disabled
status-firefox88:
--- → affected
status-firefox-esr78:
--- → disabled
Assignee | ||
Comment 3•4 years ago
|
||
The webNavigation.getAllFrames
functionality needed for this test is being fixed in bug 1698398.
Additionally, this also depends webNavigation.onCompleted
event on iframes, which is being fixed in bug 1581859.
Updated•4 years ago
|
Fission Milestone: M7 → M7a
Assignee | ||
Comment 4•4 years ago
|
||
Pushed by tjovanovic@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/50aeb68f1f5a
Enable test_ext_contentscript_activeTab test with fission r=robwu
Comment 6•4 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox89:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
Updated•4 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•