Summary: | [chromium] Add a method didChangeFormState to WebViewClient. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Oli Lan <olilan> | ||||||
Component: | New Bugs | Assignee: | Oli Lan <olilan> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, dglazkov, eric, fishd, jamesr, tkent+wkapi, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Oli Lan
2012-07-04 09:08:17 PDT
Created attachment 150811 [details]
Patch
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI. Comment on attachment 150811 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=150811&action=review > Source/WebKit/chromium/src/ChromeClientImpl.cpp:845 > + if (node->focused() && m_webView->client()) > + m_webView->client()->didChangeFocusedFormNodeState(); Should we pass |node| to the client, or does it not care? Should we have a client callback that we call unconditionally and let the embedder test for node->focused() ? Comment on attachment 150811 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=150811&action=review >> Source/WebKit/chromium/src/ChromeClientImpl.cpp:845 >> + m_webView->client()->didChangeFocusedFormNodeState(); > > Should we pass |node| to the client, or does it not care? Should we have a client callback that we call unconditionally and let the embedder test for node->focused() ? For the current purposes of the Android port this is sufficient, but passing the node unconditionally and allowing the client to perform whatever logic is required makes sense. Happy to make this change. Yeah, let's do that. It will probably save us from having to add another callback later. Created attachment 150935 [details]
Patch
OK, the new patch implements the above. As WebNode did not have a focused method, the patch also adds a focused method to WebNode to allow the new method to be used as discussed. Comment on attachment 150935 [details]
Patch
Thanks!
Comment on attachment 150935 [details] Patch Clearing flags on attachment: 150935 Committed r121904: <http://trac.webkit.org/changeset/121904> All reviewed patches have been landed. Closing bug. |