Update chrome.settingsPrivate to support CrOS-only settings.

chrome/browser/extensions/api/settings_private/prefs_util.cc

 // Copyright 2015 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 "base/json/json_reader.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/api/settings_private/prefs_util.h"
+#include "chrome/browser/extensions/chrome_extension_function.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/pref_names.h"
+#include "components/url_fixer/url_fixer.h"
+
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/ownership/owner_settings_service_chromeos.h"
+#include "chrome/browser/chromeos/ownership/owner_settings_service_chromeos_factory.h"
+#include "chrome/browser/chromeos/settings/cros_settings.h"
+#endif
 
 namespace extensions {
 
 namespace settings_private = api::settings_private;
 
-namespace prefs_util {
+PrefsUtil::PrefsUtil(Profile* profile) : profile_(profile) {
+}
 
-const TypedPrefMap& GetWhitelistedKeys() {
-  static TypedPrefMap* s_whitelist = nullptr;
+PrefsUtil::~PrefsUtil() {
+}
+
+#if defined(OS_CHROMEOS)
+using CrosSettings = chromeos::CrosSettings;
+#endif
+
+const PrefsUtil::TypedPrefMap& PrefsUtil::GetWhitelistedKeys() {
+  static PrefsUtil::TypedPrefMap* s_whitelist = nullptr;
   if (s_whitelist)
     return *s_whitelist;
-  s_whitelist = new TypedPrefMap();
+  s_whitelist = new PrefsUtil::TypedPrefMap();
   (*s_whitelist)["browser.show_home_button"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["bookmark_bar.show_on_all_tabs"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["download.default_directory"] =
       settings_private::PrefType::PREF_TYPE_STRING;
   (*s_whitelist)["download.prompt_for_download"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["homepage"] = settings_private::PrefType::PREF_TYPE_URL;
 
 #if defined(OS_CHROMEOS)
+  (*s_whitelist)["cros.accounts.allowBWSI"] =
+      settings_private::PrefType::PREF_TYPE_BOOLEAN;
+  (*s_whitelist)["cros.accounts.supervisedUsersEnabled"] =
+      settings_private::PrefType::PREF_TYPE_BOOLEAN;
+  (*s_whitelist)["cros.accounts.showUserNamesOnSignIn"] =
+      settings_private::PrefType::PREF_TYPE_BOOLEAN;
+  (*s_whitelist)["cros.accounts.allowGuest"] =
+      settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.accessibility"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.a11y.autoclick"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.a11y.autoclick_delay_ms"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.a11y.enable_menu"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.a11y.high_contrast_enabled"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.a11y.large_cursor_enabled"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.a11y.screen_magnifier"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.a11y.sticky_keys_enabled"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.a11y.virtual_keyboard"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.clock.use_24hour_clock"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
   (*s_whitelist)["settings.touchpad.enable_tap_dragging"] =
       settings_private::PrefType::PREF_TYPE_BOOLEAN;
 #endif
 
   return *s_whitelist;
 }
 
-scoped_ptr<api::settings_private::PrefObject> GetPref(Profile* profile,
-                                                      const std::string& name) {
+api::settings_private::PrefType PrefsUtil::GetType(const std::string& name,
+                                                   base::Value::Type type) {
+  switch (type) {
+    case base::Value::Type::TYPE_BOOLEAN:
+      return api::settings_private::PrefType::PREF_TYPE_BOOLEAN;
+    case base::Value::Type::TYPE_INTEGER:
+    case base::Value::Type::TYPE_DOUBLE:
+      return api::settings_private::PrefType::PREF_TYPE_NUMBER;
+    case base::Value::Type::TYPE_STRING:
+      return IsPrefTypeURL(name)
+                 ? api::settings_private::PrefType::PREF_TYPE_URL
+                 : api::settings_private::PrefType::PREF_TYPE_STRING;
+    case base::Value::Type::TYPE_LIST:
+      return api::settings_private::PrefType::PREF_TYPE_LIST;
+    default:
+      return api::settings_private::PrefType::PREF_TYPE_NONE;
+  }
+}
+
+scoped_ptr<api::settings_private::PrefObject> PrefsUtil::GetCrosSettingsPref(
+    const std::string& name) {
   scoped_ptr<api::settings_private::PrefObject> pref_object(
       new api::settings_private::PrefObject());
 
-  PrefService* pref_service = FindServiceForPref(profile, name);
+#if defined(OS_CHROMEOS)
+  const base::Value* value = CrosSettings::Get()->GetPref(name);
+  pref_object->key = name;
+  pref_object->type = GetType(name, value->GetType());

[Oren Blasberg, 2015/06/06 00:42:55]

Somehow, this line segfaults if name is 'cros.accounts.users' (and if we also
add cros.accounts.users to the whitelist, of course).  According to gdb, the
value of type_ is 0x8 (a pointer to 0x8.... umm) Very weird :\

[stevenjb, 2015/06/06 00:48:17]

Is value maybe NULL?

[Oren Blasberg, 2015/06/06 01:02:11]

Whoops, you're right, it's null.
However, I don't get why. I'm basically doing the same thing as what
accounts_options_handler.cc does [1], except that file is going through
GetList() which calls GetPref, and I'm just calling GetPref directly. GetList
calls "GetAsList" on the value but if the value's null that wouldn't make a
difference.

Maybe some sort of timing/lifetime issue.. basically since we load all prefs on
startup, adding 'cros.users.accounts' to the prefs_util.cc whitelist causes
md-settings to crash on load.

Hopefully the clarity of Monday morning will bring fresh insight.


[1]
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...

+  pref_object->value.reset(value->DeepCopy());
+#endif
+
+  return pref_object.Pass();
+}
+
+scoped_ptr<api::settings_private::PrefObject> PrefsUtil::GetPref(
+    const std::string& name) {
+  scoped_ptr<api::settings_private::PrefObject> pref_object(
+      new api::settings_private::PrefObject());
+
+  if (IsCrosSetting(name))
+    return GetCrosSettingsPref(name);
+
+  PrefService* pref_service = FindServiceForPref(name);
   const PrefService::Preference* pref = pref_service->FindPreference(name);
   if (!pref)
     return pref_object.Pass();
 
   pref_object->key = pref->name();
-  switch (pref->GetType()) {
-    case base::Value::Type::TYPE_BOOLEAN:
-      pref_object->type = api::settings_private::PrefType::PREF_TYPE_BOOLEAN;
-      break;
-    case base::Value::Type::TYPE_INTEGER:
-    case base::Value::Type::TYPE_DOUBLE:
-      pref_object->type = api::settings_private::PrefType::PREF_TYPE_NUMBER;
-      break;
-    case base::Value::Type::TYPE_STRING:
-      pref_object->type =
-          IsPrefTypeURL(name)
-              ? api::settings_private::PrefType::PREF_TYPE_URL
-              : api::settings_private::PrefType::PREF_TYPE_STRING;
-      break;
-    case base::Value::Type::TYPE_LIST:
-      pref_object->type = api::settings_private::PrefType::PREF_TYPE_LIST;
-      break;
-    default:
-      break;
-  }
-
+  pref_object->type = GetType(name, pref->GetType());
   pref_object->value.reset(pref->GetValue()->DeepCopy());
 
   if (pref->IsManaged()) {
     if (pref->IsManagedByCustodian()) {
       pref_object->policy_source =
           api::settings_private::PolicySource::POLICY_SOURCE_DEVICE;
     } else {
       pref_object->policy_source =
           api::settings_private::PolicySource::POLICY_SOURCE_USER;
     }
     pref_object->policy_enforcement =
         pref->IsRecommended() ? api::settings_private::PolicyEnforcement::
                                     POLICY_ENFORCEMENT_RECOMMENDED
                               : api::settings_private::PolicyEnforcement::
                                     POLICY_ENFORCEMENT_ENFORCED;
-  } else if (!IsPrefUserModifiable(profile, name)) {
+  } else if (!IsPrefUserModifiable(name)) {
     pref_object->policy_source =
         api::settings_private::PolicySource::POLICY_SOURCE_USER;
     pref_object->policy_enforcement =
         api::settings_private::PolicyEnforcement::POLICY_ENFORCEMENT_ENFORCED;
   }
 
   return pref_object.Pass();
 }
 
-bool IsPrefTypeURL(const std::string& pref_name) {
+bool PrefsUtil::SetPref(const std::string& pref_name,
+                        const base::Value* value) {
+  if (IsCrosSetting(pref_name))
+    return SetCrosSettingsPref(pref_name, value);
+
+  PrefService* pref_service = FindServiceForPref(pref_name);
+
+  if (!IsPrefUserModifiable(pref_name))
+    return false;
+
+  const PrefService::Preference* pref =
+      pref_service->FindPreference(pref_name.c_str());
+  if (!pref)
+    return false;
+
+  DCHECK_EQ(pref->GetType(), value->GetType());
+
+  scoped_ptr<base::Value> temp_value;
+
+  switch (pref->GetType()) {
+    case base::Value::TYPE_INTEGER: {
+      // In JS all numbers are doubles.
+      double double_value;
+      if (!value->GetAsDouble(&double_value))
+        return false;
+
+      int int_value = static_cast<int>(double_value);
+      temp_value.reset(new base::FundamentalValue(int_value));
+      value = temp_value.get();
+      break;
+    }
+    case base::Value::TYPE_STRING: {
+      std::string original;
+      if (!value->GetAsString(&original))
+        return false;
+
+      if (IsPrefTypeURL(pref_name)) {
+        GURL fixed = url_fixer::FixupURL(original, std::string());
+        temp_value.reset(new base::StringValue(fixed.spec()));
+        value = temp_value.get();
+      }
+      break;
+    }
+    case base::Value::TYPE_LIST: {
+      // In case we have a List pref we got a JSON string.
+      std::string json_string;
+      if (!value->GetAsString(&json_string))
+        return false;
+
+      temp_value.reset(base::JSONReader::DeprecatedRead(json_string));
+      value = temp_value.get();
+      if (!value->IsType(base::Value::TYPE_LIST))
+        return false;
+
+      break;
+    }
+    case base::Value::TYPE_BOOLEAN:
+    case base::Value::TYPE_DOUBLE:
+      break;
+    default:
+      return false;
+  }
+
+  // TODO(orenb): Process setting metrics here and in the CrOS setting method
+  // too (like "ProcessUserMetric" in CoreOptionsHandler).
+  pref_service->Set(pref_name.c_str(), *value);
+  return true;
+}
+
+bool PrefsUtil::SetCrosSettingsPref(const std::string& pref_name,
+                                    const base::Value* value) {
+#if defined(OS_CHROMEOS)
+  chromeos::OwnerSettingsServiceChromeOS* service =
+      chromeos::OwnerSettingsServiceChromeOSFactory::GetForBrowserContext(
+          profile_);
+
+  // Returns false if not the owner, for settings requiring owner.
+  if (service && service->HandlesSetting(pref_name))
+    return service->Set(pref_name, *value);
+
+  chromeos::CrosSettings::Get()->Set(pref_name, *value);
+  return true;
+#else
+  return false;
+#endif
+}
+
+bool PrefsUtil::AppendToListCrosSetting(const std::string& setting,
+                                        const base::Value& value) {
+#if defined(OS_CHROMEOS)
+  chromeos::OwnerSettingsServiceChromeOS* service =
+      chromeos::OwnerSettingsServiceChromeOSFactory::GetForBrowserContext(
+          profile_);
+  DCHECK(service);
+  return service->AppendToList(setting, value);
+#else
+  return false;
+#endif
+}
+
+bool PrefsUtil::RemoveFromListCrosSetting(const std::string& setting,
+                                          const base::Value& value) {
+#if defined(OS_CHROMEOS)
+  chromeos::OwnerSettingsServiceChromeOS* service =
+      chromeos::OwnerSettingsServiceChromeOSFactory::GetForBrowserContext(
+          profile_);
+  DCHECK(service);
+  return service->RemoveFromList(setting, value);
+#else
+  return false;
+#endif
+}
+
+bool PrefsUtil::IsPrefTypeURL(const std::string& pref_name) {
   settings_private::PrefType pref_type =
       settings_private::PrefType::PREF_TYPE_NONE;
 
   const TypedPrefMap keys = GetWhitelistedKeys();
   const auto& iter = keys.find(pref_name);
   if (iter != keys.end())
     pref_type = iter->second;
 
   return pref_type == settings_private::PrefType::PREF_TYPE_URL;
 }
 
-bool IsPrefUserModifiable(Profile* profile, const std::string& pref_name) {
+bool PrefsUtil::IsPrefUserModifiable(const std::string& pref_name) {
   if (pref_name != prefs::kBrowserGuestModeEnabled &&
       pref_name != prefs::kBrowserAddPersonEnabled) {
     return true;
   }
 
-  PrefService* pref_service = profile->GetPrefs();
+  PrefService* pref_service = profile_->GetPrefs();
   const PrefService::Preference* pref =
       pref_service->FindPreference(pref_name.c_str());
-  if (!pref || !pref->IsUserModifiable() || profile->IsSupervised())
+  if (!pref || !pref->IsUserModifiable() || profile_->IsSupervised())
     return false;
 
   return true;
 }
 
-PrefService* FindServiceForPref(Profile* profile,
-                                const std::string& pref_name) {
-  PrefService* user_prefs = profile->GetPrefs();
+PrefService* PrefsUtil::FindServiceForPref(const std::string& pref_name) {
+  PrefService* user_prefs = profile_->GetPrefs();
 
   // Proxy is a peculiar case: on ChromeOS, settings exist in both user
   // prefs and local state, but chrome://settings should affect only user prefs.
   // Elsewhere the proxy settings are stored in local state.
   // See http://crbug.com/157147
 
   if (pref_name == prefs::kProxy) {
 #if defined(OS_CHROMEOS)
     return user_prefs;
 #else
     return g_browser_process->local_state();
 #endif
   }
 
   // Find which PrefService contains the given pref. Pref names should not
   // be duplicated across services, however if they are, prefer the user's
   // prefs.
   if (user_prefs->FindPreference(pref_name))
     return user_prefs;
 
   if (g_browser_process->local_state()->FindPreference(pref_name))
     return g_browser_process->local_state();
 
   return user_prefs;
 }
 
-}  // namespace prefs_util
+bool PrefsUtil::IsCrosSetting(const std::string& pref_name) {
+#if defined(OS_CHROMEOS)
+  return CrosSettings::Get()->IsCrosSettings(pref_name);
+#else
+  return false;
+#endif
+}
 
 }  // namespace extensions