WK2 Icon Database API needs to be fleshed out At least by bringing it to parity with the WK1 API in terms of notifications, etc.
Created attachment 87945 [details] Patch v1
Comment on attachment 87945 [details] Patch v1 View in context: https://bugs.webkit.org/attachment.cgi?id=87945&action=review > Source/WebKit2/UIProcess/WebIconDatabase.cpp:208 > +{ Stray whitespace crept in here. > Source/WebKit2/UIProcess/WebIconDatabase.cpp:220 > + RefPtr<WebURL> url = WebURL::create(pageURL); > + m_iconDatabaseClient.didChangeIconForPageURL(this, url.get()); I think this would read better without the local variable.
http://trac.webkit.org/changeset/82839