Decouple RVH creation from CrossProcessFrameConnector.

content/browser/frame_host/cross_process_frame_connector.cc

 // Copyright 2014 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/frame_host/cross_process_frame_connector.h"
 
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/browser/frame_host/render_widget_host_view_child_frame.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
 #include "content/common/frame_messages.h"
 #include "content/common/gpu/gpu_messages.h"
 #include "third_party/WebKit/public/web/WebInputEvent.h"
 
 namespace content {
 
 CrossProcessFrameConnector::CrossProcessFrameConnector(
     RenderFrameHostImpl* frame_proxy_in_parent_renderer)
     : frame_proxy_in_parent_renderer_(frame_proxy_in_parent_renderer),
       view_(NULL),
       device_scale_factor_(1) {
+  LOG(ERROR) << "CPFC::CPFC[" << this << "]:"
+    << " rfh:" << frame_proxy_in_parent_renderer;
+
   frame_proxy_in_parent_renderer->set_cross_process_frame_connector(this);
 }
 
 CrossProcessFrameConnector::~CrossProcessFrameConnector() {
   if (view_)
     view_->set_cross_process_frame_connector(NULL);
 }
 
 bool CrossProcessFrameConnector::OnMessageReceived(const IPC::Message& msg) {
   bool handled = true;
@@ skipping 85 matching lines @@
 void CrossProcessFrameConnector::OnReclaimCompositorResources(
     const FrameHostMsg_ReclaimCompositorResources_Params& params) {
   RenderWidgetHostImpl::SendReclaimCompositorResources(params.route_id,
                                                        params.output_surface_id,
                                                        params.renderer_host_id,
                                                        params.ack);
 }
 
 void CrossProcessFrameConnector::OnInitializeChildFrame(gfx::Rect frame_rect,
                                                         float scale_factor) {
+  LOG(ERROR) << "CPFC::OnInitializeChildFrame[" << this << "]";
   if (scale_factor != device_scale_factor_) {
     device_scale_factor_ = scale_factor;
     if (view_) {
       RenderWidgetHostImpl* child_widget =
           RenderWidgetHostImpl::From(view_->GetRenderWidgetHost());
       child_widget->NotifyScreenInfoChanged();
+      LOG(ERROR) << "CPFC::OnInitializeChildFrame[" << this << "]: notify";
     }
   }
 
   if (!frame_rect.size().IsEmpty()) {
     child_frame_rect_ = frame_rect;
-    if (view_)
+    if (view_) {
       view_->SetSize(frame_rect.size());
+      LOG(ERROR) << "CPFC::OnInitializeChildFrame[" << this << "]: set size";
+    }
   }
 }
 
 gfx::Rect CrossProcessFrameConnector::ChildFrameRect() {
   return child_frame_rect_;
 }
 
 void CrossProcessFrameConnector::OnForwardInputEvent(
     const blink::WebInputEvent* event) {
+  LOG(ERROR) << "CPFC::OnForwardInputEvent[" << this << "]: " << view_;
   if (!view_)
     return;
 
   RenderWidgetHostImpl* child_widget =
       RenderWidgetHostImpl::From(view_->GetRenderWidgetHost());
   RenderWidgetHostImpl* parent_widget =
       frame_proxy_in_parent_renderer_->render_view_host();
 
   if (blink::WebInputEvent::isKeyboardEventType(event->type)) {
     if (!parent_widget->GetLastKeyboardEvent())
       return;
     NativeWebKeyboardEvent keyboard_event(
         *parent_widget->GetLastKeyboardEvent());
     child_widget->ForwardKeyboardEvent(keyboard_event);
+    LOG(ERROR) << "CPFC::OnForwardInputEvent[" << this << "]: kbd";
     return;
   }
 
   if (blink::WebInputEvent::isMouseEventType(event->type)) {
     child_widget->ForwardMouseEvent(
         *static_cast<const blink::WebMouseEvent*>(event));
+    LOG(ERROR) << "CPFC::OnForwardInputEvent[" << this << "]: mouse";
     return;
   }
 
   if (event->type == blink::WebInputEvent::MouseWheel) {
     child_widget->ForwardWheelEvent(
         *static_cast<const blink::WebMouseWheelEvent*>(event));
+    LOG(ERROR) << "CPFC::OnForwardInputEvent[" << this << "]: wheel";
     return;
   }
 }
 
 }  // namespace content