[Webkit-unassigned] [Bug 119025] Rename WebColorChooserProxy

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 24 14:25:15 PDT 2013


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





--- Comment #7 from Brady Eidson <beidson at apple.com>  2013-07-24 14:25:06 PST ---
(In reply to comment #6)
> (From update of attachment 207410 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=207410&action=review
> 
> > Source/WebKit2/UIProcess/PageClient.h:190
> > +    virtual PassRefPtr<WebColorPickerUI> createColorChooserProxy(WebPageProxy*, const WebCore::Color& initialColor, const WebCore::IntRect&) = 0;
> 
> This method is poorly named now, isn't it?


And is there any advantage to calling the new object WebColorPickerUI instead of WebColorPicker, considering that the abstract concept of a WebColorPicker encapsulates more than just its UI?

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



More information about the webkit-unassigned mailing list