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

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index e0a78f13cc0cfa418f7a77d94a49778acbb02e1f..47c4b0c24721a2f7bdb23a34749e925568fc7a2a 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -3722,6 +3722,9 @@ void RenderViewImpl::DidStopLoadingIcons() {
 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 {