Refactor prefs.js for MD-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/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"
@@ skipping 163 matching lines @@
   } 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 PrefsUtil::SetPref(const std::string& pref_name,
-                        const base::Value* value) {
+PrefsUtil::SetPrefResult 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;
+    return PREF_NOT_MODIFIABLE;
 
   const PrefService::Preference* pref =
       pref_service->FindPreference(pref_name);
   if (!pref)
-    return false;
+    return PREF_NOT_FOUND;
 
   DCHECK_EQ(pref->GetType(), value->GetType());
 
   switch (pref->GetType()) {
     case base::Value::TYPE_BOOLEAN:
     case base::Value::TYPE_DOUBLE:
     case base::Value::TYPE_LIST:
       pref_service->Set(pref_name, *value);
       break;
     case base::Value::TYPE_INTEGER: {
       // In JS all numbers are doubles.
       double double_value;
       if (!value->GetAsDouble(&double_value))
-        return false;
+        return PREF_TYPE_MISMATCH;
 
       pref_service->SetInteger(pref_name, static_cast<int>(double_value));
       break;
     }
     case base::Value::TYPE_STRING: {
       std::string string_value;
       if (!value->GetAsString(&string_value))
-        return false;
+        return PREF_TYPE_MISMATCH;
 
       if (IsPrefTypeURL(pref_name)) {
         GURL fixed = url_formatter::FixupURL(string_value, std::string());
         string_value = fixed.spec();
       }
 
       pref_service->SetString(pref_name, string_value);
       break;
     }
     default:
-      return false;
+      return PREF_TYPE_UNSUPPORTED;
   }
 
   // TODO(orenb): Process setting metrics here and in the CrOS setting method
   // too (like "ProcessUserMetric" in CoreOptionsHandler).
-  return true;
+  return SUCCESS;
 }
 
-bool PrefsUtil::SetCrosSettingsPref(const std::string& pref_name,
-                                    const base::Value* value) {
+PrefsUtil::SetPrefResult 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);
+  // Check if setting requires owner.
+  if (service && service->HandlesSetting(pref_name)) {
+    if (service->Set(pref_name, *value))
+      return SUCCESS;
+    return PREF_NOT_MODIFIABLE;
+  }
 
   chromeos::CrosSettings::Get()->Set(pref_name, *value);
-  return true;
+  return SUCCESS;
 #else
-  return false;
+  return PREF_NOT_FOUND;
 #endif
 }
 
 bool PrefsUtil::AppendToListCrosSetting(const std::string& pref_name,
                                         const base::Value& value) {
 #if defined(OS_CHROMEOS)
   chromeos::OwnerSettingsServiceChromeOS* service =
       chromeos::OwnerSettingsServiceChromeOSFactory::GetForBrowserContext(
           profile_);
 
@@ skipping 85 matching lines @@
 
 bool PrefsUtil::IsCrosSetting(const std::string& pref_name) {
 #if defined(OS_CHROMEOS)
   return CrosSettings::Get()->IsCrosSettings(pref_name);
 #else
   return false;
 #endif
 }
 
 }  // namespace extensions