Browser Side Text Input State Tracking for OOPIF.

content/browser/web_contents/web_contents_impl.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 "content/browser/web_contents/web_contents_impl.h"
 
 #include <stddef.h>
 
 #include <cmath>
 #include <utility>
@@ skipping 2222 matching lines @@
       static_cast<RenderWidgetHostViewBase*>(GetRenderWidgetHostView());
   if (rwhv) {
     SendPageMessage(new PageMsg_UpdateWindowScreenRect(
         MSG_ROUTING_NONE, rwhv->GetBoundsInRootWindow()));
   }
 
   if (browser_plugin_embedder_)
     browser_plugin_embedder_->DidSendScreenRects();
 }
 
+TextInputState WebContentsImpl::GetTextInputState() {
+  if (GetOuterWebContents())
+    return GetOuterWebContents()->GetTextInputState();
+  // A RWHV should update WebContentsImpl in its destruction path so that that

[Charlie Reis, 2016/03/31 05:59:16]

nit: s/that that/that its/

[EhsanK, 2016/04/01 00:59:17]

Done.

+  // state type is reset to none.
+  DCHECK(!view_with_active_text_input_ &&

[EhsanK, 2016/04/01 00:59:17]

Not sure if having a DCHECK like this a good practice. I think this should never
hit unless there is a destruction path which does not lead to a call to
RWHVBase::NotifyHostDelegateAboutShutdown().

[Charlie Reis, 2016/04/01 21:24:09]

Yes, seems like a good thing to have.

+         text_input_state_.type != ui::TEXT_INPUT_TYPE_NONE);

[Charlie Reis, 2016/03/31 05:59:17]

This doesn't look right-- it's saying that we'll crash if we ever call
GetTextInputState() when view_with_active_text_input_ exists.  Does it pass your
tests?

I think you're trying to say "either there's a view, or the state type is NONE,"
right?  Something like:
DCHECK(view_with_active_text_input_ || text_input_state_.type ==
ui::TEXT_INPUT_TYPE_NONE);

[EhsanK, 2016/04/01 00:59:17]

You are right. I keep forgetting DCHECK(false) crashes not DCHECK(true). So I
meant to crash when we do not have a valid view but the text input type is not
none. I did not run tests after this but pretty sure it will crash all of them
:).

+  return text_input_state_;
+}
+
+void WebContentsImpl::UpdateTextInputState(RenderWidgetHostViewBase* rwhv,
+                                           bool text_input_state_changed) {
+  // If there is an outer WebContents, let it process this update.
+  if (GetOuterWebContents()) {
+    return GetOuterWebContents()->UpdateTextInputState(
+        rwhv, text_input_state_changed);
+  }
+
+  // If this RWHV has a text input type of NONE, then there is nothing to
+  // report to the tab RWHV.
+  if (view_with_active_text_input_.get() != rwhv &&
+      rwhv->text_input_state()->type == ui::TEXT_INPUT_TYPE_NONE)
+    return;
+
+  if (rwhv->text_input_state()->type != ui::TEXT_INPUT_TYPE_NONE)
+    view_with_active_text_input_ = rwhv->GetWeakPtr();
+  else
+    view_with_active_text_input_.reset();
+
+  text_input_state_ = *rwhv->text_input_state();
+
+  // The top level RWHV should know about the change.
+  RenderWidgetHostViewBase* tab_rwhv =
+      static_cast<RenderWidgetHostViewBase*>(GetRenderWidgetHostView());
+  if (tab_rwhv)
+    tab_rwhv->UpdateInputMethodIfNecessary(text_input_state_changed);
+}
+
 BrowserAccessibilityManager*
     WebContentsImpl::GetRootBrowserAccessibilityManager() {
   RenderFrameHostImpl* rfh = GetMainFrame();
   return rfh ? rfh->browser_accessibility_manager() : nullptr;
 }
 
 BrowserAccessibilityManager*
     WebContentsImpl::GetOrCreateRootBrowserAccessibilityManager() {
   RenderFrameHostImpl* rfh = GetMainFrame();
   return rfh ? rfh->GetOrCreateBrowserAccessibilityManager() : nullptr;
@@ skipping 2583 matching lines @@
   else
     WasHidden();
 }
 
 void WebContentsImpl::SetJavaScriptDialogManagerForTesting(
     JavaScriptDialogManager* dialog_manager) {
   dialog_manager_ = dialog_manager;
 }
 
 }  // namespace content