1041569 - browser_windowRestore_perwindowpb.js leaks until shutdown when run as a directory instead of a full suite

Status: RESOLVED DUPLICATE of bug 1041537

(Firefox :: Session Restore, defect)

Description

[Vaibhav (:vaibhav1994)]

In bug 992911, we are working on running tests per directory, to make the tests more independent. When we do that, we see a browser_windowRestore_perwindowpb.js fails for bc1 jobs on linux debug, windows debug, and osx debug. Example: https://tbpl.mozilla.org/?tree=Try&rev=efdddb5118dd , https://tbpl.mozilla.org/?tree=Try&rev=8e0e61b09e07

Here's a direct link to one of the logs: https://tbpl.mozilla.org/php/getParsedLog.php?id=44177365&tree=Try&full=1

We are planning on switching to this mode of running tests in 2 weeks and would like to do what we can to fix the tests.

Comment 1

[Vaibhav (:vaibhav1994)]

Tim, can you figure out why this test is failing when we are running tests only in the directory: browser/components/sessionstore/test instead of the full test suite?

Comment 2

[Tim Taubert [:ttaubert] (inactive)]

Is there a good way to reproduce this locally? That would be a lot easier to investigate.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

I wish there was a way.  Many of these I have downloaded the builds/tests package from the try run and still was not able to reproduce the problem.  For this specific one, I haven't tried that, but I could.  Otherwise, we have to do some try server print statement debugging.

There are ways to shortcut try server and adjust the test without a new push, you can just do artitrary build triggers.

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

Tim, I do not see this anymore, any objections to closing thas as a duplicate of bug 1041537

Comment 5

[Tim Taubert [:ttaubert] (inactive)]

No objections!

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

done!  thanks for the awesome fix.