Refactored SettingsFrontend and forwarding of calls to the backends.

Added a SettingsNamespaceFrontend interface for namespace frontends, that allows different namespaces to forward StorageCallbacks in their own way.
The LOCAL and SYNC namespaces use an implementation that does the same as before, but the new MANAGED namespace can now be implemented using a ValueStore that loads settings from policies, in the UI thread.

BUG=108992
TEST=Everything works as before, all tests are green


Review URL: https://chromiumcodereview.appspot.com/10784035

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147464 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 2 deletions

diff --git a/chrome/browser/extensions/script_badge_controller_unittest.cc b/chrome/browser/extensions/script_badge_controller_unittest.cc
index cf5a8b0..8b0623b 100644
--- a/chrome/browser/extensions/script_badge_controller_unittest.cc
+++ b/chrome/browser/extensions/script_badge_controller_unittest.cc
@@ -35,8 +35,8 @@ namespace {
 class ScriptBadgeControllerTest : public TabContentsTestHarness {
  public:
   ScriptBadgeControllerTest()
-      : ui_thread_(BrowserThread::UI, MessageLoop::current()) {
-  }
+      : ui_thread_(BrowserThread::UI, MessageLoop::current()),
+        file_thread_(BrowserThread::FILE, MessageLoop::current()) {}
 
   virtual void SetUp() OVERRIDE {
     // Note that this sets a PageActionController into the
@@ -82,6 +82,7 @@ class ScriptBadgeControllerTest : public TabContentsTestHarness {
 
  private:
   content::TestBrowserThread ui_thread_;
+  content::TestBrowserThread file_thread_;
 };
 
 struct CountingNotificationObserver : public content::NotificationObserver {