Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(173)

Unified Diff: chrome/browser/ui/zoom/chrome_zoom_level_prefs.h

Issue 541103002: Introduce ChromeZoomLevelPref, make zoom level prefs independent of profile prefs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove obsolete access of ZoomController from WebUILoginView. Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/zoom/chrome_zoom_level_prefs.h
diff --git a/chrome/browser/ui/zoom/chrome_zoom_level_prefs.h b/chrome/browser/ui/zoom/chrome_zoom_level_prefs.h
new file mode 100644
index 0000000000000000000000000000000000000000..8506eb7df690d00f5dd1234ba3acd8c18a59fb82
--- /dev/null
+++ b/chrome/browser/ui/zoom/chrome_zoom_level_prefs.h
@@ -0,0 +1,76 @@
+// Copyright 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 CHROME_BROWSER_UI_ZOOM_CHROME_ZOOM_LEVEL_PREFS_H_
+#define CHROME_BROWSER_UI_ZOOM_CHROME_ZOOM_LEVEL_PREFS_H_
+
+#include "base/callback.h"
+#include "base/files/file_path.h"
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "base/observer_list.h"
+#include "base/prefs/json_pref_store.h"
+#include "base/prefs/pref_change_registrar.h"
+#include "base/prefs/pref_service.h"
+#include "base/prefs/pref_store.h"
+#include "content/public/browser/host_zoom_map.h"
+
+namespace base {
+class DictionaryValue;
+}
+
+namespace chrome {
+
+// A class to manage per-partition default and per-host zoom levels in Chrome's
+// preference system. It implements an interface between the content/ zoom
+// levels in HostZoomMap and Chrome's preference system. All changes
+// to the per-partition default zoom levels from chrome/ flow through this
+// class. Any changes to per-host levels are updated when HostZoomMap calls
+// OnZoomLevelChanged.
+class ChromeZoomLevelPrefs {
+ public:
+ typedef base::CallbackList<void(void)>::Subscription
+ DefaultZoomLevelSubscription;
+
+ // Initialize the pref_service and the profile_path via the constructor,
+ // as these concepts won't be available in the content base class
+ // (to be added later) which will define the InitPrefsAndCopyToHostZoomMap
+ // interface. |pref_service_| must outlive this class.
+ ChromeZoomLevelPrefs(PrefService* pref_service,
+ const base::FilePath& profile_path);
+ virtual ~ChromeZoomLevelPrefs();
+
+ static std::string GetHashForTesting(const base::FilePath& relative_path);
+
+ virtual void InitPrefsAndCopyToHostZoomMap(
+ const base::FilePath& partition_path,
+ content::HostZoomMap* host_zoom_map);
+
+ void SetDefaultZoomLevelPref(double level);
+ double GetDefaultZoomLevelPref() const;
+ scoped_ptr<DefaultZoomLevelSubscription> RegisterDefaultZoomLevelCallback(
+ const base::Closure& callback);
+
+ void ExtractPerHostZoomLevels(
+ const base::DictionaryValue* host_zoom_dictionary);
+
+ private:
+ // This is a callback function that receives notifications from HostZoomMap
+ // when per-host zoom levels change. It is used to update the per-host
+ // zoom levels (if any) managed by this class (for its associated partition).
+ void OnZoomLevelChanged(const content::HostZoomMap::ZoomLevelChange& change);
+
+ PrefService* pref_service_;
+ const base::FilePath profile_path_;
+ content::HostZoomMap* host_zoom_map_;
+ scoped_ptr<content::HostZoomMap::Subscription> zoom_subscription_;
+ std::string partition_key_;
+ base::CallbackList<void(void)> default_zoom_changed_callbacks_;
+
+ DISALLOW_COPY_AND_ASSIGN(ChromeZoomLevelPrefs);
+};
+
+} // namespace chrome
+
+#endif // CHROME_BROWSER_UI_ZOOM_CHROME_ZOOM_LEVEL_PREFS_H_
« no previous file with comments | « chrome/browser/ui/webui/options/browser_options_handler.cc ('k') | chrome/browser/ui/zoom/chrome_zoom_level_prefs.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698