Ensure zoom level is not updated if main frame is WebRemoteFrame.

content/renderer/render_view_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/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/auto_reset.h"
@@ skipping 3704 matching lines @@
     if (!url.isEmpty())
       urls.push_back(
           FaviconURL(url, ToFaviconType(icon_urls[i].iconType()), sizes));
   }
   SendUpdateFaviconURL(urls);
 }
 
 void RenderViewImpl::UpdateWebViewWithDeviceScaleFactor() {
   if (!webview())
     return;
+  if (webview()->mainFrame()->isWebRemoteFrame())

[dcheng, 2015/12/22 18:27:36]

Is this a temporary workaround? Or is this one of those 'view method needs to
move to widget' things? Or some other category?

[nasko, 2015/12/22 18:31:06]

The whole zoom feature needs to be updated for out-of-process iframes, which
will move this around. I don't expect it to stay in RenderView and will move to
RenderFrame or possibly the equivalent of Page.

In the meantime, without out-of-process iframes, but with swapped out disabled,
we have a race where the zoom level IPC is sent from the browser right after the
swap out, which causes crashes.

[dcheng, 2015/12/22 18:35:13]

Maybe add a TODO to that effect?

+    return;
+
   if (IsUseZoomForDSFEnabled()) {
     webview()->setZoomFactorForDeviceScaleFactor(device_scale_factor_);
   } else {
     webview()->setDeviceScaleFactor(device_scale_factor_);
   }
   webview()->settings()->setPreferCompositingToLCDTextEnabled(
       PreferCompositingToLCDText(compositor_deps_, device_scale_factor_));
 }
 
 }  // namespace content