Redesigns the text input focus handling.

ui/views/controls/webview/webview.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/views/controls/webview/webview.h"
 
 #include "content/public/browser/browser_accessibility_state.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_view.h"
 #include "ipc/ipc_message.h"
 #include "ui/accessibility/ax_enums.h"
 #include "ui/accessibility/ax_view_state.h"
+#include "ui/base/ui_base_switches_util.h"
 #include "ui/events/event.h"
 #include "ui/views/accessibility/native_view_accessibility.h"
 #include "ui/views/controls/native/native_view_host.h"
 #include "ui/views/focus/focus_manager.h"
 #include "ui/views/views_delegate.h"
 
 namespace views {
 
 // static
 const char WebView::kViewClassName[] = "WebView";
@@ skipping 122 matching lines @@
   } else {
     web_contents()->GetView()->Focus();
   }
 }
 
 void WebView::AboutToRequestFocusFromTabTraversal(bool reverse) {
   if (web_contents())
     web_contents()->FocusThroughTabTraversal(reverse);
 }
 
+ui::TextInputClient* WebView::GetTextInputClient() {
+  if (!switches::IsNewTextInputFocusEnabled())

[msw, 2014/03/11 00:58:50]

nit: just make this condition part of the if statement below.

[Yuki, 2014/03/11 15:27:37]

Done.

+    return NULL;
+
+  if (web_contents() && !web_contents()->IsBeingDestroyed()) {
+    content::RenderWidgetHostView* host_view =
+        web_contents()->GetRenderWidgetHostView();
+    if (host_view)
+      return host_view->GetTextInputClient();
+  }
+  return NULL;
+}
+
 void WebView::GetAccessibleState(ui::AXViewState* state) {
   state->role = ui::AX_ROLE_GROUP;
 }
 
 gfx::NativeViewAccessible WebView::GetNativeViewAccessible() {
   if (web_contents()) {
     content::RenderWidgetHostView* host_view =
         web_contents()->GetRenderWidgetHostView();
     if (host_view)
       return host_view->GetNativeViewAccessible();
@@ skipping 18 matching lines @@
 }
 
 bool WebView::EmbedsFullscreenWidget() const {
   DCHECK(wc_owner_.get());
   return embed_fullscreen_widget_mode_enabled_;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // WebView, content::WebContentsObserver implementation:
 
+void WebView::RenderViewDeleted(content::RenderViewHost* render_view_host) {
+  // WebView::GetTextInputClient() delegates the text input handling to the

[msw, 2014/03/11 00:58:50]

This comment doesn't explain why you'd want to set the focused text input client
to |this| when the underlying render view host is deleted. I'm generally
confused by what's happening here and why.

[Yuki, 2014/03/11 15:27:37]

I've updated the comment trying to make it clearer.

Let me explain in pseudo code.

FocusManager::SetFocusedView(view) {
  TextInputClient* text_input_client = view->GetTextInputClient();
  // Remember the pointer of |text_input_client|.

  // Need to forget the pointer when the object dies.
}

WebView::GetTextInputClient() {
  return rwhv_->GetTextInputClient();
}

WebView::RenderViewDeleted() {
  // This method is called when |rwhv_| was deleted.
  // rwhv_->GetTextInputClient() must have been deleted, too.

  // Note that it's possible that |rwhv_| points to a new RWHV
  // and this WebView is still focused.

  // Tell the focus manager that |this->GetTextInputClient()|
  // will return a new TextInputClient.
  FocusManager->OnTextInputClientChanged(this);
}

[msw, 2014/03/11 23:24:37]

That makes sense, thanks for the explanation and comment.

+  // underlying content::RenderWidgetHostView.  So when it's destroyed, we have
+  // to change the focused text input client.
+  FocusManager* const focus_manager = GetFocusManager();
+  if (focus_manager)
+    focus_manager->OnTextInputClientChanged(this);
+}
+
 void WebView::RenderViewHostChanged(content::RenderViewHost* old_host,
                                     content::RenderViewHost* new_host) {
   FocusManager* const focus_manager = GetFocusManager();
   if (focus_manager && focus_manager->GetFocusedView() == this)
     OnFocus();
 }
 
 void WebView::WebContentsDestroyed(content::WebContents* web_contents) {
   // We watch for destruction of WebContents that we host but do not own. If we
   // own a WebContents that is being destroyed, we're doing the destroying, so
@@ skipping 83 matching lines @@
   if (!contents) {
     content::WebContents::CreateParams create_params(
         browser_context, NULL);
     return content::WebContents::Create(create_params);
   }
 
   return contents;
 }
 
 }  // namespace views