Change preference APIs to take std::string instead of const char*.

base/prefs/testing_pref_service.h

 // Copyright (c) 2012 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 BASE_PREFS_TESTING_PREF_SERVICE_H_
 #define BASE_PREFS_TESTING_PREF_SERVICE_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/prefs/pref_registry.h"
 #include "base/prefs/pref_service.h"
 #include "base/prefs/testing_pref_store.h"
 
 class PrefNotifierImpl;
 class PrefRegistrySimple;
 class TestingPrefStore;
 
 // A PrefService subclass for testing. It operates totally in memory and
 // provides additional API for manipulating preferences at the different levels
 // (managed, extension, user) conveniently.
 //
 // Use this via its specializations, e.g. TestingPrefServiceSimple.
 template <class SuperPrefService, class ConstructionPrefRegistry>
 class TestingPrefServiceBase : public SuperPrefService {
  public:
   virtual ~TestingPrefServiceBase();
 
   // Read the value of a preference from the managed layer. Returns NULL if the
   // preference is not defined at the managed layer.
-  const base::Value* GetManagedPref(const char* path) const;
+  const base::Value* GetManagedPref(const std::string& path) const;
 
   // Set a preference on the managed layer and fire observers if the preference
   // changed. Assumes ownership of |value|.
-  void SetManagedPref(const char* path, base::Value* value);
+  void SetManagedPref(const std::string& path, base::Value* value);
 
   // Clear the preference on the managed layer and fire observers if the
   // preference has been defined previously.
-  void RemoveManagedPref(const char* path);
+  void RemoveManagedPref(const std::string& path);
 
   // Similar to the above, but for user preferences.
-  const base::Value* GetUserPref(const char* path) const;
-  void SetUserPref(const char* path, base::Value* value);
-  void RemoveUserPref(const char* path);
+  const base::Value* GetUserPref(const std::string& path) const;
+  void SetUserPref(const std::string& path, base::Value* value);
+  void RemoveUserPref(const std::string& path);
 
   // Similar to the above, but for recommended policy preferences.
-  const base::Value* GetRecommendedPref(const char* path) const;
-  void SetRecommendedPref(const char* path, base::Value* value);
-  void RemoveRecommendedPref(const char* path);
+  const base::Value* GetRecommendedPref(const std::string& path) const;
+  void SetRecommendedPref(const std::string& path, base::Value* value);
+  void RemoveRecommendedPref(const std::string& path);
 
   // Do-nothing implementation for TestingPrefService.
   static void HandleReadError(PersistentPrefStore::PrefReadError error) {}
 
  protected:
   TestingPrefServiceBase(
       TestingPrefStore* managed_prefs,
       TestingPrefStore* user_prefs,
       TestingPrefStore* recommended_prefs,
       ConstructionPrefRegistry* pref_registry,
       PrefNotifierImpl* pref_notifier);
 
  private:
   // Reads the value of the preference indicated by |path| from |pref_store|.
   // Returns NULL if the preference was not found.
   const base::Value* GetPref(TestingPrefStore* pref_store,
-                             const char* path) const;
+                             const std::string& path) const;
 
   // Sets the value for |path| in |pref_store|.
-  void SetPref(TestingPrefStore* pref_store, const char* path,
+  void SetPref(TestingPrefStore* pref_store,
+               const std::string& path,
                base::Value* value);
 
   // Removes the preference identified by |path| from |pref_store|.
-  void RemovePref(TestingPrefStore* pref_store, const char* path);
+  void RemovePref(TestingPrefStore* pref_store, const std::string& path);
 
   // Pointers to the pref stores our value store uses.
   scoped_refptr<TestingPrefStore> managed_prefs_;
   scoped_refptr<TestingPrefStore> user_prefs_;
   scoped_refptr<TestingPrefStore> recommended_prefs_;
 
   DISALLOW_COPY_AND_ASSIGN(TestingPrefServiceBase);
 };
 
 // Test version of PrefService.
@@ skipping 20 matching lines @@
     TestingPrefStore* user_prefs,
     TestingPrefStore* recommended_prefs,
     PrefRegistry* pref_registry,
     PrefNotifierImpl* pref_notifier);
 
 template<class SuperPrefService, class ConstructionPrefRegistry>
 TestingPrefServiceBase<
     SuperPrefService, ConstructionPrefRegistry>::~TestingPrefServiceBase() {
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
+template <class SuperPrefService, class ConstructionPrefRegistry>
 const base::Value* TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::GetManagedPref(
-        const char* path) const {
+    SuperPrefService,
+    ConstructionPrefRegistry>::GetManagedPref(const std::string& path) const {
   return GetPref(managed_prefs_.get(), path);
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-void TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::SetManagedPref(
-        const char* path, base::Value* value) {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+void TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::
+    SetManagedPref(const std::string& path, base::Value* value) {
   SetPref(managed_prefs_.get(), path, value);
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-void TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::RemoveManagedPref(
-        const char* path) {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+void TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::
+    RemoveManagedPref(const std::string& path) {
   RemovePref(managed_prefs_.get(), path);
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-const base::Value* TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::GetUserPref(
-        const char* path) const {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+const base::Value*
+TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::GetUserPref(
+    const std::string& path) const {
   return GetPref(user_prefs_.get(), path);
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-void TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::SetUserPref(
-        const char* path, base::Value* value) {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+void TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::
+    SetUserPref(const std::string& path, base::Value* value) {
   SetPref(user_prefs_.get(), path, value);
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-void TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::RemoveUserPref(
-        const char* path) {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+void TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::
+    RemoveUserPref(const std::string& path) {
   RemovePref(user_prefs_.get(), path);
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-const base::Value* TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::GetRecommendedPref(
-        const char* path) const {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+const base::Value*
+TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::
+    GetRecommendedPref(const std::string& path) const {
   return GetPref(recommended_prefs_, path);
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-void TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::SetRecommendedPref(
-        const char* path, base::Value* value) {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+void TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::
+    SetRecommendedPref(const std::string& path, base::Value* value) {
   SetPref(recommended_prefs_.get(), path, value);
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-void TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::RemoveRecommendedPref(
-        const char* path) {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+void TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::
+    RemoveRecommendedPref(const std::string& path) {

[gab, 2014/12/01 21:13:01]

I assume this wrapping is git cl format's style choice?

[Georges Khalil, 2014/12/01 21:28:21]

Correct.

   RemovePref(recommended_prefs_.get(), path);
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-const base::Value* TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::GetPref(
-        TestingPrefStore* pref_store, const char* path) const {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+const base::Value*
+TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::GetPref(
+    TestingPrefStore* pref_store,
+    const std::string& path) const {
   const base::Value* res;
   return pref_store->GetValue(path, &res) ? res : NULL;
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-void TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::SetPref(
-        TestingPrefStore* pref_store, const char* path, base::Value* value) {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+void TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::
+    SetPref(TestingPrefStore* pref_store,
+            const std::string& path,
+            base::Value* value) {
   pref_store->SetValue(path, value);
 }
 
-template<class SuperPrefService, class ConstructionPrefRegistry>
-void TestingPrefServiceBase<
-    SuperPrefService, ConstructionPrefRegistry>::RemovePref(
-        TestingPrefStore* pref_store, const char* path) {
+template <class SuperPrefService, class ConstructionPrefRegistry>
+void TestingPrefServiceBase<SuperPrefService, ConstructionPrefRegistry>::
+    RemovePref(TestingPrefStore* pref_store, const std::string& path) {
   pref_store->RemoveValue(path);
 }
 
 #endif  // BASE_PREFS_TESTING_PREF_SERVICE_H_