[UseZoomForDSF] Guest view support

content/renderer/browser_plugin/browser_plugin.cc

Index: content/renderer/browser_plugin/browser_plugin.cc
diff --git a/content/renderer/browser_plugin/browser_plugin.cc b/content/renderer/browser_plugin/browser_plugin.cc
index b56112781909e34a5864d55dca1824d3160e9e66..309ad7f761184e6aee54a9e28a2f75f6ec6eee28 100644
--- a/content/renderer/browser_plugin/browser_plugin.cc
+++ b/content/renderer/browser_plugin/browser_plugin.cc
@@ -129,7 +129,6 @@ void BrowserPlugin::OnSetChildFrameSurface(
 
   EnableCompositing(true);
   DCHECK(compositing_helper_.get());
-
   compositing_helper_->OnSetSurface(surface_id, frame_size, scale_factor,
                                     sequence);
 }
@@ -440,7 +439,11 @@ void BrowserPlugin::updateGeometry(const WebRect& window_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_)