Open Bug 1699008 Opened 3 years ago Updated 2 years ago

Intermittent devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js | the markup view is still rendered fine -

Categories

(DevTools :: Framework, defect, P2)

defect

Tracking

(firefox-esr78 unaffected, firefox86 unaffected, firefox87 unaffected, firefox88 fix-optional)

ASSIGNED
Tracking Status
firefox-esr78 --- unaffected
firefox86 --- unaffected
firefox87 --- unaffected
firefox88 --- fix-optional

People

(Reporter: intermittent-bug-filer, Assigned: nchevobbe)

References

(Regression)

Details

(Keywords: intermittent-failure, leave-open, regression, Whiteboard: [stockwell disabled][stockwell needswork:owner])

Attachments

(1 file, 1 obsolete file)

Filed by: nerli [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=333469618&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/ByN54UKjTB-U95ri8XwfZQ/runs/0/artifacts/public/logs/live_backing.log


[task 2021-03-17T00:25:57.514Z] 00:25:57     INFO - TEST-PASS | devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js | Toolbar is still visible - 
[task 2021-03-17T00:25:57.515Z] 00:25:57     INFO - Check that the markup view is rendered correctly and elements can be selected
[task 2021-03-17T00:25:57.515Z] 00:25:57     INFO - Buffered messages finished
[task 2021-03-17T00:25:57.523Z] 00:25:57     INFO - TEST-UNEXPECTED-FAIL | devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js | the markup view is still rendered fine - 
[task 2021-03-17T00:25:57.524Z] 00:25:57     INFO - Stack trace:
[task 2021-03-17T00:25:57.525Z] 00:25:57     INFO - chrome://mochikit/content/browser-test.js:test_ok:1331
[task 2021-03-17T00:25:57.526Z] 00:25:57     INFO - chrome://mochitests/content/browser/devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js:checkToolboxState:83
[task 2021-03-17T00:25:57.527Z] 00:25:57     INFO - chrome://mochitests/content/browser/devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js:null:70
[task 2021-03-17T00:25:57.528Z] 00:25:57     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1089
[task 2021-03-17T00:25:57.529Z] 00:25:57     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1129
[task 2021-03-17T00:25:57.530Z] 00:25:57     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:949
[task 2021-03-17T00:25:57.531Z] 00:25:57     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:1037
[task 2021-03-17T00:25:57.532Z] 00:25:57     INFO - Selecting the node for 'ul.logs'
[task 2021-03-17T00:25:57.533Z] 00:25:57     INFO - GECKO(1529) | [Parent 1529, Main Thread] WARNING: NS_ENSURE_TRUE(GetOwner() && !mCanceled) failed: file /builds/worker/checkouts/gecko/image/imgRequestProxy.cpp:909
[task 2021-03-17T00:25:57.534Z] 00:25:57     INFO - GECKO(1529) | [Parent 1529, Main Thread] WARNING: NS_ENSURE_TRUE(GetOwner() && !mCanceled) failed: file /builds/worker/checkouts/gecko/image/imgRequestProxy.cpp:909
[task 2021-03-17T00:25:57.535Z] 00:25:57     INFO - GECKO(1529) | console.log: "[DISPATCH] action type:" "CLEAR_FLEXBOX"
[task 2021-03-17T00:25:57.536Z] 00:25:57     INFO - Check that the debugger has some sources
[task 2021-03-17T00:25:57.614Z] 00:25:57     INFO - Wait for doc_backward_forward_navigation.html to be displayed in the debugger source panel
[task 2021-03-17T00:25:57.723Z] 00:25:57     INFO - GECKO(1529) | [Child 1828: Main Thread]: I/DocShellAndDOMWindowLeak --DOMWINDOW == 5 (7f43e7cc0000) [pid = 1828] [serial = 41] [outer = 0] [url = about:blank]
[task 2021-03-17T00:25:58.000Z] 00:25:58     INFO - GECKO(1529) | console.log: "[DISPATCH] action type:" "CLEAR_FLEXBOX"
[task 2021-03-17T00:25:58.042Z] 00:25:58     INFO - GECKO(1529) | [Parent 1529, Main Thread] WARNING: NS_ENSURE_TRUE(weakFrame.IsAlive()) failed: file /builds/worker/checkouts/gecko/layout/xul/nsXULPopupManager.cpp:1068
[task 2021-03-17T00:25:58.090Z] 00:25:58     INFO - TEST-PASS | devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js | The debugger has the expected source - 
[task 2021-03-17T00:25:58.091Z] 00:25:58     INFO - Leaving test bound 
[task 2021-03-17T00:25:58.359Z] 00:25:58     INFO - GECKO(1529) | console.error: (new Error("Connection closed, pending request to server0.conn45.child4/inspectorActor4, type supportsHighlighters failed\n\nRequest stack:\nrequest@resource://devtools/shared/protocol/Front.js:288:14\ngenerateRequestMethods/</frontProto[name]@resource://devtools/shared/protocol/Front/FrontClassWithSpec.js:46:19\nsupportsEyeDropper@resource://devtools/client/inspector/inspector.js:1232:40\nsetupToolbar@resource://devtools/client/inspector/inspector.js:1250:42\nonRootNodeAvailable@resource://devtools/client/inspector/inspector.js:1340:12\nAsync*onResourceAvailable@resource://devtools/client/inspector/inspector.js:1310:16\n_notifyWatchers@resource://devtools/shared/resources/resource-watcher.js:634:24\nlater@resource://devtools/shared/throttle.js:33:19\nnotify@resource://gre/modules/Timer.jsm:62:17\n", "resource://devtools/shared/protocol/Front.js", 99))
[task 2021-03-17T00:26:01.195Z] 00:26:01     INFO - Removing tab.
[task 2021-03-17T00:26:01.196Z] 00:26:01     INFO - Waiting for event: 'TabClose' on [object XULElement].
[task 2021-03-17T00:26:01.211Z] 00:26:01     INFO - Got event: 'TabClose' on [object XULElement].
[task 2021-03-17T00:26:01.219Z] 00:26:01     INFO - GECKO(1529) | [Parent 1529, Main Thread] WARNING: NS_ENSURE_TRUE(uri) failed: file /builds/worker/checkouts/gecko/caps/BasePrincipal.cpp:1327
[task 2021-03-17T00:26:01.227Z] 00:26:01     INFO - GECKO(1529) | [Parent 1529, Main Thread] WARNING: '!inner', file /builds/worker/checkouts/gecko/dom/ipc/jsactor/JSWindowActorProtocol.cpp:181
[task 2021-03-17T00:26:01.235Z] 00:26:01     INFO - Tab removed and finished closing```

This landed a week ago in bug 1697450. Is this bug a duplicate of bug 1697542?

Severity: S4 → --
Flags: needinfo?(nchevobbe)
Keywords: regression
Priority: P5 → --
Regressed by: 1697450
See Also: → 1697542
Has Regression Range: --- → yes

Probably, yes

Flags: needinfo?(nchevobbe)

In the last 7 days there have been 34 occurrences, most on linux1804-64-asan opt, some on linux1804-64 debug.

Recent failure: https://treeherder.mozilla.org/logviewer?job_id=334655915&repo=autoland&lineNumber=3240

Alexandre, is this concerning?

Flags: needinfo?(poirot.alex)

I'm going to have a look

Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED

This patch does a few thing to try to reduce intermittent failures on this test.

First, the test document has a script that adds dom nodes, js sources and stylesheet
to the document every 200ms. It might that the navigateTo call we were using to navigate
to the document would never resolve, as the load event won't fire because of the constant
addition of stylesheets and scripts.
This is fixed by not using navigateTo, as we don't really need the document to be ready,
but only register the navigation so later in the test we can do backward and forward navigations.

On some platform, the test takes a very long time, so we need to extend the timeout.

We're adding some guards around sources, so we don't try to handle them when the toolbox
is already destroyed.

On the inspector side, we were waiting for the inspector front to be listening for stylesheets
before initializing the walker front, which possibly introduces some delay. As the walker
does not need to rely on listening to stylesheet, we retrieve it earlier.
We also set the will-navigate event listener on the targetFront before waiting for the
inspector front to be ready, so we avoid potential race.

Finally, we added some optional chaining operator to avoid a few exceptions while the
toolbox navigates/closes.

Severity: -- → S3
Priority: -- → P2

Update:

There have been 38 failures within the last 7 days:

  • 1 failure on Windows 10 x64 asan
  • 25 failures on Linux 18.04 x64 asan opt
  • 12 failures on Linux 18.04 x64 debug

Recent failure: https://treeherder.mozilla.org/logviewer?job_id=337515399&repo=autoland&lineNumber=4581

[task 2021-04-23T00:09:08.272Z] 00:09:08     INFO - TEST-PASS | devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js | Toolbar is still visible - 
[task 2021-04-23T00:09:08.273Z] 00:09:08     INFO - Check that the markup view is rendered correctly and elements can be selected
[task 2021-04-23T00:09:08.274Z] 00:09:08     INFO - Buffered messages finished
[task 2021-04-23T00:09:08.275Z] 00:09:08     INFO - TEST-UNEXPECTED-FAIL | devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js | the markup view is still rendered fine - 
[task 2021-04-23T00:09:08.275Z] 00:09:08     INFO - Stack trace:
[task 2021-04-23T00:09:08.276Z] 00:09:08     INFO - chrome://mochikit/content/browser-test.js:test_ok:1341
[task 2021-04-23T00:09:08.276Z] 00:09:08     INFO - chrome://mochitests/content/browser/devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js:checkToolboxState:83
[task 2021-04-23T00:09:08.277Z] 00:09:08     INFO - chrome://mochitests/content/browser/devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js:null:70
[task 2021-04-23T00:09:08.277Z] 00:09:08     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1096
[task 2021-04-23T00:09:08.278Z] 00:09:08     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1136
[task 2021-04-23T00:09:08.278Z] 00:09:08     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:953
[task 2021-04-23T00:09:08.279Z] 00:09:08     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:1037
[task 2021-04-23T00:09:08.280Z] 00:09:08     INFO - Selecting the node for 'ul.logs'
[task 2021-04-23T00:09:08.280Z] 00:09:08     INFO - GECKO(1536) | console.log: "[DISPATCH] action type:" "CLEAR_FLEXBOX"
[task 2021-04-23T00:09:08.281Z] 00:09:08     INFO - GECKO(1536) | [Parent 1536, Main Thread] WARNING: NS_ENSURE_TRUE(GetOwner() && !mCanceled) failed: file /builds/worker/checkouts/gecko/image/imgRequestProxy.cpp:909
[task 2021-04-23T00:09:08.289Z] 00:09:08     INFO - GECKO(1536) | [Parent 1536, Main Thread] WARNING: NS_ENSURE_TRUE(GetOwner() && !mCanceled) failed: file /builds/worker/checkouts/gecko/image/imgRequestProxy.cpp:909
[task 2021-04-23T00:09:08.350Z] 00:09:08     INFO - GECKO(1536) | [Parent 1536, Main Thread] WARNING: NS_ENSURE_TRUE(GetOwner() && !mCanceled) failed: file /builds/worker/checkouts/gecko/image/imgRequestProxy.cpp:909
[task 2021-04-23T00:09:08.351Z] 00:09:08     INFO - GECKO(1536) | [Parent 1536, Main Thread] WARNING: NS_ENSURE_TRUE(GetOwner() && !mCanceled) failed: file /builds/worker/checkouts/gecko/image/imgRequestProxy.cpp:909
[task 2021-04-23T00:09:08.371Z] 00:09:08     INFO - Check that the debugger has some sources
[task 2021-04-23T00:09:08.459Z] 00:09:08     INFO - Wait for doc_backward_forward_navigation.html to be displayed in the debugger source panel
[task 2021-04-23T00:09:08.540Z] 00:09:08     INFO - GECKO(1536) | console.log: "[DISPATCH] action type:" "UPDATE_GRIDS"
[task 2021-04-23T00:09:08.544Z] 00:09:08     INFO - GECKO(1536) | console.log: "[DISPATCH] action type:" "CLEAR_FLEXBOX"
[task 2021-04-23T00:09:08.544Z] 00:09:08     INFO - GECKO(1536) | console.log: "[DISPATCH] action type:" "CLEAR_FLEXBOX"
[task 2021-04-23T00:09:08.702Z] 00:09:08     INFO - GECKO(1536) | [Parent 1536, Main Thread] WARNING: NS_ENSURE_TRUE(weakFrame.IsAlive()) failed: file /builds/worker/checkouts/gecko/layout/xul/nsXULPopupManager.cpp:1266
[task 2021-04-23T00:09:08.722Z] 00:09:08     INFO - TEST-PASS | devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js | The debugger has the expected source -
Whiteboard: [stockwell disable-recommended] → [stockwell disable-recommended][stockwell needswork:owner]
Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2165389b8c69
disable browser_toolbox_backward_forward_navigation.js on Linux_64 for frequent failures. r=intermittent-reviewers,jmaher DONTBUILD
Keywords: leave-open
Whiteboard: [stockwell disable-recommended][stockwell needswork:owner] → [stockwell disabled][stockwell needswork:owner]
Depends on: 1707823

The test has been fixed and re-enabled in bug 1707823.

Flags: needinfo?(poirot.alex)
Attachment #9215807 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: