Factor IconController out of FrameLoader
Created attachment 96873 [details] Patch
Comment on attachment 96873 [details] Patch Attachment 96873 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/8826893
Created attachment 96877 [details] Patch
Comment on attachment 96877 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=96877&action=review This is awesome. Just one question :) > Source/WebCore/loader/icon/IconController.h:51 > + KURL url(); > + IconURLs urls(int iconTypes); Is there anything we can do to make these names more distinct?
(In reply to comment #4) > (From update of attachment 96877 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=96877&action=review > > This is awesome. Just one question :) > > > Source/WebCore/loader/icon/IconController.h:51 > > + KURL url(); > > + IconURLs urls(int iconTypes); > > Is there anything we can do to make these names more distinct? Sure. We can rename urls to urlsForTypes.
Committed r88682: <http://trac.webkit.org/changeset/88682>