Remove //chrome dependency from ExtensionPrefs

extensions/browser/extension_prefs_factory.cc

 // Copyright 2014 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/command_line.h"
 #include "base/prefs/pref_service.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 #include "content/public/browser/browser_context.h"
 #include "extensions/browser/extension_pref_value_map.h"
 #include "extensions/browser/extension_pref_value_map_factory.h"
@@ skipping 26 matching lines @@
           "ExtensionPrefs",
           BrowserContextDependencyManager::GetInstance()) {
 }
 
 ExtensionPrefsFactory::~ExtensionPrefsFactory() {
 }
 
 KeyedService* ExtensionPrefsFactory::BuildServiceInstanceFor(
     content::BrowserContext* context) const {
   ExtensionsBrowserClient* client = ExtensionsBrowserClient::Get();
+  std::vector<ExtensionPrefsObserver*> prefs_observers;
+  client->GetAdditionalExtensionPrefsObservers(context, &prefs_observers);

[Yoyo Zhou, 2014/04/02 01:40:33]

Why not call this GetEarlyExtensionPrefsObservers (to match the argument name to
ExtensionPrefs::Create)?

[Ken Rockot(use gerrit already), 2014/04/02 03:53:01]

SGTM - Done.

   return ExtensionPrefs::Create(
       client->GetPrefServiceForContext(context),
       context->GetPath().AppendASCII(extensions::kInstallDirectoryName),
       ExtensionPrefValueMapFactory::GetForBrowserContext(context),
       client->CreateAppSorting().Pass(),
-      client->AreExtensionsDisabled(
-          *CommandLine::ForCurrentProcess(), context));
+      client->AreExtensionsDisabled(*CommandLine::ForCurrentProcess(), context),
+      prefs_observers);
 }
 
 content::BrowserContext* ExtensionPrefsFactory::GetBrowserContextToUse(
     content::BrowserContext* context) const {
   return ExtensionsBrowserClient::Get()->GetOriginalContext(context);
 }
 
 }  // namespace extensions