diff options
| author | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-03 06:19:51 +0000 |
|---|---|---|
| committer | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-03 06:19:51 +0000 |
| commit | ac1293c00113fd7b034161e8903f25f98d7a22d4 (patch) | |
| tree | 1e8d76ba28a8d1b378c506a2a6af661dcc83c9f0 /webkit/glue/webframeloaderclient_impl.cc | |
| parent | 8973f52efb98f531369f0b9daba397b1bf26d07b (diff) | |
| download | chromium_src-ac1293c00113fd7b034161e8903f25f98d7a22d4.zip chromium_src-ac1293c00113fd7b034161e8903f25f98d7a22d4.tar.gz chromium_src-ac1293c00113fd7b034161e8903f25f98d7a22d4.tar.bz2 | |
We used to store/restore the frame and node that were focused last when a age would lose/gain focus.
This patches lets WebKit takes care of it by using the new FocusController::setFocused method.
BUG=http://crbug.com/15777
TEST=Open www.google.com, the focus should be on the text-field (focus ring + blinking caret). Activate another window. The page should not show the text-field with the focus. Switch tabs, make sure the focus is remembered correctly. Do the same tests in a page with frames. Run the layout tests.
Review URL: http://codereview.chromium.org/151195
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19899 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue/webframeloaderclient_impl.cc')
| -rw-r--r-- | webkit/glue/webframeloaderclient_impl.cc | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/webkit/glue/webframeloaderclient_impl.cc b/webkit/glue/webframeloaderclient_impl.cc index 58fe059..3a65c91 100644 --- a/webkit/glue/webframeloaderclient_impl.cc +++ b/webkit/glue/webframeloaderclient_impl.cc @@ -644,10 +644,6 @@ void WebFrameLoaderClient::dispatchDidChangeLocationWithinPage() { void WebFrameLoaderClient::dispatchWillClose() { WebViewImpl* webview = webframe_->GetWebViewImpl(); - // Make sure WebViewImpl releases the references it uses to restore focus. - // If we didn't do this, WebViewImpl might try to restore focus to an invalid - // element. - webview->ReleaseFocusReferences(); WebViewDelegate* d = webview->delegate(); if (d) d->WillCloseFrame(webview, webframe_); |