[Webkit-unassigned] [Bug 48177] Expose |sendChangeEvent| parameter in WebInputElement::setValue() API

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 28 23:03:49 PDT 2010


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





--- Comment #6 from Ilya Sherman <isherman at chromium.org>  2010-10-28 23:03:48 PST ---
(In reply to comment #5)
> (From update of attachment 71631 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=71631&action=review
> 
> > WebKit/chromium/public/WebInputElement.h:69
> > +        WEBKIT_API void setValue(const WebString&); // TODO(isherman): Remove once chromium code is updated.
> 
> We use some default arguments in WebKit API.
> It might be ok to use default argument though Chromium style doesn't allow it.  I'm not 100% sure.

I'm not really opposed to using a default argument here -- I was just trying to stay consistent with Chromium style =)

What say ye, wise guardians of the Chromium WebKit API?

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