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

Unified Diff: components/dom_distiller/core/distilled_page_prefs.cc

Issue 341563002: Theme Preferences for Distilled Pages (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Minor change - dependency order Created 6 years, 5 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: components/dom_distiller/core/distilled_page_prefs.cc
diff --git a/components/dom_distiller/core/distilled_page_prefs.cc b/components/dom_distiller/core/distilled_page_prefs.cc
new file mode 100644
index 0000000000000000000000000000000000000000..5c23fb43f221134fcba473b215f7291a1f605e21
--- /dev/null
+++ b/components/dom_distiller/core/distilled_page_prefs.cc
@@ -0,0 +1,71 @@
+// 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.
+
+#include "components/dom_distiller/core/distilled_page_prefs.h"
+
+#include "base/bind.h"
+#include "base/memory/weak_ptr.h"
+#include "base/message_loop/message_loop.h"
+#include "base/observer_list.h"
+#include "base/prefs/pref_service.h"
+#include "components/pref_registry/pref_registry_syncable.h"
+
+namespace {
+
+// Path to the integer corresponding to user's preference theme.
+const char kThemePref[] = "dom_distiller.theme";
+}
+
+namespace dom_distiller {
+
+DistilledPagePrefs::DistilledPagePrefs(PrefService* pref_service)
+ : pref_service_(pref_service), weak_ptr_factory_(this) {
+}
+
+DistilledPagePrefs::~DistilledPagePrefs() {
+}
+
+// static
+void DistilledPagePrefs::RegisterProfilePrefs(
+ user_prefs::PrefRegistrySyncable* registry) {
+ registry->RegisterIntegerPref(
+ kThemePref,
+ DistilledPagePrefs::LIGHT,
+ user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
+}
+
+void DistilledPagePrefs::SetTheme(DistilledPagePrefs::Theme new_theme) {
+ pref_service_->SetInteger(kThemePref, new_theme);
+ base::MessageLoop::current()->PostTask(
+ FROM_HERE,
+ base::Bind(&DistilledPagePrefs::NotifyOnChangeTheme,
+ weak_ptr_factory_.GetWeakPtr(),
+ new_theme));
+}
+
+DistilledPagePrefs::Theme DistilledPagePrefs::GetTheme() {
+ int theme = pref_service_->GetInteger(kThemePref);
+ if (theme < 0 || theme >= DistilledPagePrefs::THEME_COUNT) {
+ // Persisted data was incorrect, trying to clean it up by storing the
+ // default.
+ SetTheme(DistilledPagePrefs::LIGHT);
+ return DistilledPagePrefs::LIGHT;
+ }
+ return (Theme) theme;
+}
+
+void DistilledPagePrefs::AddObserver(Observer* obs) {
+ observers_.AddObserver(obs);
+}
+
+void DistilledPagePrefs::RemoveObserver(Observer* obs) {
+ observers_.RemoveObserver(obs);
+}
+
+void DistilledPagePrefs::NotifyOnChangeTheme(
+ DistilledPagePrefs::Theme new_theme) {
+ FOR_EACH_OBSERVER(Observer, observers_, OnChangeTheme(new_theme));
+}
+
+} // namespace dom_distiller

Powered by Google App Engine
This is Rietveld 408576698