Introduce incognito preference settings.

chrome/test/testing_pref_service.h

 // Copyright (c) 2010 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 CHROME_TEST_TESTING_PREF_SERVICE_H_
 #define CHROME_TEST_TESTING_PREF_SERVICE_H_
 #pragma once
 
+#include "base/ref_counted.h"
 #include "chrome/browser/prefs/pref_service.h"
 
+class ExtensionPrefStore;
 class TestingPrefStore;
 
+// Helper base class to allow simple construction of TestPrefService objects.
+class TestingPrefServiceBase : public PrefService {
+ public:
+  virtual ~TestingPrefServiceBase();
+
+ protected:
+  TestingPrefServiceBase(
+      TestingPrefStore* managed_platform_prefs,
+      TestingPrefStore* device_management_prefs,
+      ExtensionPrefStore* extension_prefs,
+      TestingPrefStore* user_prefs);
+
+  // Pointers to the pref stores our value store uses.
+  scoped_refptr<TestingPrefStore> managed_platform_prefs_;
+  scoped_refptr<TestingPrefStore> device_management_prefs_;
+  scoped_refptr<ExtensionPrefStore> extension_prefs_;
+  scoped_refptr<TestingPrefStore> user_prefs_;

[Mattias Nissler (ping if slow), 2010/12/22 12:09:03]

Didn't we agree to just move all the interface into this class and have the
actual TestingPrefService only have a constructor for initialization? Keeping
the pointers here, but the interface in the subclass seems odd to me.

[battre, 2010/12/22 18:34:53]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(TestingPrefServiceBase);
+};
+
 // 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.
-class TestingPrefService : public PrefService {
+class TestingPrefService : public TestingPrefServiceBase {
  public:
-  // Create an empty instance.
   TestingPrefService();
-  virtual ~TestingPrefService() {}
+  virtual ~TestingPrefService();
 
   // Read the value of a preference from the managed layer. Returns NULL if the
   // preference is not defined at the managed layer.
   const Value* GetManagedPref(const char* 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, Value* value);
 
   // Clear the preference on the managed layer and fire observers if the
   // preference has been defined previously.
   void RemoveManagedPref(const char* path);
 
   // Similar to the above, but for user preferences.
   const Value* GetUserPref(const char* path) const;
   void SetUserPref(const char* path, Value* value);
   void RemoveUserPref(const char* path);
 
+  ExtensionPrefStore* GetExtensionPrefs();

[Mattias Nissler (ping if slow), 2010/12/22 12:09:03]

No! This class is meant for the case when you need a PrefService in which you
can control all layers without the normally used dependencies. So we should
really use a plain PrefStore pointer for the extension store here and construct
a TestingPrefStore internally.

[battre, 2010/12/22 18:34:53]

This method was introduced because the ExtensionPrefs constructor called in
TestingProfile::CreateExtensionService needs an ExtensionPrefStore. The
TestingPrefService owns and creates the pref stores. If we want to work around
this, we probably need to touch all occurrences where TestingPrefService is used
and inject an ExtensionPrefStore. This will be messy.

+
  private:
   // Reads the value of the preference indicated by |path| from |pref_store|.
   // Returns NULL if the preference was not found.
   const Value* GetPref(TestingPrefStore* pref_store, const char* path) const;
 
   // Sets the value for |path| in |pref_store|.
   void SetPref(TestingPrefStore* pref_store, const char* path, Value* value);
 
   // Removes the preference identified by |path| from |pref_store|.
   void RemovePref(TestingPrefStore* pref_store, const char* path);
 
-  // Pointers to the pref stores our value store uses.
-  TestingPrefStore* managed_platform_prefs_;  // weak
-  TestingPrefStore* device_management_prefs_;  // weak
-  TestingPrefStore* user_prefs_;  // weak
-
   DISALLOW_COPY_AND_ASSIGN(TestingPrefService);
 };
 
 #endif  // CHROME_TEST_TESTING_PREF_SERVICE_H_