[UseZoomForDSF] Guest view support

content/browser/frame_host/render_widget_host_view_child_frame.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/render_widget_host_view_child_frame.h"
 
 #include <algorithm>
 #include <utility>
 #include <vector>
 
@@ skipping 30 matching lines @@
       ack_pending_count_(0),
       frame_connector_(nullptr),
       weak_factory_(this) {
   if (use_surfaces_) {
     id_allocator_ = CreateSurfaceIdAllocator();
     if (host_->delegate() && host_->delegate()->GetInputEventRouter()) {
       host_->delegate()->GetInputEventRouter()->AddSurfaceIdNamespaceOwner(
           GetSurfaceIdNamespace(), this);
     }
   }
-
+  // Input for child frames are already scaled.
+  // TODO(oshima): This should be true for OOPIF.
+  host_->set_scale_input_to_viewport(false);

[wjmaclean, 2016/01/20 19:41:54]

If it should be true for OOPIF, can't we just do

host_->set_scale_input_to_viewport(AreCrossProcessFramesPossible() || <the new
WebView specific OOPIF flag>);

now, as opposed to later?

[oshima, 2016/01/20 21:38:49]

I looked into more, and I think we can just move this to
RenderWidgetHostViewGuest, because this is used only when scaling is necessary.
(and not used when oopif is used)

   host_->SetView(this);
 }
 
 RenderWidgetHostViewChildFrame::~RenderWidgetHostViewChildFrame() {
   if (!surface_id_.is_null())
     surface_factory_->Destroy(surface_id_);
 }
 
 void RenderWidgetHostViewChildFrame::InitAsChild(
     gfx::NativeView parent_view) {
@@ skipping 435 matching lines @@
 #endif
 }
 
 void RenderWidgetHostViewChildFrame::ClearCompositorSurfaceIfNecessary() {
   if (surface_factory_ && !surface_id_.is_null())
     surface_factory_->Destroy(surface_id_);
   surface_id_ = cc::SurfaceId();
 }
 
 }  // namespace content