Closed Bug 1322627 Opened 8 years ago Closed 8 years ago

Intermittent devtools/client/inspector/test/browser_inspector_textbox-menu.js | Test timed out -

Categories

(DevTools :: Inspector, defect, P3)

Product:
DevTools
Component:
Inspector
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED INCOMPLETE

People

(Reporter: intermittent-bug-filer, Unassigned)

Details

(Keywords: bulk-close-intermittents, intermittent-failure)

Inspector bug triage (filter on CLIMBING SHOES).
Priority: -- → P3
The test intermittently fails to open the context menu. Here's the stack trace I found (with the protocol and promise stuff removed): TEST-UNEXPECTED-FAIL | devtools/client/inspector/test/browser_inspector_textbox-menu.js | The menu is now visible - Got closed, expected open Stack trace: chrome://mochikit/content/browser-test.js:test_is:913 chrome://mochitests/content/browser/devtools/client/inspector/test/browser_inspector_textbox-menu.js:checkTextBox:82 [...] handler@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/event-emitter.js:137:9 emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/event-emitter.js:191:13 checkDone@chrome://devtools/content/inspector/inspector.js:866:11 done@chrome://devtools/content/inspector/inspector.js:874:7 onNewSelection/<@chrome://devtools/content/inspector/inspector.js:827:9 DevToolsUtils.executeSoon*exports.executeSoon@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/DevToolsUtils.js:42:19 onNewSelection@chrome://devtools/content/inspector/inspector.js:825:5 emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/event-emitter.js:191:13 setNodeFront@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/framework/selection.js:127:5 selectNode<@chrome://mochitests/content/browser/devtools/client/inspector/test/shared-head.js:154:3 [...] focusNode@chrome://mochitests/content/browser/devtools/client/inspector/test/head.js:114:9 [...] WalkerFront<.querySelector<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/fronts/inspector.js:621:12 getNodeFront@chrome://mochitests/content/browser/devtools/client/inspector/test/shared-head.js:137:10 focusNode@chrome://mochitests/content/browser/devtools/client/inspector/test/head.js:112:25 [...] getOriginalLocation@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/fronts/styles.js:237:12 getOriginalSourceStrings@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/rules/models/rule.js:139:12 updateSourceLink@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/rules/views/rule-editor.js:261:7 _create@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/rules/views/rule-editor.js:124:5 RuleEditor@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/rules/views/rule-editor.js:77:3 _createEditors@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/rules/rules.js:1070:23 _populate/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/rules/rules.js:874:28 [...] PageStyleFront<.getApplied<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/fronts/styles.js:67:21 [...] populate@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/rules/models/element-style.js:90:21 _populate@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/rules/rules.js:868:12 selectElement/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/rules/rules.js:783:16 [...] WalkerFront<.querySelector<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/fronts/inspector.js:621:12 getNodeFront@chrome://mochitests/content/browser/devtools/client/inspector/test/shared-head.js:137:10 Closing the menu Waiting for event: 'popuphidden' on [object XULElement]. So, the test fails on this assertion: http://searchfox.org/mozilla-central/rev/30fcf167af036aeddf322de44a2fadd370acfd2f/devtools/client/inspector/test/browser_inspector_textbox-menu.js#82 Here: info("Simulating context click on the textbox and expecting the menu to open"); let onContextMenu = once(textBoxContextMenuPopup, "popupshown"); EventUtils.synthesizeMouse(textBox, 2, 2, {type: "contextmenu", button: 2}, textBox.ownerDocument.defaultView); yield onContextMenu; is(textBoxContextMenuPopup.state, "open", "The menu is now visible"); So, waiting for the popupshown event doesn't always seem to work, and in some cases the state isn't "open" yet.
Gabriel, this started getting frequent on inbound, especially on OS X. Can you take a look, please? https://treeherder.mozilla.org/logviewer.html#?job_id=89294561&repo=mozilla-inbound
Flags: needinfo?(gl)
Assignee: nobody → gl
Status: NEW → ASSIGNED
Flags: needinfo?(gl)
Assignee: gl → nobody
Status: ASSIGNED → NEW
Status: NEW → RESOLVED
Closed: 8 years ago
Keywords: bulk-close-intermittents
Resolution: --- → INCOMPLETE
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.