Migrate HostZoomMap to live in StoragePartition.

chrome/browser/profiles/off_the_record_profile_impl.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 CHROME_BROWSER_PROFILES_OFF_THE_RECORD_PROFILE_IMPL_H_
 #define CHROME_BROWSER_PROFILES_OFF_THE_RECORD_PROFILE_IMPL_H_
 
 #include <string>
 
 #include "base/gtest_prod_util.h"
@@ skipping 85 matching lines @@
   net::URLRequestContextGetter* GetMediaRequestContextForStoragePartition(
       const base::FilePath& partition_path,
       bool in_memory) override;
   content::ResourceContext* GetResourceContext() override;
   content::BrowserPluginGuestManager* GetGuestManager() override;
   storage::SpecialStoragePolicy* GetSpecialStoragePolicy() override;
   content::PushMessagingService* GetPushMessagingService() override;
   content::SSLHostStateDelegate* GetSSLHostStateDelegate() override;
 
  private:
-  FRIEND_TEST_ALL_PREFIXES(OffTheRecordProfileImplTest, GetHostZoomMap);
   void InitIoData();
-  void InitHostZoomMap();
+  // Allows a profile to track changes in zoom levels in its parent profile.
+  void TrackZoomLevelsFromParent();
 
 #if defined(OS_ANDROID) || defined(OS_IOS)
   void UseSystemProxy();
 #endif  // defined(OS_ANDROID) || defined(OS_IOS)
 
-  void OnZoomLevelChanged(const content::HostZoomMap::ZoomLevelChange& change);
   PrefProxyConfigTracker* CreateProxyConfigTracker();
+  // Callback function for tracking parent's zoom level changes.
+  void TrackParentZoomLevelChanged(

[Charlie Reis, 2014/11/04 23:43:16]

The original OnZoomLevelChanged name sounds better to me, or maybe
OnParentZoomLevelChanged.  "Track" is a present-tense verb instructing this
class to start tracking change, not an action to take when the change has
occurred.

[wjmaclean, 2014/11/05 21:55:42]

Done.

I chose OnParentZoomLevelChanged ... I wanted something to clearly indicate that
we're monitor parent profile zoom, and not our own.

+      const content::HostZoomMap::ZoomLevelChange& change);
 
   // The real underlying profile.
   Profile* profile_;
 
   // Weak pointer owned by |profile_|.
   PrefServiceSyncable* prefs_;
 
+  scoped_ptr<content::HostZoomMap::Subscription> track_zoom_subscription_;
   scoped_ptr<OffTheRecordProfileIOData::Handle> io_data_;
 
   // We use a non-persistent content settings map for OTR.
   scoped_refptr<HostContentSettingsMap> host_content_settings_map_;
 
   // Time we were started.
   Time start_time_;
 
   base::FilePath last_selected_directory_;
 
   scoped_ptr<PrefProxyConfigTracker> pref_proxy_config_tracker_;
 
-  scoped_ptr<content::HostZoomMap::Subscription> zoom_subscription_;
-
   DISALLOW_COPY_AND_ASSIGN(OffTheRecordProfileImpl);
 };
 
 #endif  // CHROME_BROWSER_PROFILES_OFF_THE_RECORD_PROFILE_IMPL_H_