Comment #7 on issue 15777 by bugdro...@chromium.org: Need to update how we  
handle focusing to match new WebKit code
http://code.google.com/p/chromium/issues/detail?id=15777

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=19899

------------------------------------------------------------------------
r19899 | jcam...@chromium.org | 2009-07-02 23:19:51 -0700 (Thu, 02 Jul  
2009) | 8 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/render_view.cc?r1=19899&r2=19898
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webframe_impl.cc?r1=19899&r2=19898
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webframeloaderclient_impl.cc?r1=19899&r2=19898
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webview.h?r1=19899&r2=19898
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webview_impl.cc?r1=19899&r2=19898
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webview_impl.h?r1=19899&r2=19898

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
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to