As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
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...
Product: SeaMonkey
Classification: Client Software
Component: Tabbed Browser (show other bugs)
: Trunk
: All All
: P2 major (vote)
: seamonkey2.10
Assigned To:
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:
QA Whiteboard:
Iteration: ---
Points: ---

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

Description User image Serge Gautherie (:sgautherie) 2012-02-16 17:27:05 PST
OS X 10.6 comm-central-trunk debug test mochitest-other on 2012/02/16 00:52:55
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]
WINNT 5.2 comm-aurora debug test mochitest-other on 2012/02/14 20:46:15
is affected too.
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 User image 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 User image 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 User image 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 User image 2012-02-17 14:30:44 PST
Pushed changeset 8d7f0654db70 to comm-central.
Comment 6 User image Serge Gautherie (:sgautherie) 2012-02-18 01:12:29 PST
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/02/17 18:55:48


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