diff options
| author | jcivelli@google.com <jcivelli@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-01 16:58:49 +0000 |
|---|---|---|
| committer | jcivelli@google.com <jcivelli@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-01 16:58:49 +0000 |
| commit | 08e9e134bca95868aa4679b67b4c7a4e1695fced (patch) | |
| tree | eed4bcda9a1cee28a7c3748fe9b363d66919d643 /chrome/renderer/render_view.cc | |
| parent | c001c4c5277e933720fdd4a2dedd152d10de118a (diff) | |
| download | chromium_src-08e9e134bca95868aa4679b67b4c7a4e1695fced.zip chromium_src-08e9e134bca95868aa4679b67b4c7a4e1695fced.tar.gz chromium_src-08e9e134bca95868aa4679b67b4c7a4e1695fced.tar.bz2 | |
Cleaning up the node focus notification.
The notification that the focus changed in the page now uses webkit::WebViewClient::focusedNodeChanged() implemented by RenderView.
It was previously relying on the more general webkit::WebWidgetClient::didFocus() implemented in RenderWidget. (that should ideally only be fired when the frame gets focused, not everytime the focus in the frame changes).
BUG=None
TEST=Interactive UI tests should still pass (especially BrowserFocus*)
Review URL: http://codereview.chromium.org/2301001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48638 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/render_view.cc')
| -rwxr-xr-x | chrome/renderer/render_view.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc index f8f3c39..d00d948 100755 --- a/chrome/renderer/render_view.cc +++ b/chrome/renderer/render_view.cc @@ -1979,6 +1979,10 @@ void RenderView::focusPrevious() { Send(new ViewHostMsg_TakeFocus(routing_id_, true)); } +void RenderView::focusedNodeChanged(const WebNode& node) { + Send(new ViewHostMsg_FocusedNodeChanged(routing_id_)); +} + void RenderView::navigateBackForwardSoon(int offset) { Send(new ViewHostMsg_GoToEntryAtOffset(routing_id_, offset)); } |