Perma browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | Uncaught exception in test - Theme is active - threw exception: TypeError: can't access property "isActive", theme is null
Categories
(Firefox :: Messaging System, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr91 | --- | unaffected |
firefox97 | --- | fixed |
firefox98 | --- | fixed |
firefox99 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: Mardak)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: intermittent-failure, regression)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
RyanVM
:
approval-mozilla-release+
|
Details |
Filed by: imoraru [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=367015082&repo=mozilla-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/dAwQ3tgQQRCP-M64vpu9_Q/runs/0/artifacts/public/logs/live_backing.log
[task 2022-02-08T00:08:27.783Z] 00:08:27 INFO - TEST-START | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js
[task 2022-02-08T00:08:28.439Z] 00:08:28 INFO - TEST-INFO | started process screencapture
[task 2022-02-08T00:08:28.554Z] 00:08:28 INFO - TEST-INFO | screencapture: exit 0
[task 2022-02-08T00:08:28.555Z] 00:08:28 INFO - Buffered messages logged at 00:08:27
[task 2022-02-08T00:08:28.555Z] 00:08:28 INFO - Entering test bound open_close_dialog
[task 2022-02-08T00:08:28.555Z] 00:08:28 INFO - TEST-PASS | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | Upgrade dialog opened and closed - true == true -
[task 2022-02-08T00:08:28.556Z] 00:08:28 INFO - Leaving test bound open_close_dialog
[task 2022-02-08T00:08:28.556Z] 00:08:28 INFO - Entering test bound double_click
[task 2022-02-08T00:08:28.557Z] 00:08:28 INFO - TEST-PASS | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | Incorrectly handling clicks would have triggered exception - true == true -
[task 2022-02-08T00:08:28.557Z] 00:08:28 INFO - Leaving test bound double_click
[task 2022-02-08T00:08:28.558Z] 00:08:28 INFO - Entering test bound theme_change
[task 2022-02-08T00:08:28.558Z] 00:08:28 INFO - Buffered messages finished
[task 2022-02-08T00:08:28.559Z] 00:08:28 INFO - TEST-UNEXPECTED-FAIL | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | Uncaught exception in test - Theme is active - threw exception: TypeError: can't access property "isActive", theme is null
[task 2022-02-08T00:08:28.559Z] 00:08:28 INFO - Not taking screenshot here: see the one that was previously logged
[task 2022-02-08T00:08:28.560Z] 00:08:28 INFO - TEST-UNEXPECTED-FAIL | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | A promise chain failed to handle a rejection: can't access property "enable" of null - stack: enableTheme@chrome://browser/content/upgradeDialog.js:93:23
[task 2022-02-08T00:08:28.560Z] 00:08:28 INFO - Rejection date: Tue Feb 08 2022 00:08:28 GMT+0000 (Greenwich Mean Time) - false == true - JS frame :: resource://testing-common/PromiseTestUtils.jsm :: assertNoUncaughtRejections :: line 271
[task 2022-02-08T00:08:28.560Z] 00:08:28 INFO - Stack trace:
[task 2022-02-08T00:08:28.560Z] 00:08:28 INFO - resource://testing-common/PromiseTestUtils.jsm:assertNoUncaughtRejections:271
[task 2022-02-08T00:08:28.560Z] 00:08:28 INFO - chrome://mochikit/content/browser-test.js:handleTask:1017
[task 2022-02-08T00:08:28.561Z] 00:08:28 INFO - Leaving test bound theme_change
[task 2022-02-08T00:08:28.561Z] 00:08:28 INFO - Entering test bound keyboard_focus_okay
[task 2022-02-08T00:08:28.561Z] 00:08:28 INFO - GECKO(3851) | JavaScript error: chrome://browser/content/upgradeDialog.js, line 93: TypeError: can't access property "enable" of null
[task 2022-02-08T00:08:28.562Z] 00:08:28 INFO - Console message: [JavaScript Error: "TypeError: can't access property "enable" of null" {file: "chrome://browser/content/upgradeDialog.js" line: 93}]
[task 2022-02-08T00:08:28.957Z] 00:08:28 INFO - GECKO(3851) | JavaScript error: chrome://browser/content/upgradeDialog.js, line 93: TypeError: can't access property "enable" of null
[task 2022-02-08T00:08:28.958Z] 00:08:28 INFO - Console message: [JavaScript Error: "TypeError: can't access property "enable" of null" {file: "chrome://browser/content/upgradeDialog.js" line: 93}]
[task 2022-02-08T00:08:56.060Z] 00:08:56 INFO - GECKO(3851) | 2022-02-08 00:08:56.059 firefox[3851:72967] Persistent UI failed to open file file:///Users/cltbld/Library/Saved%20Application%20State/org.mozilla.nightly.savedState/window_1.data: No such file or directory (2)
[task 2022-02-08T00:09:12.976Z] 00:09:12 INFO - Not taking screenshot here: see the one that was previously logged
[task 2022-02-08T00:09:12.977Z] 00:09:12 INFO - TEST-UNEXPECTED-FAIL | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | Test timed out -
[task 2022-02-08T00:09:12.977Z] 00:09:12 INFO - Not taking screenshot here: see the one that was previously logged
[task 2022-02-08T00:09:12.978Z] 00:09:12 INFO - TEST-UNEXPECTED-FAIL | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | Uncaught exception received from previously timed out test - subdialog-loaded observer not removed before the end of test
[task 2022-02-08T00:09:12.978Z] 00:09:12 INFO - Entering test bound keep_home
[task 2022-02-08T00:09:12.979Z] 00:09:12 INFO - Not taking screenshot here: see the one that was previously logged
[task 2022-02-08T00:09:12.979Z] 00:09:12 INFO - TEST-UNEXPECTED-FAIL | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | A promise chain failed to handle a rejection: can't access property "enable" of null - stack: enableTheme@chrome://browser/content/upgradeDialog.js:93:23
[task 2022-02-08T00:09:12.980Z] 00:09:12 INFO - Rejection date: Tue Feb 08 2022 00:08:28 GMT+0000 (Greenwich Mean Time) - false == true - JS frame :: resource://testing-common/PromiseTestUtils.jsm :: assertNoUncaughtRejections :: line 271
[task 2022-02-08T00:09:12.980Z] 00:09:12 INFO - Stack trace:
[task 2022-02-08T00:09:12.980Z] 00:09:12 INFO - resource://testing-common/PromiseTestUtils.jsm:assertNoUncaughtRejections:271
[task 2022-02-08T00:09:12.980Z] 00:09:12 INFO - chrome://mochikit/content/browser-test.js:nextTest:625
[task 2022-02-08T00:09:12.980Z] 00:09:12 INFO - GECKO(3851) | MEMORY STAT | vsize 7883MB | residentFast 293MB | heapAllocated 119MB
[task 2022-02-08T00:09:12.981Z] 00:09:12 INFO - TEST-OK | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | took 45198ms
[task 2022-02-08T00:09:12.990Z] 00:09:12 INFO - GECKO(3851) | JavaScript error: chrome://browser/content/tabbrowser-tabs.js, line 1131: TypeError: can't access property "closing", selectedTab is null
[task 2022-02-08T00:09:12.991Z] 00:09:12 INFO - Console message: [JavaScript Error: "TypeError: can't access property "closing", selectedTab is null" {file: "chrome://browser/content/tabbrowser-tabs.js" line: 1131}]
[task 2022-02-08T00:15:23.046Z] 00:15:23 INFO - Buffered messages finished
Comment 1•3 years ago
|
||
This started to appear on this merge on m-c after that, the same failure started to appear on autoland.
Something strange is happening because when I added the job on the previous merge(the job was green and this test pass), the job failed with the same failure line. https://treeherder.mozilla.org/jobs?repo=mozilla-central&group_state=expanded&resultStatus=retry%2Cpending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception%2Crunnable&searchStr=os%2Cx%2C10.15%2Cwebrender%2Cshippable%2Copt%2Cmochitests%2Cwith%2Cfission%2Cenabled%2Ctest-macosx1015-64-shippable-qr%2Fopt-mochitest-browser-chrome-fis-e10s%2Cbc5&selectedTaskRun=O2CVSVeRRFmdhE8ejTKNLQ.0
Comment 2•3 years ago
|
||
Hi Dave! Can you please take a look at this? Maybe you could find the problem or what triggered this behaviour.
Reporter | ||
Updated•3 years ago
|
Assignee | ||
Comment 3•3 years ago
|
||
The test uses a theme that expired on a specific date (February 8th)
https://searchfox.org/mozilla-central/rev/2a0b0ababd4541ecffb74cbe0820a9d0a25da636/browser/components/tests/browser/browser_browserGlue_upgradeDialog.js#32
https://searchfox.org/mozilla-central/rev/2a0b0ababd4541ecffb74cbe0820a9d0a25da636/browser/themes/BuiltInThemeConfig.jsm#215
Comment 5•3 years ago
|
||
Test got temporary disabled on all platforms to reduce noise in trees.
Updated•3 years ago
|
Comment 6•3 years ago
|
||
bugherder |
Assignee | ||
Comment 7•3 years ago
|
||
Silently ignore themes that are expired and can't be enabled.
Assignee | ||
Updated•3 years ago
|
Updated•3 years ago
|
Comment hidden (Intermittent Failures Robot) |
Comment 10•3 years ago
|
||
bugherder |
Comment hidden (Intermittent Failures Robot) |
Reporter | ||
Updated•3 years ago
|
Comment 12•3 years ago
|
||
(In reply to Atila Butkovits from comment #10)
Looks like this might need to be uplifted to beta too, failures.
Updated•3 years ago
|
Assignee | ||
Comment 13•3 years ago
|
||
Comment on attachment 9262873 [details]
Bug 1754126 - Skip colorway theme test if it has already expired r=pdahiya
Beta/Release Uplift Approval Request
- User impact if declined: Basically none. Except for users who force the upgrade spotlight to show (it's off by default as the UI is outdated showing expired themes), trying to select the expired theme would not result in an browser console error message.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Skips error messages for feature that's off by default. Patch in phabricator rebased onto mozilla-beta with different browser/components/tests/browser/browser.ini (from mozilla-central)
- String changes made/needed: None
Comment hidden (Intermittent Failures Robot) |
Comment 15•3 years ago
|
||
Comment on attachment 9262873 [details]
Bug 1754126 - Skip colorway theme test if it has already expired r=pdahiya
Approved for 98 beta 4, thanks.
Comment 16•3 years ago
|
||
bugherder uplift |
Updated•3 years ago
|
Comment 17•3 years ago
|
||
Comment on attachment 9262873 [details]
Bug 1754126 - Skip colorway theme test if it has already expired r=pdahiya
We need this on Release too.
Comment hidden (Intermittent Failures Robot) |
Updated•3 years ago
|
Comment 19•3 years ago
|
||
bugherder uplift |
Description
•