Migrate HostZoomMap to live in StoragePartition.

content/browser/renderer_host/render_view_host_impl.cc

Index: content/browser/renderer_host/render_view_host_impl.cc
diff --git a/content/browser/renderer_host/render_view_host_impl.cc b/content/browser/renderer_host/render_view_host_impl.cc
index 3ab2edb0687ed6f1f95841c0a1b1a02a3990f776..ab0d3605d31a1a18aea93889ea74a263b1879a48 100644
--- a/content/browser/renderer_host/render_view_host_impl.cc
+++ b/content/browser/renderer_host/render_view_host_impl.cc
@@ -1113,8 +1113,8 @@ void RenderViewHostImpl::OnDocumentAvailableInMainFrame(
   if (!uses_temporary_zoom_level)
     return;
 
-  HostZoomMapImpl* host_zoom_map = static_cast<HostZoomMapImpl*>(
-      HostZoomMap::GetForBrowserContext(GetProcess()->GetBrowserContext()));
+  HostZoomMapImpl* host_zoom_map =
+      static_cast<HostZoomMapImpl*>(HostZoomMap::GetForSite(GetSiteInstance()));

[awong, 2014/08/12 21:07:06]

GetForSite() is incorrectly named...This is a SiteInstance, not a Site...

A "Site" actually has specific meaning of the top-level-domain + 1 level and the
url scheme. So http://docs.google.com:123 http://www.google.com are the same
site but https://docs.google.com isn't. There's also a trick with
www.google.co.uk (and a few other multi-component top level domains) where tld +
1 == google.co.uk.

Maybe just leave it as HostZoomMap::Get(xxx)?

[wjmaclean, 2014/08/13 17:16:03]

I'm ok with this, though I suppose it does break the GetForX() pattern elsewhere
in HostZoomMap retrievers. An alternative could be GetForSiteInstance()?

I've changed it to plain-old "Get()".

Done.

   host_zoom_map->SetTemporaryZoomLevel(GetProcess()->GetID(),
                                        GetRoutingID(),
                                        host_zoom_map->GetDefaultZoomLevel());
@@ -1471,7 +1471,7 @@ void RenderViewHostImpl::NotifyMoveOrResizeStarted() {
 void RenderViewHostImpl::OnDidZoomURL(double zoom_level,
                                       const GURL& url) {
   HostZoomMapImpl* host_zoom_map = static_cast<HostZoomMapImpl*>(
-      HostZoomMap::GetForBrowserContext(GetProcess()->GetBrowserContext()));
+      HostZoomMap::GetForSite(GetSiteInstance()));
 
   host_zoom_map->SetZoomLevelForView(GetProcess()->GetID(),
                                      GetRoutingID(),