493962 - Intermittent failure of test_bug_461710.html: Visited link coloring should work outside of private mode - got "rgb(0, 0, 255)", expected "rgb(255, 0, 0)"

Status: RESOLVED FIXED

(Toolkit :: Places, defect)

Description

[(no longer active)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.5/1242824775.1242830393.10558.gz
OS X 10.5.2 mozilla-1.9.1 unit test on 2009/05/20 06:06:15

*** 77113 INFO Running /tests/toolkit/components/places/tests/test_bug_461710.html...
*** 77114 INFO TEST-PASS | /tests/toolkit/components/places/tests/test_bug_461710.html | Test #4 should be run outside of private mode
*** 77115 INFO TEST-PASS | /tests/toolkit/components/places/tests/test_bug_461710.html | Visited link coloring should work outside of private mode
*** 77116 INFO TEST-PASS | /tests/toolkit/components/places/tests/test_bug_461710.html | Test #5 should be run inside of private mode
*** 77117 INFO TEST-PASS | /tests/toolkit/components/places/tests/test_bug_461710.html | Visited link coloring should not work inside of private mode
*** 77118 INFO TEST-PASS | /tests/toolkit/components/places/tests/test_bug_461710.html | Test #6 should be run outside of private mode
*** 77119 ERROR TEST-UNEXPECTED-FAIL | /tests/toolkit/components/places/tests/test_bug_461710.html | Visited link coloring should work outside of private mode - got "rgb(0, 0, 255)", expected "rgb(255, 0, 0)"

Comment 1

[Daniel Holbert [:dholbert]]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.5/1243015194.1243022422.25301.gz
Linux mozilla-1.9.1 unit test on 2009/05/22 10:59:54

Comment 2

[Shawn Wilsher :sdwilsh]

Linux mozilla-central test debug mochitests-5/5 [testfailed] Started 14:54, finished 15:22
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256248445.1256250062.12045.gz&fulltext=1

Comment 3

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256652729.1256654121.12633.gz
Linux mozilla-central test debug mochitests-5/5 on 2009/10/27 07:12:09

Comment 4

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256742654.1256744027.28776.gz
Linux mozilla-central test debug mochitests-5/5 on 2009/10/28 08:10:54

Comment 5

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256767245.1256768955.25142.gz
Linux mozilla-central test debug mochitests-5/5 on 2009/10/28 15:00:45

Comment 6

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256850824.1256852782.22076.gz
Linux mozilla-central test debug mochitests-5/5 on 2009/10/29 14:13:44

Comment 7

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256926935.1256927742.27367.gz
Linux mozilla-central test debug mochitests-5/5 on 2009/10/30 11:22:15

Comment 8

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1257127216.1257128429.4172.gz
Linux mozilla-central test debug mochitests-5/5 on 2009/11/01 18:00:16

Comment 9

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1257162667.1257164054.3980.gz
Linux mozilla-central test debug mochitests-5/5 on 2009/11/02 03:51:07

Comment 10

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1257202426.1257204133.18372.gz
Linux mozilla-central test debug mochitests-5/5 on 2009/11/02 14:53:46

Comment 11

[(no longer active)]

FWIW, my patch in bug 514214 turns this from an intermittent problem into a consistently reproducible problem, and mak is investigating it.  So far we know that the problem has something to do with places lazy queues.  See bug 514214 comment 23 for more details.

Comment 12

[Shawn Wilsher :sdwilsh]

We should probably rewrite this test to poll the visitedness of a link for a duration of time instead of assuming it is synchronous.

Comment 13

[Marco Bonardo [:mak]]

(In reply to comment #12)
> We should probably rewrite this test to poll the visitedness of a link for a
> duration of time instead of assuming it is synchronous.

you should file this as a new bug and block on the new isVisited, does not look like related to this actual random failure, even if that's a pretty valid point.

Comment 14

[Marco Bonardo [:mak]]

If we won't see the failure for a week or so, we can close this, as solved by bug 514214

Comment 15

[Shawn Wilsher :sdwilsh]

Per comment 14, we can close this as fixed by bug 514214.