Ensure ime adapter gets connected to RWHVA

content/browser/android/render_widget_host_connector.cc

Index: content/browser/android/render_widget_host_connector.cc
diff --git a/content/browser/android/render_widget_host_connector.cc b/content/browser/android/render_widget_host_connector.cc
index 9f0e0bb3c4b96c6ffcd038f00c42f9be414f874d..d5da7d00a414a7de321ab7e78a28874cf10ba315 100644
--- a/content/browser/android/render_widget_host_connector.cc
+++ b/content/browser/android/render_widget_host_connector.cc
@@ -127,6 +127,10 @@ RenderWidgetHostConnector::Observer::GetRenderWidgetHostViewAndroid() const {
 RenderWidgetHostConnector::RenderWidgetHostConnector(WebContents* web_contents)
     : render_widget_observer_(new Observer(web_contents, this)) {}
 
+void RenderWidgetHostConnector::Initialize() {
+  render_widget_observer_->RenderViewReady();

[boliu, 2017/04/21 16:20:02]

what it does is correct, but the RenderViewReady call makes no sense. you have
to actually check it is ready before you fake the signal

[Jinsuk Kim, 2017/04/21 16:30:04]

It would be like this:
auto* rwhva = GetRenderWidgetViewAndroid();
if (rwhva)
  UpdateRenderWidgetHostView(rwhva);

But then |if| is kind of not necessary because if it is null, the call will be
just no-op. Choose to just use RenderViewReady. Do you see it done without using
it?

[boliu, 2017/04/21 16:33:39]

Is that in RenderViewReady or in initialize? It should be correct to add a
DCHECK that GetRenderWidgetViewAndroid is non-null in RVReady, instead of the
runtime check.

If it's in Initialize, then yeah, just call UpdateRenderWidgetHostView without
the null check.

[Jinsuk Kim, 2017/04/21 16:53:16]

Inititialize(). Done.

+}
+
 RenderWidgetHostConnector::~RenderWidgetHostConnector() {}
 
 RenderWidgetHostViewAndroid* RenderWidgetHostConnector::GetRWHVAForTesting()