[Webkit-unassigned] [Bug 91832] [EFL][WK2] Implemented color chooser proxy and input picker interface

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 22 02:46:46 PDT 2012


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





--- Comment #76 from Kenneth Rohde Christiansen <kenneth at webkit.org>  2012-08-22 02:46:43 PST ---
(In reply to comment #75)
> (From update of attachment 159883 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=159883&action=review
> 
> >>> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_view.cpp:238
> >>> +}
> >> 
> >> Isn't it more important knowing that it was actually asked to hide, than whether it being the right type. Could you test the former?
> > 
> > Well, may I test the picker status here? I mean, whether showcolorPicker() is called or not.
> 
> Or, I can check ewk_view_color_picker_color_set() returns false here.
> 
> The application should do something here, so I'm not making many test here.

Why not just add a static bool wasHidden = false; then set it to true and test later

-- 
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