Browser Side Text Input State Tracking for OOPIF.

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index e02c77031447bd168bd78da1759ae6fdceb488a9..82ad4008a02cfcd262a4ed51af2e31c4b9062f6d 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1671,6 +1671,22 @@ RenderWidgetHostImpl* WebContentsImpl::GetFocusedRenderWidgetHost(
   return RenderWidgetHostImpl::From(view->GetRenderWidgetHost());
 }
 
+RenderWidgetHostViewBase* WebContentsImpl::GetFocusedView() {
+  FrameTreeNode* focused_frame = frame_tree_.GetFocusedFrame();
+  if (!focused_frame)
+    return nullptr;
+
+  RenderFrameHostImpl* rfhi = focused_frame->current_frame_host();
+  if (rfhi->GetRenderWidgetHost())
+    return rfhi->GetRenderWidgetHost()->GetView();
+  else if (rfhi->GetRenderViewHost())

[Charlie Reis, 2016/02/17 06:22:14]

nit: No else needed after return.

Also, this looks wrong according to the comment on GetRenderWidgetHost.  That
already returns the RVH's RWH for the main frame (making this branch
unnecessary), but it returns null for subframes that don't own their own widget
(which aren't handled here).

[kenrb, 2016/02/17 23:05:53]

I hadn't noticed this, but right. I think everything from line 1679 to the end
of the method can be replaced by
return focused_frame->current_frame_host()->GetView();

[EhsanK, 2016/02/18 05:48:01]

Yes this is the correct way. Thanks!

[EhsanK, 2016/02/18 05:48:01]

I guess using RFH::GetView() resolves all the issues.
Done.

+    return RenderWidgetHostImpl::From(rfhi->GetRenderViewHost()->GetWidget())
+        ->GetView();
+
+  NOTREACHED();
+  return nullptr;
+}
+
 void WebContentsImpl::EnterFullscreenMode(const GURL& origin) {
   // This method is being called to enter renderer-initiated fullscreen mode.
   // Make sure any existing fullscreen widget is shut down first.