[Webkit-unassigned] [Bug 88665] Favicon URL list sent with favicon updated message contains out of date icon URLs

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=88665


Pete Williamson <petewil at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #147678|0                           |1
        is obsolete|                            |
 Attachment #147678|review?                     |
               Flag|                            |
 Attachment #148447|                            |review?
               Flag|                            |




--- Comment #16 from Pete Williamson <petewil at chromium.org>  2012-06-19 15:55:47 PST ---
Created an attachment (id=148447)
 --> (https://bugs.webkit.org/attachment.cgi?id=148447&action=review)
Patch with a proposed fix for the bug (with layout tests, more recently synced, no extra files, better style)

Addresses Kent's concern about not adding methods to the public interface of WebDocument by using "Internals" instead.  Now instead of using a dump routine, we have an internals routine that provides a list of the current Icon URLs attached to the head of a document as a string, and the unit test checks that string.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.