[Webkit-unassigned] [Bug 38508] [chromium] WebViewClient should have a notification when a new node gets the focus

Thu May 6 11:36:39 PDT 2010

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

Darin Fisher (:fishd, Google) <fishd at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #54998|review?                     |review-
               Flag|                            |

--- Comment #2 from Darin Fisher (:fishd, Google) <fishd at chromium.org>  2010-05-06 11:36:39 PST ---
(From update of attachment 54998)
WebKit/chromium/public/WebViewClient.h:106
 +      virtual void focusNodeChanged(const WebNode&) { }
How about moving this down near the other focus related methods?
(In the UI section.)

WebKit/chromium/src/ChromeClientImpl.cpp:218
 +      m_webView->client()->focusNodeChanged(WebNode(node));
focusedNodeChanged seems like a better name for this method.

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