Propagate auto-resize viewport values to OOPIF processes

content/browser/renderer_host/render_view_host_impl.cc

 // Copyright (c) 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/browser/renderer_host/render_view_host_impl.h"
 
 #include <set>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 915 matching lines @@
                                           const gfx::Size& max_size) {
   GetWidget()->SetAutoResize(true, min_size, max_size);
   Send(new ViewMsg_EnableAutoResize(GetRoutingID(), min_size, max_size));
 }
 
 void RenderViewHostImpl::DisableAutoResize(const gfx::Size& new_size) {
   GetWidget()->SetAutoResize(false, gfx::Size(), gfx::Size());
   Send(new ViewMsg_DisableAutoResize(GetRoutingID(), new_size));
   if (!new_size.IsEmpty())
     GetWidget()->GetView()->SetSize(new_size);
+  // This clears the cached value in the WebContents, so that OOPIFs will
+  // stop using it.
+  if (GetWidget()->delegate()) {
+    GetWidget()->delegate()->ResizeDueToAutoResize(GetWidget(),
+                                                   gfx::Size(0, 0));

[alexmos, 2017/06/02 17:03:18]

optional: can just use "gfx::Size()" here

[kenrb, 2017/06/02 17:23:27]

Yes, but I had to remove this call anyway because it broke a webview API test.
In the new PS I have added a separate call on the delegate interface, because we
don't want this to generate a resize notification to observers.

+  }
 }
 
 void RenderViewHostImpl::ExecuteMediaPlayerActionAtLocation(
   const gfx::Point& location, const blink::WebMediaPlayerAction& action) {
   Send(new ViewMsg_MediaPlayerActionAt(GetRoutingID(), location, action));
 }
 
 void RenderViewHostImpl::ExecutePluginActionAtLocation(
   const gfx::Point& location, const blink::WebPluginAction& action) {
   Send(new ViewMsg_PluginActionAt(GetRoutingID(), location, action));
@@ skipping 18 matching lines @@
 }
 
 void RenderViewHostImpl::ClosePageTimeout() {
   if (delegate_->ShouldIgnoreUnresponsiveRenderer())
     return;
 
   ClosePageIgnoringUnloadEvents();
 }
 
 }  // namespace content