812417 - window tests are not properly waiting for full cleanup before other window tests are run

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect, P1)

Description

[Erik Vold [:erikvold] (please needinfo? me)]

This was causing an issue landing the patch for bug 803065 so I had to hack around the issue for a quick fix with https://github.com/erikvold/jetpack-sdk/commit/96d7cb0b51486048ade4437414e77437c46a4b5d  the better fix is obviously to fix all of the other tests, but that will take much more time.

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Dave said he has something that might help with this. Ccing him for posterity.

Comment 2

[Dave Townsend [:mossop]]

I was thinking of the fixes that are now in bug 819587 but looks like they aren't window tests.

Comment 3

[Alexandre Poirot [:ochameau]]

Attachment: Pull request 944

Here is a patch that may fix various intermittent we are seeing.

We will most likely want to fix test that are just calling close without waiting for any event... (if there is any?)

Comment 4

[[github robot]]

Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/ccd30b439997ec1081e596d16ca3e4a5f34c041a
Bug 812417: Listen for domwindowclosed instead of unload when closing a
toplevel window.

https://github.com/mozilla/addon-sdk/commit/b7111ece7367bea97d1582b935cd6bca2101fc85
Merge pull request #944 from ochameau/bug812417

Bug 812417: Listen for domwindowclosed instead of unload when closing a toplevel window. r=@erikvold

Comment 5

[[github robot]]

Commit pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/053d759deab7b4fb5964f0d9e46b46a8f4519901
Backout bug 812417 "Pull request #944 from ochameau/bug812417"

This reverts commit b7111ece7367bea97d1582b935cd6bca2101fc85, reversing
changes made to 8ba3879d832ecb554e27147e77bd4060b5bb70b8.

Comment 6

[Alexandre Poirot [:ochameau]]

Attachment: Pull request 948

The issue was that we were closing a child window in private browsing test.
I introduce a getToplevelWindow helper and use that in close() in order to ensure checking for the toplevel window when listening for domwindowclosed event.

Comment 7

[[github robot]]

Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/2c9bff26b5a8ce39ea2b5690a3c7d5a26c026e80
Bug 812417: Listen for domwindowclosed instead of unload when closing a toplevel window.

https://github.com/mozilla/addon-sdk/commit/c9afeec18e22101c5f90f4773caac4cb98a9a9d7
Merge pull request #948 from ochameau/bug812417-2

Bug 812417: Listen for domwindowclosed instead of unload when closing a toplevel window. r=@mossop

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

I'm going through the list of open bugs that github robot has commented in. Is this bug fixed, Alex?

Comment 9

[Alexandre Poirot [:ochameau]]

Looks like it.