Closed Bug 1761550 Opened 2 years ago Closed 2 years ago

Intermittent toolkit/components/extensions/test/mochitest/test_ext_scripting_contentScripts.html | [object Object] - Should not throw any errors

Categories

(WebExtensions :: General, defect, P5)

defect

Tracking

(firefox-esr91 unaffected, firefox98 unaffected, firefox99 unaffected, firefox100 wontfix, firefox101 fixed)

RESOLVED FIXED
101 Branch
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

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

Flags: needinfo?(tomica)
Regressed by: 1745819

Set release status flags based on info from the regressing bug 1745819

Flags: needinfo?(tomica)
See Also: → 1757007

:zombie, since you are the author of the regressor, bug 1745819, could you take a look?
For more information, please visit auto_nag documentation.

Flags: needinfo?(tomica)
Has Regression Range: --- → yes

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
Whiteboard: [stockwell needswork]
Flags: needinfo?(tomica)

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

Flags: needinfo?(tomica)

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

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?

Flags: needinfo?(tomica) → needinfo?(wdurand)

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.

Assignee: nobody → wdurand
Status: NEW → ASSIGNED

(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
Flags: needinfo?(wdurand)
Pushed by wdurand@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4a00821f64ad
Remove tab after having unloaded the extensions. r=rpl
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch
See Also: → 1768532
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: