Closed Bug 1945179 Opened 11 months ago Closed 11 months ago

Perma devtools/client/storage/test/browser_storage_indexeddb_navigation.js | Uncaught exception received from previously timed out test bound - AbortError: Actor 'SpecialPowers' destroyed before query 'Spawn' when Gecko 136 merges to beta on 2025-02-03

Categories

(DevTools :: Storage Inspector, defect)

Product:
DevTools
Component:
Storage Inspector
Type:
defect

Tracking

(firefox-esr115 unaffected, firefox-esr128 unaffected, firefox134 unaffected, firefox135 unaffected, firefox136+ verified)

Status:
VERIFIED FIXED
Milestone:
136 Branch
Tracking Flags:
Tracking Status
firefox-esr115 --- unaffected
firefox-esr128 --- unaffected
firefox134 --- unaffected
firefox135 --- unaffected
firefox136 + verified

People

(Reporter: SerbanS, Assigned: jdescottes)

References

(Regression)

Details

(Keywords: perf-alert, regression)

Attachments

(2 files)

Bug 1945179 - [webcompat] Convert content_scripts to an array if needed
48 bytes, text/x-phabricator-request
Details | Review
Bug 1945179 - [webcompat] Bump version number to 136.7.0
48 bytes, text/x-phabricator-request
Details | Review

Central-as-beta-simulation

How to run these simulations

Failure log

[task 2025-01-31T13:35:10.390Z] 13:35:10     INFO - TEST-PASS | devtools/client/storage/test/browser_storage_indexeddb_navigation.js | indexedDB > https://example.com is in the tree - 
[task 2025-01-31T13:35:10.390Z] 13:35:10     INFO - Selecting "indexedDB,https://example.com,db (default),store".
[task 2025-01-31T13:35:10.391Z] 13:35:10     INFO - TEST-PASS | devtools/client/storage/test/browser_storage_indexeddb_navigation.js | Table row has an entry for: lorem with value: {"key":"lorem","value":"ipsum"} - 
[task 2025-01-31T13:35:10.391Z] 13:35:10     INFO - Removing database…
[task 2025-01-31T13:35:10.392Z] 13:35:10     INFO - Buffered messages logged at 13:33:40
[task 2025-01-31T13:35:10.392Z] 13:35:10     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 2
[task 2025-01-31T13:35:10.392Z] 13:35:10     INFO - Buffered messages logged at 13:34:25
[task 2025-01-31T13:35:10.393Z] 13:35:10     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 1
[task 2025-01-31T13:35:10.393Z] 13:35:10     INFO - Buffered messages finished
[task 2025-01-31T13:35:10.394Z] 13:35:10     INFO - TEST-UNEXPECTED-FAIL | devtools/client/storage/test/browser_storage_indexeddb_navigation.js | Test timed out - 
[task 2025-01-31T13:35:10.394Z] 13:35:10     INFO - GECKO(2873) | Completed ShutdownLeaks collections in process 2873
[task 2025-01-31T13:35:10.395Z] 13:35:10     INFO - TEST-START | Shutdown
[task 2025-01-31T13:35:10.395Z] 13:35:10     INFO - Browser Chrome Test Summary
[task 2025-01-31T13:35:10.396Z] 13:35:10     INFO - Passed:  956
[task 2025-01-31T13:35:10.396Z] 13:35:10     INFO - Failed:  1
[task 2025-01-31T13:35:10.397Z] 13:35:10     INFO - Todo:    0
[task 2025-01-31T13:35:10.397Z] 13:35:10     INFO - Mode:    e10s
[task 2025-01-31T13:35:10.397Z] 13:35:10     INFO - *** End BrowserChrome Test Results ***
[task 2025-01-31T13:35:10.398Z] 13:35:10     INFO - Not taking screenshot here: see the one that was previously logged
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - TEST-UNEXPECTED-FAIL | devtools/client/storage/test/browser_storage_indexeddb_navigation.js | Uncaught exception received from previously timed out test bound  - AbortError: Actor 'SpecialPowers' destroyed before query 'Spawn' was resolved
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - Stack trace:
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - unregisterActor@resource://testing-common/SpecialPowersParent.sys.mjs:200:17
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - onShutdown@jar:file:///var/folders/4q/yq2hzn2s72qbfhztbdx4fpww0000jb/T/tmp97dqvebn.mozrunner/extensions/special-powers@mozilla.org.xpi!/api.js:64:25
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - ExtensionAPI/<@resource://gre/modules/ExtensionCommon.sys.mjs:365:14
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - wrapper@resource://gre/modules/ExtensionCommon.sys.mjs:309:14
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - emit@resource://gre/modules/ExtensionCommon.sys.mjs:336:32
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - emit@resource://gre/modules/Extension.sys.mjs:3310:25
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - shutdown@resource://gre/modules/Extension.sys.mjs:4147:10
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - shutdown@resource://gre/modules/Extension.sys.mjs:3005:39
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - callBootstrapMethod@resource://gre/modules/addons/XPIProvider.sys.mjs:1854:33
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - _shutdown@resource://gre/modules/addons/XPIProvider.sys.mjs:1997:17
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - async*shutdown@resource://gre/modules/addons/XPIProvider.sys.mjs:1990:33
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - startup/<@resource://gre/modules/addons/XPIProvider.sys.mjs:2664:33
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - async*trigger@resource://gre/modules/AsyncShutdown.sys.mjs:754:23
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - _wait@resource://gre/modules/AsyncShutdown.sys.mjs:914:15
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - wait@resource://gre/modules/AsyncShutdown.sys.mjs:895:34
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - observe@resource://gre/modules/AsyncShutdown.sys.mjs:554:10
[task 2025-01-31T13:35:10.401Z] 13:35:10     INFO - testsFinished@chrome://mochikit/content/browser-harness.xhtml:340:28
[task 2025-01-31T13:35:10.402Z] 13:35:10     INFO - Tester_finish@chrome://mochikit/content/browser-test.js:556:10
[task 2025-01-31T13:35:10.402Z] 13:35:10     INFO - timeoutFn@chrome://mochikit/content/browser-test.js:1455:20
[task 2025-01-31T13:35:10.402Z] 13:35:10     INFO - setTimeout handler*SimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:922:41
[task 2025-01-31T13:35:10.402Z] 13:35:10     INFO - timeoutFn@chrome://mochikit/content/browser-test.js:1413:52
[task 2025-01-31T13:35:10.402Z] 13:35:10     INFO - setTimeout handler*SimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:922:41
[task 2025-01-31T13:35:10.402Z] 13:35:10     INFO - timeoutFn@chrome://mochikit/content/browser-test.js:1413:52
[task 2025-01-31T13:35:10.402Z] 13:35:10     INFO - setTimeout handler*Tester_execTest@chrome://mochikit/content/browser-test.js:1394:45
[task 2025-01-31T13:35:10.402Z] 13:35:10     INFO - nextTest/<@chrome://mochikit/content/browser-test.js:1136:14
[task 2025-01-31T13:35:10.402Z] 13:35:10     INFO - SimpleTest.waitForFocus/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:1058:13
[task 2025-01-31T13:35:10.403Z] 13:35:10     INFO - GECKO(2873) | 1738330510334	Marionette	TRACE	Received observer notification quit-application
[task 2025-01-31T13:35:10.403Z] 13:35:10     INFO - GECKO(2873) | 1738330510334	Marionette	INFO	Stopped listening on port 2828
[task 2025-01-31T13:35:10.404Z] 13:35:10     INFO - GECKO(2873) | 1738330510335	Marionette	DEBUG	Marionette stopped listening
[task 2025-01-31T13:35:10.406Z] 13:35:10     INFO - GECKO(2873) | JavaScript error: , line 0: NotFoundError: No such JSProcessActor 'DevToolsProcess'
[task 2025-01-31T13:35:10.407Z] 13:35:10     INFO - GECKO(2873) | JavaScript error: , line 0: NotFoundError: No such JSProcessActor 'DevToolsProcess'
[task 2025-01-31T13:35:10.410Z] 13:35:10     INFO - GECKO(2873) | JavaScript error: , line 0: NotFoundError: No such JSProcessActor 'DevToolsProcess'
[task 2025-01-31T13:35:10.592Z] 13:35:10     INFO - TEST-INFO | Main app process: exit 0
[task 2025-01-31T13:35:10.593Z] 13:35:10     INFO - TEST-UNEXPECTED-FAIL | devtools/client/storage/test/browser_storage_indexeddb_navigation.js | Application shut down (without crashing) in the middle of a test!
[task 2025-01-31T13:35:10.593Z] 13:35:10     INFO - TEST-INFO took 135122ms

Hi Julian! Could you please take a look at this? It permafailing starting with this beta simulation and I'm not sure if Bug 1944361 has something to do with it. Maybe you could tell us better if it's another changeset that started this.

Thank you!

Flags: needinfo?(jdescottes)

Hi! 99% sure this is not related to Bug 1944361, which should only impact network monitor tests.

Looks like we fail to remove an indexeddb database. I would rather look for Nightly only prefs impacting the storage area?

Flags: needinfo?(jdescottes)

