Last Comment Bug 728096 - [SeaMonkey] TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/docshell/test/navigation/browser_bug343515.js | Tab N should be inactive
: [SeaMonkey] TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/docshe...
Status: VERIFIED FIXED
[perma-orange]
:
Product: SeaMonkey
Classification: Client Software
Component: Tabbed Browser (show other bugs)
: Trunk
: All All
: P2 major (vote)
: seamonkey2.10
Assigned To: neil@parkwaycc.co.uk
:
:
Mentors:
Depends on: 684805
Blocks: SmTestFail
  Show dependency treegraph
 
Reported: 2012-02-16 17:27 PST by Serge Gautherie (:sgautherie)
Modified: 2012-02-18 01:12 PST (History)
3 users (show)
bugzillamozillaorg_serge_20140323: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
wontfix
fixed
fixed
verified


Attachments
Proposed patch (1.20 KB, patch)
2012-02-17 01:21 PST, neil@parkwaycc.co.uk
iann_bugzilla: review+
iann_bugzilla: approval‑comm‑aurora+
iann_bugzilla: approval‑comm‑beta+
Details | Diff | Splinter Review

Description Serge Gautherie (:sgautherie) 2012-02-16 17:27:05 PST
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1329382375.1329388034.29824.gz
OS X 10.6 comm-central-trunk debug test mochitest-other on 2012/02/16 00:52:55
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1329360749.1329365432.19045.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/02/15 18:52:29
{
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/docshell/test/navigation/browser_bug343515.js | Tab 0 should be inactive
[and more]
}

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey-Aurora/1329281175.1329285614.3539.gz
WINNT 5.2 comm-aurora debug test mochitest-other on 2012/02/14 20:46:15
is affected too.

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey-Beta/1329289731.1329293499.17853.gz
WINNT 5.2 comm-beta debug test mochitest-other on 2012/02/14 23:08:51
{
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/docshell/test/navigation/browser_bug343515.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/docshell/test/navigation/browser_bug343515.js | Found a tab after previous test timed out: http://mochi.test:8888/browser/docshell/test/navigation/bug343515_pg1.html
}
Most likely affected too, but can't tell in that state.
Comment 1 Serge Gautherie (:sgautherie) 2012-02-16 17:43:31 PST
Then I notice bug 684805: I would better let Neil have a look at the current situation.
Comment 2 neil@parkwaycc.co.uk 2012-02-17 01:21:13 PST
Created attachment 598171 [details] [diff] [review]
Proposed patch

I forgot to make background tabs inactive. Oops. This fixes that test too.
Comment 3 Ian Neal 2012-02-17 05:20:33 PST
Comment on attachment 598171 [details] [diff] [review]
Proposed patch

r=me, a=me for c-a/c-b too as other patch landed on 2.8
Comment 4 neil@parkwaycc.co.uk 2012-02-17 14:30:44 PST
Pushed changeset 8d7f0654db70 to comm-central.
Comment 6 Serge Gautherie (:sgautherie) 2012-02-18 01:12:29 PST
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1329533748.1329538419.29635.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/02/17 18:55:48

V.Fixed

Note You need to log in before you can comment on or make changes to this bug.