Bug 81696 – REGRESSION (r97596): Enabling :visited in layout test is not compatible with running arbitrary subsets of tests in arbitrary order

• Status: NEW
• Component: Tools / Tests

<http://trac.webkit.org/r97596> made render tree dumps expose visited link styling. When using run-webkit-tests, which URLs are visited when a given test is run is arbitrary (it depends on things such as the number of instances of the test program running, the number of tests before a given test, and the subset of tests being run), so results are inconsistent.

[Radar WebKit Bug Importer]

<rdar://problem/11084564>

Created attachment 132892 [details]
Work around test failures caused by this issue in some fast/block tests

[Jessie Berlin]

Comment on attachment 132892 [details]
Work around test failures caused by this issue in some fast/block tests

On IRC, Dan assured me that there are no other platform-specific or cross platform results that need to be updated.

Committed attachment 132892 [details] as in <http://trac.webkit.org/r111437>.

fast/block/float/float-in-float-hit-testing.html is another test affected by this

(In reply to comment #5)
> fast/block/float/float-in-float-hit-testing.html is another test affected by this

although the problem there seems to be inconsistency between DumpRenderTree and WebKitTestRunner.

[Eric Seidel (no email)]

Attachment 132892 [details] was posted by a committer and has review+, assigning to Dan Bernstein for commit.

(In reply to comment #7)
> Attachment 132892 [details] was posted by a committer and has review+, assigning to Dan Bernstein for commit.

See comment #4, which states specifically the attachment and the revision in which it was committed.

[Eric Seidel (no email)]

It's just my automated script, assigning patches to committers if they're in the commit queue.

I assume that since this has been committed it should be closed?

[Eric Seidel (no email)]

Or alternatively, we should obsolete the above patch now that its' been landed (since I assume you're waiting for hte actual regression to be fixed?)

[Eric Seidel (no email)]

(None of this is a big deal.  Just explaining why the scripts (and people) see this patch as being in the pending-commit list.)