Hi Alexandre! Do you have any idea about this? You seem to be the only one who added devtools related changes from the last beta simulation beside @jdescottes . Can this be a fallout from Bug 1941886? Thank you!

Flags: needinfo?(poirot.alex)

Test seems to fail on Beta only after Bug 1936031 only, I have no idea why.

During the test, in the browser console I see many errors such as

Error enabling injection for undefined TypeError: content_scripts.entries is not a function
    _buildContentScriptRegistrations moz-extension://8761b898-aded-4752-a331-b951b17208af/lib/injections.js:198
    _enableInjectionNow moz-extension://8761b898-aded-4752-a331-b951b17208af/lib/injections.js:145
    _enableInjectionsNow moz-extension://8761b898-aded-4752-a331-b951b17208af/lib/injections.js:93
    _enablingOrDisablingOperationInProgress moz-extension://8761b898-aded-4752-a331-b951b17208af/lib/injections.js:62
    enableInjections moz-extension://8761b898-aded-4752-a331-b951b17208af/lib/injections.js:61
    checkInjectionPref moz-extension://8761b898-aded-4752-a331-b951b17208af/lib/injections.js:45
    checkInjectionPref moz-extension://8761b898-aded-4752-a331-b951b17208af/lib/injections.js:39
    bootup moz-extension://8761b898-aded-4752-a331-b951b17208af/lib/injections.js:33
    checkInjectionPref moz-extension://8761b898-aded-4752-a331-b951b17208af/lib/injections.js:41

Tom any idea?

Flags: needinfo?(poirot.alex) → needinfo?(twisniewski)
Keywords: regression
Regressed by: 1936031

Looks like an isArray check was removed and makes the whole contentScript handling fail. Still not sure why that only fails in a devtools browser chrome mochitest in beta though :)

content_scripts is always an object in the current interventions.json
Need to convert to an array to be able to use entries().

Assignee: nobody → jdescottes
Status: NEW → ASSIGNED

Patch is working in latest beta-simulation

Pushed by csabou@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5afb46d0b2a0 [webcompat] Convert content_scripts to an array if needed r=webcompat-reviewers,twisniewski
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bac4912a9687 [webcompat] Bump version number to 136.7.0 r=twisniewski,webcompat-reviewers

https://hg.mozilla.org/mozilla-central/rev/5afb46d0b2a0
https://hg.mozilla.org/mozilla-central/rev/bac4912a9687

Status: ASSIGNED → RESOLVED
Closed: 11 months ago
status-firefox136: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch

The patch is verified in today's beta simulation.

Status: RESOLVED → VERIFIED
status-firefox136: fixed → verified
Flags: needinfo?(twisniewski)
Regressions: 1946058

Perfherder has detected a browsertime performance change from push 5afb46d0b2a001e5bc64b0b37293b726361e03ea.

Improvements:

Ratio Test Platform Options Absolute values (old vs new) Performance Profiles
4% cnn PerceptualSpeedIndex linux1804-64-shippable-qr cold fission webrender 990.39 -> 946.88 Before/After
4% cnn SpeedIndex linux1804-64-shippable-qr cold fission webrender 996.19 -> 953.55 Before/After
4% cnn ContentfulSpeedIndex linux1804-64-shippable-qr cold fission webrender 992.74 -> 952.95 Before/After
4% cnn fcp linux1804-64-shippable-qr cold fission webrender 937.35 -> 900.68 Before/After
4% cnn FirstVisualChange linux1804-64-shippable-qr cold fission webrender 958.97 -> 923.32 Before/After

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a sheriff to do that for you.

You can run these tests on try with ./mach try perf --alert 43639

For more information on performance sheriffing please see our FAQ.

Keywords: perf-alert

(In reply to Florin Bilt from comment #14)

Perfherder has detected a browsertime performance change from push 5afb46d0b2a001e5bc64b0b37293b726361e03ea.

Same as for Bug 1946058, even if this one is an improvement we are just returning to the baseline

Blocks: 1946368

(In reply to Pulsebot from comment #9)

Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5afb46d0b2a0
[webcompat] Convert content_scripts to an array if needed
r=webcompat-reviewers,twisniewski

Perfherder has detected a devtools performance change from push 5afb46d0b2a001e5bc64b0b37293b726361e03ea.

Improvements:

Ratio Test Platform Options Absolute values (old vs new)
5% damp console.log-in-loop-content-process-nan windows11-64-shippable-qr e10s fission stylo webrender 24.78 -> 23.42

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a sheriff to do that for you.

You can run these tests on try with ./mach try perf --alert 43640

For more information on performance sheriffing please see our FAQ.

Regressions: 1947423
Regressions: 1947726
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: