Bug 27586 – URL appears in back/forward button menu instead of title for items with custom representation

• Component: History
• Status: RESOLVED FIXED

[Darin Adler]

To reproduce:

1. visit bookmarks
2. visit some other page
3. press and hold on the Back button in the location field

The menu contains "bookmarks:". It should contain "Bookmarks".

The history code expects that the DocumentLoader will store a copy of the title. This is subtly incompatible with having a title method on the representation -- we can ask what the title is, but we won't know when the title changes so we won't be able to update the title in the DocumentLoader.

Here are some ways to fix this:

    1) change the history item code to call via the client to get the title, so it can pass through the representation object in Mac WebKit

    2) have the WebDataSource call setTitle on the DocumentLoader, passing the current title from the representation, exactly once, after the representation gets its setDataSource: call

    3) same as (2), but after every method call from WebDataSource to the representation

    4) change the representation API so there's a setTitle: method on WebDataSource instead of a title method on the representation

I am planning to do (2) largely because we want to phase out this representation feature anyway.

[Darin Adler]

<rdar://problem/5060337>

[Darin Adler]

Created attachment 33307 [details]
patch

[Brady Eidson]

Comment on attachment 33307 [details]
patch

Nice!

[Darin Adler]

http://trac.webkit.org/changeset/46298
http://trac.webkit.org/changeset/46300