Closed Bug 1007304 Opened 12 years ago Closed 12 years ago

Intermittent browser_987492_window_api.js | Should have gotten onWindowClosed event - Didn't expect null, but got it (and more)

Categories

(Firefox :: Toolbars and Customization, defect)

Product:
Firefox
Component:
Toolbars and Customization
Platform:
x86_64
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 32
Tracking Flags:
Tracking Status
firefox30 --- wontfix
firefox31 --- fixed
firefox32 --- fixed
firefox-esr24 --- unaffected

People

(Reporter: ryanvm, Assigned: Gijs)

References

Details

(Keywords: intermittent-failure, Whiteboard: p=0 [qa-])

This started on the push from bug 805068, which seems quite relevant. https://tbpl.mozilla.org/php/getParsedLog.php?id=39212300&tree=Fx-Team Rev4 MacOSX Snow Leopard 10.6 fx-team debug test mochitest-browser-chrome-1 on 2014-05-07 07:44:55 PDT for push 0c890d80e9a6 slave: talos-r4-snow-122 08:05:16 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | Should have gotten onWindowOpen event 08:05:16 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | onWindowOpen event should have received expected window 08:05:16 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | Should have two customizable windows 08:05:16 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | Current window should be in window collection. 08:05:16 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | New window should be in window collection. 08:05:16 INFO - [Parent 959] WARNING: attempt to modify an immutable nsStandardURL: file /builds/slave/fx-team-osx64-d-00000000000000/build/netwerk/base/src/nsStandardURL.cpp, line 1216 08:05:16 WARNING - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | Should have gotten onWindowClosed event - Didn't expect null, but got it 08:05:16 INFO - Stack trace: 08:05:16 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: test_isnot :: line 756 08:05:16 INFO - JS frame :: chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js :: testOpenCloseWindow :: line 44 08:05:16 INFO - JS frame :: resource://gre/modules/Task.jsm :: TaskImpl_run :: line 282 08:05:16 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Handler.prototype.process :: line 863 08:05:16 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: this.PromiseWalker.walkerLoop :: line 742 08:05:16 INFO - native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0 08:05:16 WARNING - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | Closed window should match previously opened window - Got [object ChromeWindow], expected null 08:05:16 INFO - Stack trace: 08:05:16 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 752 08:05:16 INFO - JS frame :: chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js :: testOpenCloseWindow :: line 45 08:05:16 INFO - JS frame :: resource://gre/modules/Task.jsm :: TaskImpl_run :: line 282 08:05:16 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Handler.prototype.process :: line 863 08:05:16 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: this.PromiseWalker.walkerLoop :: line 742 08:05:16 INFO - native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0 08:05:16 WARNING - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | Should have one customizable window - Got 2, expected 1 08:05:16 INFO - Stack trace: 08:05:16 INFO - JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 752 08:05:16 INFO - JS frame :: chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js :: testOpenCloseWindow :: line 51 08:05:16 INFO - JS frame :: resource://gre/modules/Task.jsm :: TaskImpl_run :: line 282 08:05:16 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Handler.prototype.process :: line 863 08:05:16 INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: this.PromiseWalker.walkerLoop :: line 742 08:05:16 INFO - native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0 08:05:16 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | Current window should be in window collection. 08:05:16 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | Closed window should not be in window collection. 08:05:16 INFO - TEST-INFO | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | Leaving test testOpenCloseWindow 08:05:16 INFO - TEST-INFO | MEMORY STAT vsize after test: 4209459200 08:05:16 INFO - TEST-INFO | MEMORY STAT residentFast after test: 730202112 08:05:16 INFO - INFO TEST-END | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_987492_window_api.js | finished in 1380ms
26, Ryan, really? :P
Version: 26 Branch → Trunk
*whistles*
Summary: Intermittent browser_987492_window_api.js | Should have gotten onWindowClosed event - Didn't expect null, but got it | Closed window should match previously opened window - Got [object ChromeWindow], expected null | Should have one customizable window ... → Intermittent browser_987492_window_api.js | Should have gotten onWindowClosed event - Didn't expect null, but got it (and more)
I'm hopeful the fix from bug 996988 will fix this bug, too.
See Also: → 996988
This looks fixed to me.
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: p=0 [qa-]
status-firefox30: --- → wontfix
status-firefox31: --- → affected
status-firefox32: --- → fixed
status-firefox-esr24: --- → unaffected
Depends on: 996988
Target Milestone: --- → Firefox 32
You need to log in before you can comment on or make changes to this bug.