Font Family Preferences for Distilled Pages

components/dom_distiller/core/distilled_page_prefs.cc

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
index 5c23fb43f221134fcba473b215f7291a1f605e21..563b7356ec9b62d92944e20589c0995460d49524 100644
--- a/components/dom_distiller/core/distilled_page_prefs.cc
+++ b/components/dom_distiller/core/distilled_page_prefs.cc
@@ -14,6 +14,8 @@
 namespace {
 
 // Path to the integer corresponding to user's preference theme.
+const char kFontPref[] = "dom_distiller.font";
+// Path to the integer corresponding to user's preference font family.
 const char kThemePref[] = "dom_distiller.theme";
 }
 
@@ -33,6 +35,31 @@ void DistilledPagePrefs::RegisterProfilePrefs(
       kThemePref,
       DistilledPagePrefs::LIGHT,
       user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      kFontPref,
+      DistilledPagePrefs::SANS_SERIF,
+      user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
+}
+
+void DistilledPagePrefs::SetFontFamily(
+    DistilledPagePrefs::FontFamily new_font_family) {
+  pref_service_->SetInteger(kFontPref, new_font_family);
+  base::MessageLoop::current()->PostTask(
+      FROM_HERE,
+      base::Bind(&DistilledPagePrefs::NotifyOnChangeFontFamily,
+                 weak_ptr_factory_.GetWeakPtr(),
+                 new_font_family));
+}
+
+DistilledPagePrefs::FontFamily DistilledPagePrefs::GetFontFamily() {
+  int font_family = pref_service_->GetInteger(kFontPref);
+  if (font_family < 0 || font_family >= DistilledPagePrefs::FONT_FAMILY_COUNT) {
+    // Persisted data was incorrect, trying to clean it up by storing the
+    // default.
+    SetFontFamily(DistilledPagePrefs::SANS_SERIF);
+    return DistilledPagePrefs::SANS_SERIF;
+  }
+  return (FontFamily)font_family;

[robliao, 2014/08/11 22:00:45]

static_cast

[sunangel, 2014/08/12 02:09:56]

Done.

 }
 
 void DistilledPagePrefs::SetTheme(DistilledPagePrefs::Theme new_theme) {
@@ -63,6 +90,11 @@ void DistilledPagePrefs::RemoveObserver(Observer* obs) {
   observers_.RemoveObserver(obs);
 }
 
+void DistilledPagePrefs::NotifyOnChangeFontFamily(
+    DistilledPagePrefs::FontFamily new_font_family) {
+  FOR_EACH_OBSERVER(Observer, observers_, OnChangeFontFamily(new_font_family));
+}
+
 void DistilledPagePrefs::NotifyOnChangeTheme(
     DistilledPagePrefs::Theme new_theme) {
   FOR_EACH_OBSERVER(Observer, observers_, OnChangeTheme(new_theme));