[UseZoomForDSF] Guest view support

content/renderer/browser_plugin/browser_plugin.cc

 // Copyright 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/renderer/browser_plugin/browser_plugin.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/command_line.h"
@@ skipping 111 matching lines @@
     int browser_plugin_instance_id,
     const cc::SurfaceId& surface_id,
     const gfx::Size& frame_size,
     float scale_factor,
     const cc::SurfaceSequence& sequence) {
   if (!attached())
     return;
 
   EnableCompositing(true);
   DCHECK(compositing_helper_.get());
-
   compositing_helper_->OnSetSurface(surface_id, frame_size, scale_factor,
                                     sequence);
 }
 
 void BrowserPlugin::SendSatisfySequence(const cc::SurfaceSequence& sequence) {
   BrowserPluginManager::Get()->Send(new BrowserPluginHostMsg_SatisfySequence(
       render_frame_routing_id_, browser_plugin_instance_id_, sequence));
 }
 
 void BrowserPlugin::UpdateDOMAttribute(const std::string& attribute_name,
@@ skipping 290 matching lines @@
     const IPC::Message& message) {
   return IPC_MESSAGE_CLASS(message) == BrowserPluginMsgStart;
 }
 
 void BrowserPlugin::updateGeometry(const WebRect& window_rect,
                                    const WebRect& clip_rect,
                                    const WebRect& unobscured_rect,
                                    const WebVector<WebRect>& cut_outs_rects,
                                    bool is_visible) {
   gfx::Rect old_view_rect = view_rect_;
-  view_rect_ = window_rect;
+  // Convert the window_rect_ in viewport to css.
+  WebRect rect_in_css(window_rect);
+  blink::WebView* webview = container()->element().document().frame()->view();
+  RenderView::FromWebView(webview)->convertViewportToWindow(&rect_in_css);

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

This sort of looks like we're converting a viewport_rect to a window_rect ... is
the input parameter to this function perhaps badly named? Should it be
|viewport_rect|?

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

I changed it in plugin_rect_in_viewport.  WDYT?

+  view_rect_ = rect_in_css;
 
   if (!ready_) {
     if (delegate_)
       delegate_->Ready();
     ready_ = true;
   }
 
   if (delegate_ && (view_rect_.size() != old_view_rect.size()))
     delegate_->DidResizeElement(view_rect_.size());
 
@@ skipping 191 matching lines @@
 bool BrowserPlugin::HandleMouseLockedInputEvent(
     const blink::WebMouseEvent& event) {
   BrowserPluginManager::Get()->Send(
       new BrowserPluginHostMsg_HandleInputEvent(browser_plugin_instance_id_,
                                                 view_rect_,
                                                 &event));
   return true;
 }
 
 }  // namespace content