Allow zooming error pages.

content/public/browser/host_zoom_map.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_HOST_ZOOM_MAP_H_
 #define CONTENT_PUBLIC_BROWSER_HOST_ZOOM_MAP_H_
 
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/callback_list.h"
 #include "content/common/content_export.h"
+#include "url/gurl.h"
 
 namespace content {
 
+class NavigationEntry;
 class BrowserContext;
 class ResourceContext;
 class WebContents;
 
 // Maps hostnames to custom zoom levels.  Written on the UI thread and read on
 // any thread.  One instance per browser context. Must be created on the UI
 // thread, and it'll delete itself on the UI thread as well.
 // Zoom can be defined at three levels: default zoom, zoom for host, and zoom
 // for host with specific scheme. Setting any of the levels leaves settings
 // for other settings intact. Getting the zoom level starts at the most
@@ skipping 16 matching lines @@
   // if not applicable to |mode|.
   struct ZoomLevelChange {
     ZoomLevelChangeMode mode;
     std::string host;
     std::string scheme;
     double zoom_level;
   };
 
   typedef std::vector<ZoomLevelChange> ZoomLevelVector;
 
+  CONTENT_EXPORT static GURL GetURLFromEntry(const NavigationEntry*);

[Charlie Reis, 2014/10/28 16:36:22]

nit: Always name parameters.

Also add a comment hinting at why this is needed, rather than just using
NavigationEntry::GetURL.

[wjmaclean, 2014/10/28 17:37:04]

Done.

+
   CONTENT_EXPORT static HostZoomMap* GetDefaultForBrowserContext(
       BrowserContext* browser_context);
 
   // Returns the current zoom level for the specified WebContents. May be
   // temporary or host-specific.
   CONTENT_EXPORT static double GetZoomLevel(const WebContents* web_contents);
 
   // Sets the current zoom level for the specified WebContents. The level may
   // be temporary or host-specific depending on the particular WebContents.
   CONTENT_EXPORT static void SetZoomLevel(const WebContents* web_contents,
                                           double level);
 
+  // Send an IPC to refresh any displayed error page's zoom levels. Needs to
+  // be called since error pages don't get loaded via the normal channel.
+  CONTENT_EXPORT static void SendErrorPageZoomLevelRefresh(

[Charlie Reis, 2014/10/28 16:36:22]

This doesn't appear to be called outside of content/, so it should be in the
impl file rather than the public interface.

From http://www.chromium.org/developers/content-module/content-api:
"only expose methods in the public API that embedders need. If a method is only
used by other code in content, it belongs in foo_impl.h and not foo.h."

[wjmaclean, 2014/10/28 17:37:04]

With the moving of the code out of NavigationControllerImpl, this is now called
outside content/.

+      const WebContents* web_contents);
+
   // Copy the zoom levels from the given map. Can only be called on the UI
   // thread.
   virtual void CopyFrom(HostZoomMap* copy) = 0;
 
   // Here |host| is the host portion of URL, or (in the absence of a host)
   // the complete spec of the URL.
   // Returns the zoom for the specified |scheme| and |host|. See class
   // description for details.
   //
   // This may be called on any thread.
@@ skipping 62 matching lines @@
   virtual scoped_ptr<Subscription> AddZoomLevelChangedCallback(
       const ZoomLevelChangedCallback& callback) = 0;
 
  protected:
   virtual ~HostZoomMap() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_HOST_ZOOM_MAP_H_