Refactor app_list_util.h into AppListService abstract base.

chrome/browser/extensions/extension_install_ui_default.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/extensions/extension_install_ui_default.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/api/infobars/confirm_infobar_delegate.h"
 #include "chrome/browser/api/infobars/infobar_service.h"
 #include "chrome/browser/extensions/app_launcher.h"
 #include "chrome/browser/extensions/extension_install_prompt.h"
 #include "chrome/browser/extensions/theme_installed_infobar_delegate.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/themes/theme_service.h"
 #include "chrome/browser/themes/theme_service_factory.h"
-#include "chrome/browser/ui/app_list/app_list_util.h"
+#include "chrome/browser/ui/app_list_service.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_navigator.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "chrome/browser/ui/simple_message_box.h"
 #include "chrome/browser/ui/singleton_tabs.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
@@ skipping 72 matching lines @@
                                   const extensions::CrxInstallerError& error) {
   infobar_service->AddInfoBar(scoped_ptr<InfoBarDelegate>(
       new ErrorInfobarDelegate(infobar_service, browser, error)));
 }
 
 void OnAppLauncherEnabledCompleted(const extensions::Extension* extension,
                                    Browser* browser,
                                    SkBitmap* icon,
                                    bool use_bubble,
                                    bool use_launcher) {
+// TODO(tapted): Remove this ifdef.

[tapted, 2013/02/11 04:04:20]

actually, looking at the function name (OnAppLauncherEnabledCompleted), maybe it
doesn't matter that the notification isn't guarded.

[koz (OOO until 15th September), 2013/02/11 04:33:38]

Sounds good to me.

[tapted, 2013/02/18 07:05:39]

Done.

 #if defined(ENABLE_APP_LIST)
   if (use_launcher) {
-    chrome::ShowAppList(browser->profile());
+    AppListService::Get()->ShowAppList(browser->profile());
 
     content::NotificationService::current()->Notify(
         chrome::NOTIFICATION_APP_INSTALLED_TO_APPLIST,
         content::Source<Profile>(browser->profile()),
         content::Details<const std::string>(&extension->id()));
     return;
   }
 #endif
 
   if (use_bubble) {
@@ skipping 138 matching lines @@
   return new ExtensionInstallPrompt(web_contents);
 }
 
 // static
 ExtensionInstallPrompt* ExtensionInstallUI::CreateInstallPromptWithProfile(
     Profile* profile) {
   Browser* browser = chrome::FindLastActiveWithProfile(profile,
       chrome::GetActiveDesktop());
   return CreateInstallPromptWithBrowser(browser);
 }