Closed Bug 1781504 Opened 3 years ago Closed 3 years ago

Intermittent devtools/client/styleeditor/test/browser_toolbox_styleeditor.js | single tracking bug

Categories

(DevTools :: Style Editor, defect, P3)

defect

Tracking

(firefox110 fixed)

RESOLVED FIXED
110 Branch
Tracking Status
firefox110 --- fixed

People

(Reporter: jmaher, Assigned: jdescottes)

References

Details

(Keywords: intermittent-failure, intermittent-testcase, Whiteboard: [stockwell unknown])

Attachments

(4 files)

No description provided.

Additional information about this bug failures and frequency patterns can be found by running: ./mach test-info failure-report --bug 1781504

Update

There have been 31 failures within the last 7 days:

  • 8 failures on Linux 18.04 x64 WebRender opt
  • 22 failures on Linux 18.04 x64 WebRender Shippable opt
  • 1 failure on Windows 10 x64 2004 WebRender opt

Recent failure log: https://treeherder.mozilla.org/logviewer?job_id=401696836&repo=mozilla-central&lineNumber=7777

[task 2023-01-07T23:27:58.115Z] 23:27:58     INFO - TEST-PASS | devtools/client/styleeditor/test/browser_toolbox_styleeditor.js | Has session state - 
[task 2023-01-07T23:27:58.116Z] 23:27:58     INFO - Buffered messages logged at 23:25:01
[task 2023-01-07T23:27:58.117Z] 23:27:58     INFO - Console message: [JavaScript Warning: "debuggee 'resource://devtools/shared/loader/base-loader.js:321' would run" {file: "resource://devtools/shared/loader/builtin-modules.js" line: 105}]
[task 2023-01-07T23:27:58.119Z] 23:27:58     INFO - Console message: [JavaScript Warning: "debuggee 'resource://devtools/shared/loader/base-loader.js:321' would run" {file: "resource://devtools/shared/loader/builtin-modules.js" line: 105}]
[task 2023-01-07T23:27:58.120Z] 23:27:58     INFO - Buffered messages logged at 23:25:42
[task 2023-01-07T23:27:58.120Z] 23:27:58     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 3
[task 2023-01-07T23:27:58.121Z] 23:27:58     INFO - Buffered messages logged at 23:26:27
[task 2023-01-07T23:27:58.122Z] 23:27:58     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 2
[task 2023-01-07T23:27:58.122Z] 23:27:58     INFO - Buffered messages logged at 23:27:12
[task 2023-01-07T23:27:58.122Z] 23:27:58     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 1
[task 2023-01-07T23:27:58.123Z] 23:27:58     INFO - Buffered messages finished
[task 2023-01-07T23:27:58.124Z] 23:27:58     INFO - TEST-UNEXPECTED-FAIL | devtools/client/styleeditor/test/browser_toolbox_styleeditor.js | Test timed out - 
[task 2023-01-07T23:27:58.124Z] 23:27:58     INFO - Removing tab.
[task 2023-01-07T23:27:58.125Z] 23:27:58     INFO - Waiting for event: 'TabClose' on [object XULElement].
[task 2023-01-07T23:27:58.125Z] 23:27:58     INFO - Got event: 'TabClose' on [object XULElement].
[task 2023-01-07T23:27:58.126Z] 23:27:58     INFO - Tab removed and finished closing
[task 2023-01-07T23:27:58.128Z] 23:27:58     INFO - TEST-PASS | devtools/client/styleeditor/test/browser_toolbox_styleeditor.js | The main process DevToolsServer has no pending connection when the test ends - 
[task 2023-01-07T23:27:58.172Z] 23:27:58     INFO - GECKO(7918) | MEMORY STAT | vsize 4230MB | residentFast 750MB | heapAllocated 412MB

Julian, any chance you could help us assign the bug to someone?
Thank you.

Flags: needinfo?(jdescottes)
Whiteboard: [stockwell needswork:owner]

It seems that the browser toolbox fails to connect, and we have the following error in the logs

console.error: (new Error("Target already destroyed, unable to fetch children fronts", "resource://devtools/client/fronts/targets/target-mixin.js", 302))

But no stack trace.

Alex, you already encountered this particular issue when investigating another intermittent (https://phabricator.services.mozilla.com/D143219#4687414). Any suggestion on how we could fix this? Otherwise we can skip on linux opt for now, it seems to only happen on this platform.

I'm doing a push to try to try to get better logs: https://treeherder.mozilla.org/jobs?repo=try&revision=52ec87bb3520b0f3770d632ff6a1a6fa31407429

Flags: needinfo?(jdescottes) → needinfo?(poirot.alex)

Ok we have a slightly helpful stacktrace:

[task 2023-01-09T10:44:08.838Z] 10:44:08 INFO - GECKO(1569) | 3494> resource://devtools/client/fronts/targets/target-mixin.js 302 getFront
[task 2023-01-09T10:44:08.838Z] 10:44:08 INFO - GECKO(1569) | 3494> resource://devtools/shared/commands/target/target-command.js 950 getAllFronts
[task 2023-01-09T10:44:08.838Z] 10:44:08 INFO - GECKO(1569) | 3494> resource://devtools/shared/commands/thread-configuration/thread-configuration-command.js 46 updateConfiguration
[task 2023-01-09T10:44:08.838Z] 10:44:08 INFO - GECKO(1569) | 3494> resource://devtools/client/debugger/src/client/firefox.js 78 onConnect
[task 2023-01-09T10:44:08.838Z] 10:44:08 INFO - GECKO(1569) | 3494> resource://devtools/client/debugger/src/main.js 104 bootstrap
[task 2023-01-09T10:44:08.838Z] 10:44:08 INFO - GECKO(1569) | 3494> resource://devtools/client/debugger/panel.js 68 open
[task 2023-01-09T10:44:08.838Z] 10:44:08 INFO - GECKO(1569) | 3494> resource://devtools/client/framework/toolbox.js 2608 onLoad

So that's very similar to the fix done on https://phabricator.services.mozilla.com/D143219#4687414 , it's just the getAllFronts call right above which fails. I can take a look.

Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Flags: needinfo?(poirot.alex)

I suspect we are using getAllTargetsInSelectedTargetTree which could theoritically
return destroyed targets as it is async.
If we are using getAllTargets, this is less clear. It shouldn't return destroyed target.

Assignee: jdescottes → poirot.alex
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9d2dacaf56c3 [devtools] Fix browser_toolbox_styleeditor.js intermittent. r=jdescottes
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: 110 Branch → ---

Let's monitor the frequency. The failure shows that we managed to start the toolbox successfully here, but we seem to fail during the toolbox task, possibly around the end of it: https://searchfox.org/mozilla-central/rev/a156a65ced2dae5913ae35a68e9445b8ee7ca457/devtools/client/styleeditor/test/browser_toolbox_styleeditor.js#64-79

For instance we could be stuck on the first await highlighted here, because the failure screenshot shows that we have not updated the stylesheet...

Resetting the assignee, the patch from Alex had the expected effect and we are now seeing a different issue.

Assignee: poirot.alex → nobody

I think the issue is just that in some cases simple.css is listed first, and therefore already selected, which makes the test timeout.

Assignee: nobody → jdescottes
Status: REOPENED → ASSIGNED
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f9d214f3f36d [devtools] Check if simple.css is already selected in browser_toolbox_styleeditor.js r=ochameau
Status: ASSIGNED → RESOLVED
Closed: 3 years ago3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch
See Also: → 1810765
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: