RESOLVED FIXED 163082
[GTK] Expose WebKitDOMHTMLInputElement APIs for form autofill
https://bugs.webkit.org/show_bug.cgi?id=163082
Summary [GTK] Expose WebKitDOMHTMLInputElement APIs for form autofill
Michael Catanzaro
Reported 2016-10-06 14:03:35 PDT
Expose WebKitDOMHTMLInputElement APIs for form autofill These are needed to make autofill work properly with Angular forms
Attachments
Patch (4.70 KB, patch)
2016-10-06 14:13 PDT, Michael Catanzaro
darin: review+
cgarcia: commit-queue-
Michael Catanzaro
Comment 1 2016-10-06 14:13:19 PDT
Darin Adler
Comment 2 2016-10-06 19:55:18 PDT
Comment on attachment 290864 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=290864&action=review > Source/WebKit2/WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMHTMLInputElement.cpp:1013 > + WTF::String convertedValue = WTF::String::fromUTF8(value); > + WebKit::core(self)->setEditingValue(convertedValue); I would have done this as a 1-liner: WebKit::core(self)->setEditingValue(String::fromUTF8(value));
Carlos Garcia Campos
Comment 3 2016-10-07 00:41:40 PDT
Comment on attachment 290864 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=290864&action=review We still have the unstable api in 2.14, so we can add this to the unstable api in the stable branch, but the patch in trunk should have a since 2.16, I'll change that when merging in stable > Source/WebKit2/WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMHTMLInputElement.h:496 > + * Since: 2.14.1 This should be 2.16 > Source/WebKit2/WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMHTMLInputElement.h:506 > + * Since: 2.14.1 Ditto. > Source/WebKit2/WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMHTMLInputElement.h:516 > + * Since: 2.14.1 Ditto.
Michael Catanzaro
Comment 4 2016-10-07 02:31:04 PDT
Note You need to log in before you can comment on or make changes to this bug.