Migrate HostZoomMap to live in StoragePartition.

content/public/browser/zoom_level_prefs_store.h

Index: content/public/browser/zoom_level_prefs_store.h
diff --git a/content/public/browser/zoom_level_prefs_store.h b/content/public/browser/zoom_level_prefs_store.h
new file mode 100644
index 0000000000000000000000000000000000000000..9bf8fb58c2dcb0bcc9cf3e4eecd9a3a10a290446
--- /dev/null
+++ b/content/public/browser/zoom_level_prefs_store.h
@@ -0,0 +1,30 @@
+// Copyright (c) 2014 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_ZOOM_LEVEL_PREFS_STORE_H_
+#define CONTENT_PUBLIC_BROWSER_ZOOM_LEVEL_PREFS_STORE_H_
+
+class PrefService;
+
+namespace base {
+class FilePath;
+}  // namespace base
+
+namespace content {
+
+class HostZoomMap;
+
+class ZoomLevelPrefsStore {

[Fady Samuel, 2014/08/15 15:03:01]

Description of what this class is.

[wjmaclean, 2014/08/15 22:13:50]

Done.

+ public:
+  virtual void InitHostZoomMap(const base::FilePath& file_path,

[Fady Samuel, 2014/08/15 15:03:01]

GetZoomLevels seems like a more appropriate name. Please also add a description
of what it does.

[wjmaclean, 2014/08/15 22:13:50]

Since we are (1) initializing the zoom-level-prefs, and (2) copying to the
HostZoomMap, how about InitPrefsAndCopyToHostZoomMap()?

I think GetZoomLevels() is *less* descriptive than InitHostZoomMap().

Done.

+                               HostZoomMap* host_zoom_map) = 0;
+  virtual ~ZoomLevelPrefsStore() {}
+
+  // Expose for tests.
+  virtual PrefService* GetPrefs() = 0;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_BROWSER_ZOOM_LEVEL_PREFS_STORE_H_