Fix page zoom to be frame-centric for out-of-process frames.

content/browser/frame_host/cross_process_frame_connector.cc

 // Copyright 2014 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/frame_host/cross_process_frame_connector.h"
 
 #include "cc/surfaces/surface.h"
 #include "cc/surfaces/surface_manager.h"
 #include "content/browser/compositor/surface_utils.h"
 #include "content/browser/frame_host/frame_tree.h"
@@ skipping 181 matching lines @@
 
   if (event->type == blink::WebInputEvent::MouseWheel) {
     view_->ProcessMouseWheelEvent(
         *static_cast<const blink::WebMouseWheelEvent*>(event));
     return;
   }
 }
 
 void CrossProcessFrameConnector::OnFrameRectChanged(
     const gfx::Rect& frame_rect) {
+  // TODO(wjmaclean) When changing the zoom of a WebView child without also
+  // changing the zoom of the embedder (e.g. using WebView.setZoom()), we
+  // shouldn't propagate this change in the frame rect. We need to find a way
+  // to detect when this happens.
+  // http://crbug.com/xxxxxx <- Don't land without filing this bug.
   if (!frame_rect.size().IsEmpty())
     SetRect(frame_rect);
 }
 
 void CrossProcessFrameConnector::OnVisibilityChanged(bool visible) {
   if (!view_)
     return;
 
   // If there is an inner WebContents, it should be notified of the change in
   // the visibility. The Show/Hide methods will not be called if an inner
@@ skipping 79 matching lines @@
 
   if (parent) {
     return static_cast<RenderWidgetHostViewBase*>(
         parent->current_frame_host()->GetView());
   }
 
   return nullptr;
 }
 
 }  // namespace content