Remove ExtensionService::extension_prefs()

chrome/browser/themes/theme_syncable_service.cc

 // 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.
 
 #include "chrome/browser/themes/theme_syncable_service.h"
 
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/themes/theme_service.h"
 #include "chrome/common/extensions/manifest_url_handler.h"
 #include "chrome/common/extensions/sync_helper.h"
 #include "extensions/browser/extension_system.h"
 #include "extensions/common/extension.h"

[James Cook, 2014/02/07 00:53:02]

nit: Should include extension_prefs.h here

[Ken Rockot(use gerrit already), 2014/02/10 18:53:02]

Done.

 #include "sync/protocol/sync.pb.h"
 #include "sync/protocol/theme_specifics.pb.h"
 
 using std::string;
 
 namespace {
 
 bool IsTheme(const extensions::Extension* extension) {
   return extension->is_theme();
 }
@@ skipping 186 matching lines @@
     ExtensionService* extensions_service =
         extensions::ExtensionSystem::Get(profile_)->extension_service();
     CHECK(extensions_service);
     const extensions::Extension* extension =
         extensions_service->GetExtensionById(id, true);
     if (extension) {
       if (!extension->is_theme()) {
         DVLOG(1) << "Extension " << id << " is not a theme; aborting";
         return;
       }
-      int disabled_reasons =
-          extensions_service->extension_prefs()->GetDisableReasons(id);
+      int disabled_reasons = extensions::ExtensionPrefs::Get(profile_)->
+          GetDisableReasons(id);
       if (!extensions_service->IsExtensionEnabled(id) &&
           disabled_reasons != extensions::Extension::DISABLE_USER_ACTION) {
         DVLOG(1) << "Theme " << id << " is disabled with reason "
                  << disabled_reasons << "; aborting";
         return;
       }
       // An enabled theme extension with the given id was found, so
       // just set the current theme to it.
       theme_service_->SetTheme(extension);
     } else {
@@ skipping 101 matching lines @@
       syncer::SyncChange(FROM_HERE, change_type,
                          syncer::SyncData::CreateLocalData(
                              kCurrentThemeClientTag, kCurrentThemeNodeTitle,
                              entity_specifics)));
 
   DVLOG(1) << "Update theme specifics from current theme: "
       << changes.back().ToString();
 
   return sync_processor_->ProcessSyncChanges(FROM_HERE, changes);
 }