Bug 58072 – URLs accessed as a result of pushState/replaceState should count as "visited" and be in the global history

• Status: RESOLVED
• Resolution: FIXED
• Component: History

[Brady Eidson]

URLs accessed as a result of pushState/replaceState should count as "visited" and be in the global history

See attached example, which works fine in Firefox 4 and Chrome.

Chrome is likely managing it themselves, but this should be something WebKit handles internally.

[Brady Eidson]

Created attachment 88678 [details]
Example showing bug (links should get visited, and should show up in Safari's history menu)

[Brady Eidson]

<rdar://problem/9251515>

[Brady Eidson]

Created attachment 89131 [details]
Patch v1

[Maciej Stachowiak]

Comment on attachment 89131 [details]
Patch v1

r=me

[Brady Eidson]

http://trac.webkit.org/changeset/83547

[Csaba Osztrogonác]

The new fast/loader/stateobjects/state-url-sets-links-visited.html fails on WK2 bot:
http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20%28WebKit2%20Tests%29/r83558%20%2810595%29/fast/loader/stateobjects/state-url-sets-links-visited-pretty-diff.html

[Brady Eidson]

(In reply to comment #6)
> The new fast/loader/stateobjects/state-url-sets-links-visited.html fails on WK2 bot:
> http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20%28WebKit2%20Tests%29/r83558%20%2810595%29/fast/loader/stateobjects/state-url-sets-links-visited-pretty-diff.html

Interesting.  Seems possible/likely that WK2 DRT doesn't do history tracking properly yet.

[Brady Eidson]

(In reply to comment #7)
> (In reply to comment #6)
> > The new fast/loader/stateobjects/state-url-sets-links-visited.html fails on WK2 bot:
> > http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20%28WebKit2%20Tests%29/r83558%20%2810595%29/fast/loader/stateobjects/state-url-sets-links-visited-pretty-diff.html
> 
> Interesting.  Seems possible/likely that WK2 DRT doesn't do history tracking properly yet.

WKTR does implement all of the related APIs, but it's returning the green unvisited link style, if I throw some alerts in.

I tested the patch and test manually in WK2, so I know there's nothing fundamentally wrong here.  I wonder if visited links round trip through the UI process and therefore we're getting hurt by IPC?  I don't know yet, still exploring.

[Brady Eidson]

(In reply to comment #8)
> (In reply to comment #7)
> > (In reply to comment #6)
> > > The new fast/loader/stateobjects/state-url-sets-links-visited.html fails on WK2 bot:
> > > http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20%28WebKit2%20Tests%29/r83558%20%2810595%29/fast/loader/stateobjects/state-url-sets-links-visited-pretty-diff.html
> > 
> > Interesting.  Seems possible/likely that WK2 DRT doesn't do history tracking properly yet.
> 
> WKTR does implement all of the related APIs, but it's returning the green unvisited link style, if I throw some alerts in.
> 
> I tested the patch and test manually in WK2, so I know there's nothing fundamentally wrong here.  I wonder if visited links round trip through the UI process and therefore we're getting hurt by IPC?  I don't know yet, still exploring.

Found 3 other tests that rely on the same layout test controller APIs that were already skipped.

FIled https://bugs.webkit.org/show_bug.cgi?id=58334 and added this test to the skipped list.