Intermittent toolkit/components/extensions/test/mochitest/test_ext_scripting_contentScripts.html | [object Object] - Should not throw any errors
Categories
(WebExtensions :: General, defect, P5)
Tracking
(firefox-esr91 unaffected, firefox98 unaffected, firefox99 unaffected, firefox100 wontfix, firefox101 fixed)
Tracking | Status | |
---|---|---|
firefox-esr91 | --- | unaffected |
firefox98 | --- | unaffected |
firefox99 | --- | unaffected |
firefox100 | --- | wontfix |
firefox101 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: willdurand)
References
(Regression)
Details
(Keywords: intermittent-failure, regression, Whiteboard: [stockwell unknown])
Attachments
(1 file)
Filed by: mlaza [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=372381281&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/a-MWvuFIRyO12L7hRPZM4g/runs/0/artifacts/public/logs/live_backing.log
[task 2022-03-25T19:50:02.762Z] 19:50:02 INFO - TEST-PASS | toolkit/components/extensions/test/mochitest/test_ext_scripting_contentScripts.html | got expected script results[{"file":"script-1.js","value":"file contains iframe"},{"file":"script-2.js","value":"file contains iframe"},{"file":"script-2.js","value":"file contains img"},{"file":"script-3.js","value":"file contains img"},{"file":"script-4-2.js","value":"SCRIPT_4_INJECTED"},{"file":"script-5.js","value":"file contains iframe"}]
[task 2022-03-25T19:50:02.762Z] 19:50:02 INFO - add_task | Leaving test test_registerContentScripts_js
[task 2022-03-25T19:50:02.763Z] 19:50:02 INFO - add_task | Entering test test_registerContentScripts_are_not_unregistered
[task 2022-03-25T19:50:02.763Z] 19:50:02 INFO - Extension loaded
[task 2022-03-25T19:50:02.764Z] 19:50:02 INFO - Buffered messages logged at 19:50:01
[task 2022-03-25T19:50:02.764Z] 19:50:02 INFO - add_task | Leaving test test_registerContentScripts_are_not_unregistered
[task 2022-03-25T19:50:02.764Z] 19:50:02 INFO - add_task | Entering test test_scripts_dont_run_after_shutdown
[task 2022-03-25T19:50:02.765Z] 19:50:02 INFO - Extension loaded
[task 2022-03-25T19:50:02.765Z] 19:50:02 INFO - Extension loaded
[task 2022-03-25T19:50:02.766Z] 19:50:02 INFO - Buffered messages logged at 19:50:02
[task 2022-03-25T19:50:02.766Z] 19:50:02 INFO - add_task | Leaving test test_scripts_dont_run_after_shutdown
[task 2022-03-25T19:50:02.767Z] 19:50:02 INFO - add_task | Entering test test_registerContentScripts_with_wrong_matches
[task 2022-03-25T19:50:02.767Z] 19:50:02 INFO - Extension loaded
[task 2022-03-25T19:50:02.768Z] 19:50:02 INFO - Extension loaded
[task 2022-03-25T19:50:02.768Z] 19:50:02 INFO - Buffered messages finished
[task 2022-03-25T19:50:02.769Z] 19:50:02 INFO - TEST-UNEXPECTED-FAIL | toolkit/components/extensions/test/mochitest/test_ext_scripting_contentScripts.html | [object Object] - Should not throw any errors
[task 2022-03-25T19:50:02.769Z] 19:50:02 INFO - add_task/nextTick/<@SimpleTest/SimpleTest.js:2103:26
[task 2022-03-25T19:50:02.769Z] 19:50:02 INFO - GECKO(2708) | MEMORY STAT | vsize 533MB | vsizeMaxContiguous 1293MB | residentFast 70MB | heapAllocated 9MB
[task 2022-03-25T19:50:02.770Z] 19:50:02 INFO - TEST-OK | toolkit/components/extensions/test/mochitest/test_ext_scripting_contentScripts.html | took 2394ms
[task 2022-03-25T19:50:02.770Z] 19:50:02 INFO - TEST-START | toolkit/components/extensions/test/mochitest/test_ext_scripting_executeScript.html
Comment 1•2 years ago
|
||
Hi Tomislav! Can you please take a look at this? This seems to be caused by your recent changes from Bug 1745819.
Thank you!
Backfill range
Comment 2•2 years ago
|
||
Set release status flags based on info from the regressing bug 1745819
Comment 3•2 years ago
|
||
:zombie, since you are the author of the regressor, bug 1745819, could you take a look?
For more information, please visit auto_nag documentation.
Comment hidden (Intermittent Failures Robot) |
Updated•2 years ago
|
Comment 5•2 years ago
|
||
Update:
There have been 34 failures within the last 7 days:
- 5 failures on Windows 10 x86 2004 WebRender opt
- 5 failures on Windows 10 x86 2004 WebRender Shippable opt
- 13 failures on Windows 10 x64 2004 WebRender opt
- 11 failures on Windows 10 x64 2004 WebRender Shippable opt
Recent failure: https://treeherder.mozilla.org/logviewer?job_id=373239786&repo=autoland&lineNumber=14036
[task 2022-04-03T23:23:26.338Z] 23:23:26 INFO - TEST-PASS | toolkit/components/extensions/test/mochitest/test_ext_scripting_contentScripts.html | got expected script results[{"file":"script-1.js","value":"file contains iframe"},{"file":"script-2.js","value":"file contains iframe"},{"file":"script-2.js","value":"file contains img"},{"file":"script-3.js","value":"file contains img"},{"file":"script-4-2.js","value":"SCRIPT_4_INJECTED"},{"file":"script-5.js","value":"file contains iframe"}]
[task 2022-04-03T23:23:26.339Z] 23:23:26 INFO - add_task | Leaving test test_registerContentScripts_js
[task 2022-04-03T23:23:26.339Z] 23:23:26 INFO - add_task | Entering test test_registerContentScripts_are_not_unregistered
[task 2022-04-03T23:23:26.339Z] 23:23:26 INFO - Extension loaded
[task 2022-04-03T23:23:26.339Z] 23:23:26 INFO - Buffered messages logged at 23:23:25
[task 2022-04-03T23:23:26.340Z] 23:23:26 INFO - add_task | Leaving test test_registerContentScripts_are_not_unregistered
[task 2022-04-03T23:23:26.340Z] 23:23:26 INFO - add_task | Entering test test_scripts_dont_run_after_shutdown
[task 2022-04-03T23:23:26.340Z] 23:23:26 INFO - Extension loaded
[task 2022-04-03T23:23:26.341Z] 23:23:26 INFO - Extension loaded
[task 2022-04-03T23:23:26.341Z] 23:23:26 INFO - add_task | Leaving test test_scripts_dont_run_after_shutdown
[task 2022-04-03T23:23:26.341Z] 23:23:26 INFO - add_task | Entering test test_registerContentScripts_with_wrong_matches
[task 2022-04-03T23:23:26.342Z] 23:23:26 INFO - Extension loaded
[task 2022-04-03T23:23:26.342Z] 23:23:26 INFO - Extension loaded
[task 2022-04-03T23:23:26.342Z] 23:23:26 INFO - Buffered messages finished
[task 2022-04-03T23:23:26.343Z] 23:23:26 INFO - TEST-UNEXPECTED-FAIL | toolkit/components/extensions/test/mochitest/test_ext_scripting_contentScripts.html | [object Object] - Should not throw any errors
[task 2022-04-03T23:23:26.343Z] 23:23:26 INFO - add_task/nextTick/<@SimpleTest/SimpleTest.js:2103:26
[task 2022-04-03T23:23:26.343Z] 23:23:26 INFO - GECKO(2984) | MEMORY STAT | vsize 2103656MB | vsizeMaxContiguous 69219190MB | residentFast 59MB | heapAllocated 8MB
[task 2022-04-03T23:23:26.344Z] 23:23:26 INFO - TEST-OK | toolkit/components/extensions/test/mochitest/test_ext_scripting_contentScripts.html | took 2074ms
Updated•2 years ago
|
Comment hidden (Intermittent Failures Robot) |
Updated•2 years ago
|
Comment 7•2 years ago
|
||
There have been 42 failures in the last 7 days.
Happens on:
- windows10-32-2004-qr opt
- windows10-32-2004-shippable-qr opt
- windows10-64-2004-qr opt
- windows10-64-2004-shippable-qr opt.
Recent failure log: https://treeherder.mozilla.org/logviewer?job_id=373979712&repo=autoland&lineNumber=12265
Tomislav, could you look into this?
Thanks
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 10•2 years ago
|
||
There have been 36 total failures in the last 7 days, recent failure log.
Affected platforms are:
- windows10-32-2004-qr
- windows10-32-2004-shippable-qr
- windows10-64-2004-qr
- windows10-64-2004-shippable-qr
Comment hidden (Intermittent Failures Robot) |
Comment 12•2 years ago
|
||
The test seems to consistently be stuck at test_registerContentScripts_with_wrong_matches
, most likely around https://searchfox.org/mozilla-central/rev/2e328e8040a4b4419647aabdc536a3f61bafc384/toolkit/components/extensions/test/mochitest/test_ext_scripting_contentScripts.html#819
Will, could you take a look when you have the chance?
Assignee | ||
Comment 13•2 years ago
|
||
Initially, I thought the bug was related to a missing extension.id
in
ExtensionProcessScript.jsm
1 but that isn't always reproducible.
There is possibly an issue there but I don't think it is the root cause
of the intermittent here (I would think that other OSes would be affected,
w hich isn't the case according to 2).
@rpl and I investigated and we noticed that the mochitest error reported
didn't contain a meaningful error:
[object Object] - Should not throw any errors
Once we fixed the test framework output, we got:
FAIL {"operation":"remove","path":"C:\\Users\\William\\AppData\\Local\\Temp\\generated-extension-23.xpi","winLastError":5,"stack":"","fileName":"(unknown module)"} - Should not throw any errors
This is why this patch attempts to fix the intermittent by unloading the
extensions first and then removing the tab. As @rpl said:
That may actually make sense, maybe removing the tab right before the
unload prevented the extension to coordinate with the child process and
make sure we flushed the cache, and that process may not have been yet
fully destroyed.
Updated•2 years ago
|
Assignee | ||
Comment 14•2 years ago
|
||
(In reply to William Durand [:willdurand] from comment #13)
Initially, I thought the bug was related to a missing
extension.id
in
ExtensionProcessScript.jsm
[1] but that isn't always reproducible.
About that, this error was thrown while executing test_registerContentScripts_with_wrong_matches
on Windows (but not all the time) and the stack trace was:
0:45.78 GECKO(1624) Console message: [JavaScript Error: "can't access property "id", extension is null" {file: "resource://gre/modules/ExtensionProcessScript.jsm" line: 124}]
0:45.78 GECKO(1624) initExtensionPolicy@resource://gre/modules/ExtensionProcessScript.jsm:124:18
0:45.78 GECKO(1624) initExtension@resource://gre/modules/ExtensionProcessScript.jsm:211:23
0:45.78 GECKO(1624) receiveMessage@resource://gre/modules/ExtensionProcessScript.jsm:229:16
0:45.78 GECKO(1624) MessageListener.receiveMessage*init@resource://gre/modules/ExtensionProcessScript.jsm:75:19
0:45.78 GECKO(1624) @resource://gre/modules/ExtensionProcessScript.jsm:472:18
0:45.78 GECKO(1624) @resource://gre/modules/extensionProcessScriptLoader.js:6:13
Comment 15•2 years ago
|
||
Pushed by wdurand@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4a00821f64ad Remove tab after having unloaded the extensions. r=rpl
Comment 16•2 years ago
|
||
bugherder |
Assignee | ||
Updated•2 years ago
|
Comment hidden (Intermittent Failures Robot) |
Updated•2 years ago
|
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Description
•