Closed
Bug 949227
Opened 10 years ago
Closed 10 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)
Tracking
()
RESOLVED
FIXED
Firefox 29
People
(Reporter: smaug, Assigned: smaug)
References
Details
(Whiteboard: [qa-])
Attachments
(1 file)
863 bytes,
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
The test fails locally always, and looks like on try/linux quite often once bug 930793 lands.
Attachment #8346228 -
Flags: review?(gavin.sharp)
Assignee | ||
Comment 1•10 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=1f0ae9ca7c0e (contains various other stuff too)
Assignee | ||
Comment 2•10 years ago
|
||
And when I say fails locally always, I mean even without bug 930793.
Comment 3•10 years ago
|
||
How does it fail without this patch? There shouldn't be any reason to have to wait for delayedStartup to close the window...
Assignee | ||
Comment 4•10 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 https://tbpl.mozilla.org/php/getParsedLog.php?id=31764832&tree=Try&full=1#error0
Comment 5•10 years ago
|
||
<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.
Updated•10 years ago
|
Attachment #8346228 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/21e875670068
Comment 7•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/21e875670068
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
Comment 8•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/efa13bb58c41 https://hg.mozilla.org/releases/mozilla-beta/rev/8bb67e4b5d97 https://hg.mozilla.org/releases/mozilla-b2g26_v1_2/rev/a886971fe2a3 https://hg.mozilla.org/releases/mozilla-esr24/rev/f1baf5f642c8
status-b2g-v1.2:
--- → fixed
status-firefox27:
--- → fixed
status-firefox28:
--- → fixed
status-firefox29:
--- → fixed
status-firefox-esr24:
--- → fixed
Updated•10 years ago
|
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•