Add scheme to HostZoomMap

content/browser/host_zoom_map_impl.h

Index: content/browser/host_zoom_map_impl.h
diff --git a/content/browser/host_zoom_map_impl.h b/content/browser/host_zoom_map_impl.h
index 7840188c0d747399cd59fbb42d0871237aafe74f..2b00e44f8ae431ec6f73cebce5b18afd9c58d41c 100644
--- a/content/browser/host_zoom_map_impl.h
+++ b/content/browser/host_zoom_map_impl.h
@@ -31,7 +31,12 @@ class CONTENT_EXPORT HostZoomMapImpl : public NON_EXPORTED_BASE(HostZoomMap),
   // HostZoomMap implementation:
   virtual void CopyFrom(HostZoomMap* copy) OVERRIDE;
   virtual double GetZoomLevel(const std::string& host) const OVERRIDE;
+  virtual double GetZoomLevel(const std::string& scheme,
+                              const std::string& host) const OVERRIDE;
   virtual void SetZoomLevel(const std::string& host, double level) OVERRIDE;
+  virtual void SetZoomLevel(const std::string& scheme,

[sky, 2013/01/17 17:08:01]

Do we really need both of these (same comment for the getter). Wouldn't it be
cleaner to have only one method that takes the scheme and host?

[Denis Kuznetsov (DE-MUC), 2013/01/21 10:49:29]

Actually, the cleanest way to do this is to make HostZoomMap  accept GURL both
for Get and Set zoom levels, and let HostZoomMap to handle zoom-equivalence
classes of URLs (e.g. for http:// and https:// it is the URLs with same domain,
for internal settings pages that is all pages with chrome:// scheme, except for
/login/ and /oobe/ parts, etc).

However, as this is the part of content-api, I've tried to extend API, but leave
it backwards-compatible. 

+                            const std::string& host,
+                            double level) OVERRIDE;
   virtual double GetDefaultZoomLevel() const OVERRIDE;
   virtual void SetDefaultZoomLevel(double level) OVERRIDE;
 
@@ -58,9 +63,11 @@ class CONTENT_EXPORT HostZoomMapImpl : public NON_EXPORTED_BASE(HostZoomMap),
 
  private:
   typedef std::map<std::string, double> HostZoomLevels;
+  typedef std::map<std::string, HostZoomLevels> SchemeHostZoomLevels;
 
   // Copy of the pref data, so that we can read it on the IO thread.
   HostZoomLevels host_zoom_levels_;
+  SchemeHostZoomLevels scheme_host_zoom_levels_;
   double default_zoom_level_;
 
   struct TemporaryZoomLevel {