Closed Bug 994747 Opened 11 years ago Closed 11 years ago

Intermittent browser_console_hide_jsterm_when_devtools_chrome_enabled_false.js | Test timed out | Found a devtools:webconsole after previous test timed out

Categories

(DevTools :: Console, defect)

Product:
DevTools
Component:
Console
Version:
26 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox29 unaffected, firefox30 fixed, firefox31 fixed, firefox-esr24 unaffected)

Status:
RESOLVED FIXED
Milestone:
Firefox 31
Tracking Flags:
Tracking Status
firefox29 --- unaffected
firefox30 --- fixed
firefox31 --- fixed
firefox-esr24 --- unaffected

People

(Reporter: RyanVM, Assigned: Optimizer)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 2 obsolete files)

fix
2.16 KB, patch
Optimizer
: review+
RyanVM
: feedback+
lsblakk
: approval-mozilla-aurora+
Details | Diff | Splinter Review
This first appeared on the Try push for bug 993580 and also shows on Cedar mochitest-dt runs. I expect we'll see this showing up on other trees once bug 993580 merges around. I assume the Promise messages in the log are relevant? https://tbpl.mozilla.org/php/getParsedLog.php?id=37537122&tree=Try Ubuntu VM 12.04 x64 try debug test mochitest-browser-chrome-3 on 2014-04-09 18:20:39 PDT for push 7eee9af25724 slave: tst-linux64-spot-536 18:31:44 INFO - TEST-INFO | chrome://mochitests/content/browser/browser/devtools/webconsole/test/browser_console_hide_jsterm_when_devtools_chrome_enabled_false.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection. 18:31:44 INFO - Date: Wed Apr 09 2014 18:31:25 GMT-0700 (PDT) 18:31:44 INFO - Full Message: null 18:31:44 INFO - Full Stack: JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: PendingErrors.register :: line 145 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: this.PromiseWalker.completePromise :: line 555 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise.reject/< :: line 416 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise :: line 271 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise.reject :: line 416 18:31:44 INFO - JS frame :: resource:///modules/devtools/gDevTools.jsm :: DT_closeToolbox :: line 320 18:31:44 INFO - JS frame :: chrome://mochitests/content/browser/browser/devtools/webconsole/test/head.js :: tearDown :: line 343 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: Tester_nextTest :: line 290 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: testScope/test_finish/< :: line 830 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: testScope/test_executeSoon/<.run :: line 743 18:31:44 INFO - native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0"] 18:31:44 INFO - TEST-INFO | chrome://mochitests/content/browser/browser/devtools/webconsole/test/browser_console_hide_jsterm_when_devtools_chrome_enabled_false.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection. 18:31:44 INFO - Date: Wed Apr 09 2014 18:31:28 GMT-0700 (PDT) 18:31:44 INFO - Full Message: null 18:31:44 INFO - Full Stack: JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: PendingErrors.register :: line 145 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: this.PromiseWalker.completePromise :: line 555 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise.reject/< :: line 416 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise :: line 271 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise.reject :: line 416 18:31:44 INFO - JS frame :: resource:///modules/devtools/gDevTools.jsm :: DT_closeToolbox :: line 320 18:31:44 INFO - JS frame :: chrome://mochitests/content/browser/browser/devtools/webconsole/test/head.js :: tearDown :: line 343 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: Tester_nextTest :: line 290 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: testScope/test_finish/< :: line 830 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: testScope/test_executeSoon/<.run :: line 743 18:31:44 INFO - native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0"] 18:31:44 INFO - TEST-INFO | chrome://mochitests/content/browser/browser/devtools/webconsole/test/browser_console_hide_jsterm_when_devtools_chrome_enabled_false.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection. 18:31:44 INFO - Date: Wed Apr 09 2014 18:31:29 GMT-0700 (PDT) 18:31:44 INFO - Full Message: null 18:31:44 INFO - Full Stack: JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: PendingErrors.register :: line 145 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: this.PromiseWalker.completePromise :: line 555 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise.reject/< :: line 416 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise :: line 271 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise.reject :: line 416 18:31:44 INFO - JS frame :: resource:///modules/devtools/gDevTools.jsm :: DT_closeToolbox :: line 320 18:31:44 INFO - JS frame :: chrome://mochitests/content/browser/browser/devtools/webconsole/test/head.js :: tearDown :: line 343 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: Tester_nextTest :: line 290 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: testScope/test_finish/< :: line 830 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: testScope/test_executeSoon/<.run :: line 743 18:31:44 INFO - native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0"] 18:31:44 INFO - TEST-INFO | chrome://mochitests/content/browser/browser/devtools/webconsole/test/browser_console_hide_jsterm_when_devtools_chrome_enabled_false.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection. 18:31:44 INFO - Date: Wed Apr 09 2014 18:31:32 GMT-0700 (PDT) 18:31:44 INFO - Full Message: null 18:31:44 INFO - Full Stack: JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: PendingErrors.register :: line 145 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: this.PromiseWalker.completePromise :: line 555 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise.reject/< :: line 416 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise :: line 271 18:31:44 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Promise.reject :: line 416 18:31:44 INFO - JS frame :: resource:///modules/devtools/gDevTools.jsm :: DT_closeToolbox :: line 320 18:31:44 INFO - JS frame :: chrome://mochitests/content/browser/browser/devtools/webconsole/test/head.js :: tearDown :: line 343 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: Tester_nextTest :: line 290 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: testScope/test_finish/< :: line 830 18:31:44 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: testScope/test_executeSoon/<.run :: line 743 18:31:44 INFO - native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0"] 18:31:44 INFO - --DOMWINDOW == 99 (0x3e79620) [pid = 3229] [serial = 1179] [outer = (nil)] [url = about:blank] 18:31:44 INFO - --DOMWINDOW == 98 (0x4b6a4b0) [pid = 3229] [serial = 1168] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:44 INFO - --DOMWINDOW == 97 (0x5915620) [pid = 3229] [serial = 1177] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:44 INFO - --DOMWINDOW == 96 (0xe33f350) [pid = 3229] [serial = 1185] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:31:44 INFO - --DOMWINDOW == 95 (0xa7532f0) [pid = 3229] [serial = 1121] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:44 INFO - --DOMWINDOW == 94 (0x9df6190) [pid = 3229] [serial = 1123] [outer = (nil)] [url = about:blank] 18:31:44 INFO - --DOMWINDOW == 93 (0x9964c00) [pid = 3229] [serial = 1094] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:31:44 INFO - --DOMWINDOW == 92 (0x70baf10) [pid = 3229] [serial = 1129] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:31:44 INFO - --DOMWINDOW == 91 (0x2078500) [pid = 3229] [serial = 1134] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:44 INFO - --DOMWINDOW == 90 (0x631a3a0) [pid = 3229] [serial = 1136] [outer = (nil)] [url = about:blank] 18:31:44 INFO - --DOCSHELL 0x5f27ac0 == 20 [pid = 3229] [id = 514] 18:31:44 INFO - --DOCSHELL 0x6a23e50 == 19 [pid = 3229] [id = 515] 18:31:44 INFO - --DOCSHELL 0x431c9f0 == 18 [pid = 3229] [id = 492] 18:31:44 INFO - --DOCSHELL 0xbbee5b0 == 17 [pid = 3229] [id = 541] 18:31:44 INFO - --DOCSHELL 0x6c3e8a0 == 16 [pid = 3229] [id = 540] 18:31:44 INFO - --DOCSHELL 0xaaad110 == 15 [pid = 3229] [id = 539] 18:31:44 INFO - --DOCSHELL 0x193b8e0 == 14 [pid = 3229] [id = 538] 18:31:44 INFO - --DOCSHELL 0x5d0d3e0 == 13 [pid = 3229] [id = 537] 18:31:44 INFO - --DOCSHELL 0x9aaac30 == 12 [pid = 3229] [id = 536] 18:31:44 INFO - --DOCSHELL 0x64b62a0 == 11 [pid = 3229] [id = 535] 18:31:44 INFO - --DOCSHELL 0x2491f80 == 10 [pid = 3229] [id = 534] 18:31:44 INFO - --DOCSHELL 0x529cec0 == 9 [pid = 3229] [id = 533] 18:31:46 INFO - --DOMWINDOW == 89 (0x65d5390) [pid = 3229] [serial = 1171] [outer = 0x8a18500] [url = http://example.com/browser/browser/devtools/webconsole/test/test-console.html] 18:31:46 INFO - --DOMWINDOW == 88 (0x828fda0) [pid = 3229] [serial = 1197] [outer = 0x6d0b010] [url = about:blank] 18:31:46 INFO - --DOMWINDOW == 87 (0x6b35770) [pid = 3229] [serial = 1187] [outer = 0x800ae70] [url = about:blank] 18:31:46 INFO - --DOMWINDOW == 86 (0x8a18500) [pid = 3229] [serial = 1163] [outer = (nil)] [url = http://example.com/browser/browser/devtools/webconsole/test/test-console.html] 18:31:46 INFO - --DOMWINDOW == 85 (0x800ae70) [pid = 3229] [serial = 1186] [outer = (nil)] [url = data:text/html;charset=utf8,<p>dead%20objects!] 18:31:46 INFO - --DOMWINDOW == 84 (0x6d0b010) [pid = 3229] [serial = 1196] [outer = (nil)] [url = data:text/html;charset=utf8,<p>browser%20console%20filters] 18:31:46 INFO - --DOMWINDOW == 83 (0x5d95db0) [pid = 3229] [serial = 1199] [outer = (nil)] [url = about:blank] 18:31:46 INFO - --DOMWINDOW == 82 (0xa69b5d0) [pid = 3229] [serial = 1099] [outer = (nil)] [url = about:blank] 18:31:46 INFO - --DOMWINDOW == 81 (0x9537b80) [pid = 3229] [serial = 1109] [outer = (nil)] [url = about:blank] 18:31:46 INFO - --DOMWINDOW == 80 (0xa734ee0) [pid = 3229] [serial = 1157] [outer = (nil)] [url = about:blank] 18:31:46 INFO - --DOMWINDOW == 79 (0x71be980) [pid = 3229] [serial = 1147] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:46 INFO - --DOMWINDOW == 78 (0x6242d40) [pid = 3229] [serial = 1080] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:46 INFO - --DOMWINDOW == 77 (0x606a790) [pid = 3229] [serial = 1103] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:31:46 INFO - --DOMWINDOW == 76 (0x6cda8f0) [pid = 3229] [serial = 1088] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:46 INFO - --DOMWINDOW == 75 (0x68b2ba0) [pid = 3229] [serial = 1113] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:31:46 INFO - --DOMWINDOW == 74 (0x61f3bd0) [pid = 3229] [serial = 1071] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:46 INFO - --DOMWINDOW == 73 (0x52d3090) [pid = 3229] [serial = 1086] [outer = (nil)] [url = data:text/html;charset=utf8,<p>test%20for%20bug%20862916] 18:31:46 INFO - --DOMWINDOW == 72 (0x68c0ce0) [pid = 3229] [serial = 1192] [outer = (nil)] [url = data:text/html;charset=utf8,<p>hello%20world%20from%20bug%20877778%20<button%20onclick='foobar.explode()'%20style='test-color:%20green-please'>click!</button>] 18:31:46 INFO - --DOMWINDOW == 71 (0x8c52fb0) [pid = 3229] [serial = 1087] [outer = (nil)] [url = about:blank] 18:31:46 INFO - --DOMWINDOW == 70 (0x1159bf0) [pid = 3229] [serial = 1091] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 69 (0x8ee26f0) [pid = 3229] [serial = 1083] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 68 (0x3bb1850) [pid = 3229] [serial = 1188] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 67 (0xdf300f0) [pid = 3229] [serial = 1161] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 66 (0x50329c0) [pid = 3229] [serial = 1150] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 65 (0x544fea0) [pid = 3229] [serial = 1203] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 64 (0xb077da0) [pid = 3229] [serial = 1201] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 63 (0x50939b0) [pid = 3229] [serial = 1074] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 62 (0x7b8a690) [pid = 3229] [serial = 1194] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 61 (0x60d4150) [pid = 3229] [serial = 1152] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 60 (0x831bb40) [pid = 3229] [serial = 1159] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:46 INFO - --DOMWINDOW == 59 (0x560c8b0) [pid = 3229] [serial = 1198] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:46 INFO - --DOMWINDOW == 58 (0x623e690) [pid = 3229] [serial = 1156] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:46 INFO - --DOMWINDOW == 57 (0x48f0210) [pid = 3229] [serial = 1190] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:31:54 INFO - --DOMWINDOW == 56 (0x68b8cd0) [pid = 3229] [serial = 1092] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 55 (0x4930f50) [pid = 3229] [serial = 1084] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 54 (0x76614d0) [pid = 3229] [serial = 1189] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 53 (0x4285040) [pid = 3229] [serial = 1162] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 52 (0x7d27ad0) [pid = 3229] [serial = 1151] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 51 (0x7161af0) [pid = 3229] [serial = 1204] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 50 (0x7c12660) [pid = 3229] [serial = 1202] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 49 (0x29f25f0) [pid = 3229] [serial = 1075] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 48 (0x3774d30) [pid = 3229] [serial = 1195] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 47 (0x8d8e6d0) [pid = 3229] [serial = 1153] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 46 (0xa787c40) [pid = 3229] [serial = 1160] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 45 (0x5380ed0) [pid = 3229] [serial = 1149] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:54 INFO - --DOMWINDOW == 44 (0x6a1d9e0) [pid = 3229] [serial = 1082] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:54 INFO - --DOMWINDOW == 43 (0x6fedf20) [pid = 3229] [serial = 1104] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:31:54 INFO - --DOMWINDOW == 42 (0x60a36c0) [pid = 3229] [serial = 1090] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:54 INFO - --DOMWINDOW == 41 (0x92c8390) [pid = 3229] [serial = 1114] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:31:54 INFO - --DOMWINDOW == 40 (0xa0ad3a0) [pid = 3229] [serial = 1073] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:54 INFO - --DOMWINDOW == 39 (0x9be53c0) [pid = 3229] [serial = 1158] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:54 INFO - --DOMWINDOW == 38 (0x55909c0) [pid = 3229] [serial = 1170] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 37 (0x5ed1150) [pid = 3229] [serial = 1191] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:31:54 INFO - --DOMWINDOW == 36 (0x653b6a0) [pid = 3229] [serial = 1200] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:54 INFO - --DOMWINDOW == 35 (0xc1e0080) [pid = 3229] [serial = 1193] [outer = (nil)] [url = about:blank] 18:31:54 INFO - --DOMWINDOW == 34 (0x3771080) [pid = 3229] [serial = 1183] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:31:55 INFO - --DOMWINDOW == 33 (0xcebe5a0) [pid = 3229] [serial = 1111] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:55 INFO - --DOMWINDOW == 32 (0x6219d80) [pid = 3229] [serial = 1101] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:55 INFO - --DOMWINDOW == 31 (0xbdbdd40) [pid = 3229] [serial = 1180] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:55 INFO - --DOMWINDOW == 30 (0x5522ba0) [pid = 3229] [serial = 1145] [outer = (nil)] [url = chrome://browser/content/devtools/webconsole.xul] 18:31:55 INFO - --DOMWINDOW == 29 (0x5300e30) [pid = 3229] [serial = 1142] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:55 INFO - --DOMWINDOW == 28 (0x84f9280) [pid = 3229] [serial = 1098] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:55 INFO - --DOMWINDOW == 27 (0x5d6bce0) [pid = 3229] [serial = 1108] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:31:55 INFO - --DOMWINDOW == 26 (0x579d510) [pid = 3229] [serial = 1115] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:32:02 INFO - --DOMWINDOW == 25 (0x6ffdd70) [pid = 3229] [serial = 1112] [outer = (nil)] [url = about:blank] 18:32:02 INFO - --DOMWINDOW == 24 (0x9ab23d0) [pid = 3229] [serial = 1102] [outer = (nil)] [url = about:blank] 18:32:02 INFO - --DOMWINDOW == 23 (0x68ad000) [pid = 3229] [serial = 1181] [outer = (nil)] [url = about:blank] 18:32:02 INFO - --DOMWINDOW == 22 (0xa261080) [pid = 3229] [serial = 1146] [outer = (nil)] [url = about:blank] 18:32:02 INFO - --DOMWINDOW == 21 (0x4f26380) [pid = 3229] [serial = 1144] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:32:02 INFO - --DOMWINDOW == 20 (0x6b30de0) [pid = 3229] [serial = 1100] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:32:02 INFO - --DOMWINDOW == 19 (0x8426ea0) [pid = 3229] [serial = 1110] [outer = (nil)] [url = chrome://browser/content/devtools/framework/toolbox.xul] 18:32:02 INFO - --DOMWINDOW == 18 (0xa8f3640) [pid = 3229] [serial = 1116] [outer = (nil)] [url = chrome://browser/content/devtools/widgets/VariablesView.xul] 18:32:18 INFO - Xlib: extension "RANDR" missing on display ":0". 18:32:19 INFO - TEST-INFO | screentopng: exit 0 18:32:19 WARNING - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/webconsole/test/browser_console_hide_jsterm_when_devtools_chrome_enabled_false.js | Test timed out 18:32:19 INFO - TEST-INFO | MEMORY STAT vsize after test: 2045136896 18:32:19 INFO - TEST-INFO | MEMORY STAT residentFast after test: 534478848 18:32:19 INFO - INFO TEST-END | chrome://mochitests/content/browser/browser/devtools/webconsole/test/browser_console_hide_jsterm_when_devtools_chrome_enabled_false.js | finished in 45237ms 18:32:19 INFO - TEST-INFO | checking window state 18:32:19 WARNING - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/webconsole/test/browser_console_hide_jsterm_when_devtools_chrome_enabled_false.js | Found a devtools:webconsole after previous test timed out 18:32:19 INFO - TEST-INFO | chrome://mochitests/content/browser/browser/devtools/webconsole/test/browser_console_hide_jsterm_when_devtools_chrome_enabled_false.js | must wait for focus
https://tbpl.mozilla.org/php/getParsedLog.php?id=37539752&tree=Try Rev4 MacOSX Snow Leopard 10.6 try debug test mochitest-browser-chrome on 2014-04-09 17:44:52 PDT for push 7eee9af25724 slave: talos-r4-snow-014 https://tbpl.mozilla.org/php/getParsedLog.php?id=37576853&tree=Cedar Ubuntu ASAN VM 12.04 x64 cedar opt test mochitest-devtools-chrome on 2014-04-10 07:36:44 PDT for push 74bf812cb781 slave: tst-linux64-spot-590
True, in head.js we close the toolbox, what so ever. But in that particular test, the console is already closed, thus no toolbox open. There are two options here: - Fix the teardown method in head.js to first check if toolbox is open or not. or. - Make gDevTools.closeToolbox always resolve the promise instead of rejecting it in non-existant-toolbox case. Joe, what say ? This should have been a perma orange FWIW.
Flags: needinfo?(jwalker)
Attached patch fix (obsolete) — — Splinter Review
Merged teardown and finishTest method and closing toolbox only if required. This patch implements first option from comment 3 All webconsole tests pass locally. full try : https://tbpl.mozilla.org/?tree=Try&rev=187715040555
Assignee: nobody → scrapmachines
Status: NEW → ASSIGNED
Attachment #8404893 - Flags: review?(mihai.sucan)
err, the patch is leaking all over the place on debugs. Will fix tomorrow.
Attachment #8404893 - Flags: review?(mihai.sucan)
(In reply to Girish Sharma [:Optimizer] from comment #3) > True, in head.js we close the toolbox, what so ever. But in that particular > test, the console is already closed, thus no toolbox open. > > There are two options here: > - Fix the teardown method in head.js to first check if toolbox is open or > not. > or. > - Make gDevTools.closeToolbox always resolve the promise instead of > rejecting it in non-existant-toolbox case. > > Joe, what say ? > > This should have been a perma orange FWIW. Yuck. This looks much better to me: /** * Close the toolbox for a given target * @return A promise that resolves to true if a toolbox was closed * if false if no toolbox was found, attached to that target */ closeToolbox: function DT_closeToolbox(target) { let toolbox = this._toolboxes.get(target); if (toolbox == null) { return promise.resolve(false); } return toolbox.destroy().then({} => true); },
Flags: needinfo?(jwalker)
(obviously without the obvious syntax errors)
(In reply to Joe Walker [:jwalker] from comment #13) > (In reply to Girish Sharma [:Optimizer] from comment #3) > > True, in head.js we close the toolbox, what so ever. But in that particular > > test, the console is already closed, thus no toolbox open. > > > > There are two options here: > > - Fix the teardown method in head.js to first check if toolbox is open or > > not. > > or. > > - Make gDevTools.closeToolbox always resolve the promise instead of > > rejecting it in non-existant-toolbox case. > > > > Joe, what say ? > > > > This should have been a perma orange FWIW. > > Yuck. This looks much better to me: > > /** > * Close the toolbox for a given target > * @return A promise that resolves to true if a toolbox was closed > * if false if no toolbox was found, attached to that target > */ > closeToolbox: function DT_closeToolbox(target) { > let toolbox = this._toolboxes.get(target); > if (toolbox == null) { > return promise.resolve(false); > } > return toolbox.destroy().then({} => true); > }, This is much cleaner. It will also fix this intermittent without the merging approach which I attached (which is leaking badly).
Attached patch better fix (obsolete) — — Splinter Review
So, this patch implements the second approach in the way Joe suggested. try: https://tbpl.mozilla.org/?tree=Try&rev=c0a93322c4f6
Attachment #8404893 - Attachment is obsolete: true
Attachment #8405407 - Flags: review?(jwalker)
Girish, the closeToolbox() fix makes more sense. Thanks!
Attachment #8405407 - Flags: review?(jwalker) → review+
ughh. there are so many other oranges in the try push, but nothing related to this one.
Keywords: checkin-needed
This seems to be happening consistently across runs: TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/commandline/test/browser_cmd_commands.js | html output for console close - Got true, expected
Keywords: checkin-needed
Attached patch fix — — Splinter Review
Thanks Ryan for catching that amongst all those oranges. Never expected some test will test the output of closeToolbox. Fixed it. Passes locally. bc try : https://tbpl.mozilla.org/?tree=Try&rev=bb6bc799edb7
Attachment #8405407 - Attachment is obsolete: true
Attachment #8405615 - Flags: review+
The devtools tests aren't running on Try at the moment until bug 995529 gets fixed. So to give this some better test coverage, I pushed it to Cedar as well. https://tbpl.mozilla.org/?tree=Cedar&jobname=mochitest.*chrome&rev=25b5c27bbdbc
Comment on attachment 8405615 [details] [diff] [review] fix Lots of other OSX/Windows devtools failures on that push, but none of them are new from this patch. And not a single instance of this bug. LGTM! Optimizer - I don't suppose I could talk you into looking at some of those other ones, could I? ;) :P
Attachment #8405615 - Flags: feedback+
Haha, all those are hardcore intermittents. I am too busy with the front end of storage inspector now :)
Keywords: checkin-needed
https://hg.mozilla.org/integration/fx-team/rev/430f2a9c2349
Flags: in-testsuite+
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/430f2a9c2349
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 31
Comment on attachment 8405615 [details] [diff] [review] fix [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 993580 User impact if declined: This bug fixes an issue introduced by 993580. Also, this patch is needed as a part of a bigger queue so as to uplift the chunks-by-dir feature of tests to aurora. see https://tbpl.mozilla.org/?tree=Try&rev=bfee331a418c Testing completed (on m-c, etc.): mc Risk to taking this patch (and alternatives if risky): none String or IDL/UUID changes made by this patch: none
Attachment #8405615 - Flags: approval-mozilla-aurora?
Attachment #8405615 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: