Closed Bug 949227 Opened 7 years ago Closed 7 years ago

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug609700.js | Found an unexpected tab at the end of test run: about:blank

Categories

(Firefox :: Tabbed Browser, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 29
Tracking Status
firefox27 --- fixed
firefox28 --- fixed
firefox29 --- fixed
firefox-esr24 --- fixed
b2g-v1.2 --- fixed

People

(Reporter: smaug, Assigned: smaug)

References

(Blocks 1 open bug)

Details

(Whiteboard: [qa-])

Attachments

(1 file)

Attached patch patchSplinter Review
The test fails locally always, and looks like on try/linux quite often once
bug 930793 lands.
Attachment #8346228 - Flags: review?(gavin.sharp)
https://tbpl.mozilla.org/?tree=Try&rev=1f0ae9ca7c0e (contains various other stuff too)
And when I say fails locally always, I mean even without bug 930793.
How does it fail without this patch? There shouldn't be any reason to have to wait for delayedStartup to close the window...
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug609700.js | Found an unexpected tab at the end of test run: about:blank

https://tbpl.mozilla.org/php/getParsedLog.php?id=31764832&tree=Try&full=1#error0
<smaug> http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.js#967 is the reason
<smaug> that all happens in _delayedStartup

So the test needs to wait for delayedStartup because otherwise duplicateTabIn leaves a tab around in the opening window.
Attachment #8346228 - Flags: review?(gavin.sharp) → review+
https://hg.mozilla.org/mozilla-central/rev/21e875670068
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
Flags: in-testsuite+
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.