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

content/browser/host_zoom_map_impl_browsertest.cc

 // Copyright 2015 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/host_zoom_map_impl.h"
 
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/test/content_browser_test.h"
+#include "content/public/test/content_browser_test_utils.h"
 #include "content/shell/browser/shell.h"
+#include "net/dns/mock_host_resolver.h"
 #include "url/gurl.h"
 
 namespace content {
 
 class HostZoomMapImplBrowserTest : public ContentBrowserTest {
+ protected:
+  void SetUpOnMainThread() override {
+    host_resolver()->AddRule("*", "127.0.0.1");
+    ASSERT_TRUE(embedded_test_server()->Start());
+  }
 };
 
 void RunTestForURL(const GURL& url,
                    Shell* shell,
                    double host_zoom_level,
                    double temp_zoom_level) {
   shell->LoadURL(url);
   WebContents* web_contents = shell->web_contents();
 
   HostZoomMapImpl* host_zoom_map = static_cast<HostZoomMapImpl*>(
@@ skipping 17 matching lines @@
   // web_contents.
   host_zoom_map->ClearTemporaryZoomLevel(render_process_id, view_id);
 }
 
 // Test to make sure that GetZoomForView() works properly for zoom levels
 // stored by host value, and can distinguish temporary zoom levels from
 // these.
 IN_PROC_BROWSER_TEST_F(HostZoomMapImplBrowserTest, GetZoomForView_Host) {
   GURL url("http://abc.com");
 
+  // We must navigate so the WebContents has a committed entry.
+  EXPECT_TRUE(NavigateToURL(shell(), url));

[alexmos, 2016/04/05 18:00:53]

It seems RunTestForURL will now redundantly navigate to this url again.

[wjmaclean, 2016/04/05 20:22:00]

Hmmm, I missed that ... I'll remove it from RunTestForURL().

+
   HostZoomMap* host_zoom_map =
       HostZoomMap::GetForWebContents(shell()->web_contents());
 
   double default_zoom_level = host_zoom_map->GetDefaultZoomLevel();
   double host_zoom_level = default_zoom_level + 1.0;
   double temp_zoom_level = default_zoom_level + 2.0;
 
   host_zoom_map->SetZoomLevelForHost(url.host(), host_zoom_level);
 
   RunTestForURL(url, shell(), host_zoom_level, temp_zoom_level);
 }
 
 // Test to make sure that GetZoomForView() works properly for zoom levels
 // stored by host and scheme values, and can distinguish temporary zoom levels
 // from these.
 IN_PROC_BROWSER_TEST_F(HostZoomMapImplBrowserTest,
                        GetZoomForView_HostAndScheme) {
   GURL url("http://abc.com");
 
+  // We must navigate so the WebContents has a committed entry.
+  EXPECT_TRUE(NavigateToURL(shell(), url));
+
   HostZoomMap* host_zoom_map =
       HostZoomMap::GetForWebContents(shell()->web_contents());
 
   double default_zoom_level = host_zoom_map->GetDefaultZoomLevel();
   double host_zoom_level = default_zoom_level + 1.0;
   double temp_zoom_level = default_zoom_level + 2.0;
 
   host_zoom_map->SetZoomLevelForHostAndScheme(url.scheme(), url.host(),
                                               host_zoom_level);
 
   RunTestForURL(url, shell(), host_zoom_level, temp_zoom_level);
 }
 
 }  // namespace content