RESOLVED FIXED 62509
Factor IconController out of FrameLoader 
https://bugs.webkit.org/show_bug.cgi?id=62509
Summary Factor IconController out of FrameLoader
Adam Barth
Reported 2011-06-12 02:34:41 PDT
Factor IconController out of FrameLoader
Attachments
Patch (42.53 KB, patch)
2011-06-12 02:38 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
Patch (42.57 KB, patch)
2011-06-12 04:11 PDT, Adam Barth 		japhet: review+
abarth: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2011-06-12 02:38:51 PDT
Created attachment 96873 [details] Patch
WebKit Review Bot
Comment 2 2011-06-12 03:52:47 PDT
Comment on attachment 96873 [details] Patch Attachment 96873 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/8826893
Adam Barth
Comment 3 2011-06-12 04:11:26 PDT
Created attachment 96877 [details] Patch
Nate Chapin
Comment 4 2011-06-13 11:39:04 PDT
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?
Adam Barth
Comment 5 2011-06-13 11:51:08 PDT
(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.
Adam Barth
Comment 6 2011-06-13 13:36:08 PDT
Committed r88682: <http://trac.webkit.org/changeset/88682>
Note You need to log in before you can comment on or make changes to this bug.