Remove more dead app list code.

chrome/browser/ui/app_list/app_list_view_delegate.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/ui/app_list/app_list_view_delegate.h"
 
 #include <stddef.h>
 
 #include <vector>
 
-#include "base/callback.h"
 #include "base/command_line.h"
-#include "base/files/file_path.h"
 #include "base/metrics/user_metrics.h"
 #include "base/profiler/scoped_tracker.h"
-#include "base/stl_util.h"
 #include "build/build_config.h"
-#include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
-#include "chrome/browser/lifetime/scoped_keep_alive.h"
-#include "chrome/browser/profiles/profile_attributes_entry.h"
-#include "chrome/browser/profiles/profile_attributes_storage.h"
-#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search/hotword_service.h"
 #include "chrome/browser/search/hotword_service_factory.h"
 #include "chrome/browser/search_engines/template_url_service_factory.h"
 #include "chrome/browser/ui/app_list/app_list_controller_delegate.h"
-#include "chrome/browser/ui/app_list/app_list_service.h"
 #include "chrome/browser/ui/app_list/app_list_syncable_service.h"
 #include "chrome/browser/ui/app_list/app_list_syncable_service_factory.h"
 #include "chrome/browser/ui/app_list/custom_launcher_page_contents.h"
 #include "chrome/browser/ui/app_list/launcher_page_event_dispatcher.h"
 #include "chrome/browser/ui/app_list/search/search_controller_factory.h"
 #include "chrome/browser/ui/app_list/search/search_resource_manager.h"
 #include "chrome/browser/ui/app_list/start_page_service.h"
 #include "chrome/browser/ui/apps/chrome_app_delegate.h"
-#include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/ash/app_list/app_sync_ui_state_watcher.h"
 #include "chrome/browser/ui/chrome_pages.h"
-#include "chrome/browser/ui/scoped_tabbed_browser_displayer.h"
-#include "chrome/browser/web_applications/web_app.h"
-#include "chrome/common/chrome_switches.h"
-#include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/pref_names.h"
-#include "chrome/common/url_constants.h"
 #include "chrome/grit/theme_resources.h"
 #include "components/prefs/pref_service.h"
-#include "components/signin/core/browser/signin_manager.h"
 #include "components/user_prefs/user_prefs.h"
-#include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
-#include "content/public/browser/page_navigator.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/render_widget_host.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "content/public/browser/speech_recognition_session_preamble.h"
 #include "content/public/browser/web_contents.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension_set.h"
 #include "extensions/common/manifest_constants.h"
 #include "extensions/common/manifest_handlers/launcher_page_info.h"
 #include "ui/app_list/app_list_switches.h"
 #include "ui/app_list/search_box_model.h"
 #include "ui/app_list/search_controller.h"
 #include "ui/app_list/speech_ui_model.h"
 #include "ui/base/resource/resource_bundle.h"
+#include "ui/keyboard/keyboard_util.h"
 #include "ui/views/controls/webview/webview.h"
 
-#if defined(USE_AURA)
-#include "ui/keyboard/keyboard_util.h"
-#endif
-
-#if defined(USE_ASH)
-#include "chrome/browser/ui/ash/app_list/app_sync_ui_state_watcher.h"
-#endif
-
-namespace chrome {
-const char kAppLauncherCategoryTag[] = "AppLauncher";
-}  // namespace chrome
-
 namespace {
 
 const int kAutoLaunchDefaultTimeoutMilliSec = 50;
 
-void PopulateUsers(const base::FilePath& active_profile_path,
-                   app_list::AppListViewDelegate::Users* users) {
-  users->clear();
-  std::vector<ProfileAttributesEntry*> entries = g_browser_process->
-      profile_manager()->GetProfileAttributesStorage().
-      GetAllProfilesAttributesSortedByName();
-  for (const auto* entry : entries) {
-    app_list::AppListViewDelegate::User user;
-    user.name = entry->GetName();
-    user.email = entry->GetUserName();
-    user.profile_path = entry->GetPath();
-    user.active = active_profile_path == user.profile_path;
-    users->push_back(user);
-  }
-}
-
 // Gets a list of URLs of the custom launcher pages to show in the launcher.
 // Returns a URL for each installed launcher page. If --custom-launcher-page is
 // specified and valid, also includes that URL.
 void GetCustomLauncherPageUrls(content::BrowserContext* browser_context,
                                std::vector<GURL>* urls) {
   // First, check the command line.
   base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
   if (command_line->HasSwitch(app_list::switches::kCustomLauncherPage)) {
     GURL custom_launcher_page_url(command_line->GetSwitchValueASCII(
         app_list::switches::kCustomLauncherPage));
@@ skipping 33 matching lines @@
   }
 }
 
 }  // namespace
 
 AppListViewDelegate::AppListViewDelegate(AppListControllerDelegate* controller)
     : controller_(controller),
       profile_(NULL),
       model_(NULL),
       is_voice_query_(false),
-      template_url_service_observer_(this),
-      scoped_observer_(this) {
+      template_url_service_observer_(this) {
   CHECK(controller_);
-  // The SigninManagerFactor and the SigninManagers are observed to keep the
-  // profile switcher menu up to date, with the correct list of profiles and the
-  // correct email address (or none for signed out users) for each.
-  SigninManagerFactory::GetInstance()->AddObserver(this);
-
-  // Start observing all already-created SigninManagers.
-  ProfileManager* profile_manager = g_browser_process->profile_manager();
-  std::vector<Profile*> profiles = profile_manager->GetLoadedProfiles();
-
-  for (std::vector<Profile*>::iterator i = profiles.begin();
-       i != profiles.end();
-       ++i) {
-    SigninManagerBase* manager =
-        SigninManagerFactory::GetForProfileIfExists(*i);
-    if (manager) {
-      DCHECK(!scoped_observer_.IsObserving(manager));
-      scoped_observer_.Add(manager);
-    }
-  }
-
-  profile_manager->GetProfileAttributesStorage().AddObserver(this);
   speech_ui_.reset(new app_list::SpeechUIModel);
 
 #if defined(GOOGLE_CHROME_BUILD)
   gfx::ImageSkia* image;
   {
     // TODO(tapted): Remove ScopedTracker below once crbug.com/431326 is fixed.
     tracked_objects::ScopedTracker tracking_profile(
         FROM_HERE_WITH_EXPLICIT_FUNCTION("431326 GetImageSkiaNamed()"));
     image = ui::ResourceBundle::GetSharedInstance().GetImageSkiaNamed(
         IDR_APP_LIST_GOOGLE_LOGO_VOICE_SEARCH);
   }
 
   speech_ui_->set_logo(*image);
 #endif
 
   registrar_.Add(this,
                  chrome::NOTIFICATION_APP_TERMINATING,
                  content::NotificationService::AllSources());
 }
 
 AppListViewDelegate::~AppListViewDelegate() {
-  // Note that the destructor is not always called. E.g. on Mac, this is owned
-  // by a leaky singleton. Essential shutdown work must be done by observing
-  // chrome::NOTIFICATION_APP_TERMINATING.
+  // Note that the destructor is not always called. Essential shutdown work must

[Matt Giuca, 2017/03/31 05:29:20]

Is this no longer true? (If the only example provided was "on Mac", maybe it
*is* always called now?)

[tapted, 2017/03/31 05:55:25]

So.. I *think* is is no longer true. Pretty sure that when we were playing with
it, AppListViewDelegate would get re-created every time the app list was shown
on ChromeOS. but I haven't kept track (and for performance, ChromeOS really
shouldn't be doing that). And if it is actually no longer true then I also need
to get rid of chrome::NOTIFICATION_APP_TERMINATING since there's no longer any
point listening for it.

Best fix for this I think is to have ChromeOS cache this lazily so that it isn't
recreated a whole bunch... which is why I left this here.

[tapted, 2017/03/31 10:59:23]

Poked around - it looks like AppListServiceAsh is now a leaky singleton the way
Mac was.

changed comment: 
  // The destructor might not be called since the delegate is owned by a leaky
  // singleton. This matches the shutdown work done in Observe() in response to
  // chrome::NOTIFICATION_APP_TERMINATING, which may happen before this.

+  // be done by observing chrome::NOTIFICATION_APP_TERMINATING.
   SetProfile(NULL);
-  g_browser_process->profile_manager()->GetProfileAttributesStorage().
-      RemoveObserver(this);
-
-  SigninManagerFactory* factory = SigninManagerFactory::GetInstance();
-  if (factory)
-    factory->RemoveObserver(this);
 }
 
 void AppListViewDelegate::SetProfile(Profile* new_profile) {
   if (profile_ == new_profile)
     return;
 
   if (profile_) {
     // Note: |search_resource_manager_| has a reference to |speech_ui_| so must
     // be destroyed first.
     search_resource_manager_.reset();
     search_controller_.reset();
     launcher_page_event_dispatcher_.reset();
     custom_page_contents_.clear();
     app_list::StartPageService* start_page_service =
         app_list::StartPageService::Get(profile_);
     if (start_page_service)
       start_page_service->RemoveObserver(this);
-#if defined(USE_ASH)
     app_sync_ui_state_watcher_.reset();
-#endif
     model_ = NULL;
   }
 
   template_url_service_observer_.RemoveAll();
 
   profile_ = new_profile;
   if (!profile_) {
     speech_ui_->SetSpeechRecognitionState(app_list::SPEECH_RECOGNITION_OFF,
                                           false);
     return;
@@ skipping 11 matching lines @@
     tracked_objects::ScopedTracker tracking_profile(
         FROM_HERE_WITH_EXPLICIT_FUNCTION(
             "431326 AppListViewDelegate TemplateURL etc."));
     TemplateURLService* template_url_service =
         TemplateURLServiceFactory::GetForProfile(profile_);
     template_url_service_observer_.Add(template_url_service);
 
     model_ = app_list::AppListSyncableServiceFactory::GetForProfile(profile_)
                  ->GetModel();
 
-#if defined(USE_ASH)
     app_sync_ui_state_watcher_.reset(
         new AppSyncUIStateWatcher(profile_, model_));
-#endif
 
     SetUpSearchUI();
-    SetUpProfileSwitcher();
     SetUpCustomLauncherPages();
     OnTemplateURLServiceChanged();
   }
 
   // Clear search query.
   model_->search_box()->SetText(base::string16());
 }
 
 void AppListViewDelegate::SetUpSearchUI() {
   app_list::StartPageService* start_page_service =
       app_list::StartPageService::Get(profile_);
   if (start_page_service)
     start_page_service->AddObserver(this);
 
   speech_ui_->SetSpeechRecognitionState(start_page_service
                                             ? start_page_service->state()
                                             : app_list::SPEECH_RECOGNITION_OFF,
                                         false);
 
   search_resource_manager_.reset(new app_list::SearchResourceManager(
       profile_,
       model_->search_box(),
       speech_ui_.get()));
 
   search_controller_ = CreateSearchController(profile_, model_, controller_);
 }
 
-void AppListViewDelegate::SetUpProfileSwitcher() {
-  // If a profile change is observed when there is no app list, there is nothing
-  // to update until SetProfile() calls this function again.
-  if (!profile_)
-    return;
-
-#if defined(USE_ASH)
-  // Don't populate the app list users if we are on the ash desktop.
-  return;
-#endif  // USE_ASH
-
-  // Populate the app list users.
-  PopulateUsers(profile_->GetPath(), &users_);
-}
-
 void AppListViewDelegate::SetUpCustomLauncherPages() {
   std::vector<GURL> custom_launcher_page_urls;
   GetCustomLauncherPageUrls(profile_, &custom_launcher_page_urls);
   if (custom_launcher_page_urls.empty())
     return;
 
   for (auto it = custom_launcher_page_urls.begin();
        it != custom_launcher_page_urls.end(); ++it) {
     std::string extension_id = it->host();
     auto page_contents = base::MakeUnique<app_list::CustomLauncherPageContents>(
@@ skipping 26 matching lines @@
   }
 }
 
 void AppListViewDelegate::OnHotwordRecognized(
     const scoped_refptr<content::SpeechRecognitionSessionPreamble>& preamble) {
   DCHECK_EQ(app_list::SPEECH_RECOGNITION_HOTWORD_LISTENING,
             speech_ui_->state());
   StartSpeechRecognitionForHotword(preamble);
 }
 
-void AppListViewDelegate::SigninManagerCreated(SigninManagerBase* manager) {
-  scoped_observer_.Add(manager);
-}
-
-void AppListViewDelegate::SigninManagerShutdown(SigninManagerBase* manager) {
-  if (scoped_observer_.IsObserving(manager))
-    scoped_observer_.Remove(manager);
-}
-
-void AppListViewDelegate::GoogleSigninFailed(
-    const GoogleServiceAuthError& error) {
-  SetUpProfileSwitcher();
-}
-
-void AppListViewDelegate::GoogleSigninSucceeded(const std::string& account_id,
-                                                const std::string& username,
-                                                const std::string& password) {
-  SetUpProfileSwitcher();
-}
-
-void AppListViewDelegate::GoogleSignedOut(const std::string& account_id,
-                                          const std::string& username) {
-  SetUpProfileSwitcher();
-}
-
-void AppListViewDelegate::OnProfileAdded(const base::FilePath& profile_path) {
-  SetUpProfileSwitcher();
-}
-
-void AppListViewDelegate::OnProfileWasRemoved(
-    const base::FilePath& profile_path,
-    const base::string16& profile_name) {
-  SetUpProfileSwitcher();
-}
-
-void AppListViewDelegate::OnProfileNameChanged(
-    const base::FilePath& profile_path,
-    const base::string16& old_profile_name) {
-  SetUpProfileSwitcher();
-}
-
-bool AppListViewDelegate::ForceNativeDesktop() const {
-  return controller_->ForceNativeDesktop();
-}
-
-void AppListViewDelegate::SetProfileByPath(const base::FilePath& profile_path) {
-  DCHECK(model_);
-  // The profile must be loaded before this is called.
-  SetProfile(
-      g_browser_process->profile_manager()->GetProfileByPath(profile_path));
-}
-
 app_list::AppListModel* AppListViewDelegate::GetModel() {
   return model_;
 }
 
 app_list::SpeechUIModel* AppListViewDelegate::GetSpeechUI() {
   return speech_ui_.get();
 }
 
 void AppListViewDelegate::StartSearch() {
   if (search_controller_) {
@@ skipping 82 matching lines @@
         // the launcher to open.
         // TODO(amistry): This only works on ChromeOS since Chrome hides the
         // launcher instead of destroying it. Make this work on Chrome.
         if (hotword_service->IsAlwaysOnEnabled())
           hotword_service->RequestHotwordSession(nullptr);
       }
     }
   }
 }
 
-void AppListViewDelegate::OpenHelp() {
-  chrome::ScopedTabbedBrowserDisplayer displayer(profile_);
-  content::OpenURLParams params(GURL(chrome::kAppLauncherHelpURL),
-                                content::Referrer(),
-                                WindowOpenDisposition::NEW_FOREGROUND_TAB,
-                                ui::PAGE_TRANSITION_LINK, false);
-  displayer.browser()->OpenURL(params);
-}
-
-void AppListViewDelegate::OpenFeedback() {
-  Browser* browser = chrome::FindTabbedBrowser(profile_, false);
-  chrome::ShowFeedbackPage(browser, std::string(),
-                           chrome::kAppLauncherCategoryTag);
-}
-
 void AppListViewDelegate::StartSpeechRecognition() {
   StartSpeechRecognitionForHotword(nullptr);
 }
 
 void AppListViewDelegate::StopSpeechRecognition() {
   app_list::StartPageService* service =
       app_list::StartPageService::Get(profile_);
   if (service)
     service->StopSpeechRecognition();
 }
@@ skipping 24 matching lines @@
   // To get around this, always stop the session when switching to speech
   // recognition.
   if (service && service->HotwordEnabled()) {
     HotwordService* hotword_service =
         HotwordServiceFactory::GetForProfile(profile_);
     if (hotword_service)
       hotword_service->StopHotwordSession(this);
   }
 }
 
-void AppListViewDelegate::ShowForProfileByPath(
-    const base::FilePath& profile_path) {
-  controller_->ShowForProfileByPath(profile_path);
-}
-
 void AppListViewDelegate::OnSpeechResult(const base::string16& result,
                                          bool is_final) {
   speech_ui_->SetSpeechResult(result, is_final);
   if (is_final) {
     auto_launch_timeout_ = base::TimeDelta::FromMilliseconds(
         kAutoLaunchDefaultTimeoutMilliSec);
     is_voice_query_ = true;
     model_->search_box()->SetText(result);
   }
 }
@@ skipping 15 matching lines @@
       service && service->HotwordEnabled() &&
       controller_->GetAppListWindow()) {
     HotwordService* hotword_service =
         HotwordServiceFactory::GetForProfile(profile_);
     if (hotword_service) {
       hotword_service->RequestHotwordSession(this);
     }
   }
 }
 
-#if defined(TOOLKIT_VIEWS)
 views::View* AppListViewDelegate::CreateStartPageWebView(
     const gfx::Size& size) {
   app_list::StartPageService* service =
       app_list::StartPageService::Get(profile_);
   if (!service)
     return NULL;
 
   service->LoadContentsIfNeeded();
 
   content::WebContents* web_contents = service->GetStartPageContents();
@@ skipping 39 matching lines @@
 
 void AppListViewDelegate::CustomLauncherPageAnimationChanged(double progress) {
   if (launcher_page_event_dispatcher_)
     launcher_page_event_dispatcher_->ProgressChanged(progress);
 }
 
 void AppListViewDelegate::CustomLauncherPagePopSubpage() {
   if (launcher_page_event_dispatcher_)
     launcher_page_event_dispatcher_->PopSubpage();
 }
-#endif
 
 bool AppListViewDelegate::IsSpeechRecognitionEnabled() {
   app_list::StartPageService* service =
       app_list::StartPageService::Get(profile_);
   return service && service->GetSpeechRecognitionContents();
 }
 
-const app_list::AppListViewDelegate::Users&
-AppListViewDelegate::GetUsers() const {
-  return users_;
-}
-
 void AppListViewDelegate::OnTemplateURLServiceChanged() {
   TemplateURLService* template_url_service =
       TemplateURLServiceFactory::GetForProfile(profile_);
   const TemplateURL* default_provider =
       template_url_service->GetDefaultSearchProvider();
   bool is_google =
       default_provider->GetEngineType(
           template_url_service->search_terms_data()) ==
       SEARCH_ENGINE_GOOGLE;
 
   model_->SetSearchEngineIsGoogle(is_google);
 
   app_list::StartPageService* start_page_service =
       app_list::StartPageService::Get(profile_);
   if (start_page_service)
     start_page_service->set_search_engine_is_google(is_google);
 }
 
 void AppListViewDelegate::Observe(int type,
                                   const content::NotificationSource& source,
                                   const content::NotificationDetails& details) {
   DCHECK_EQ(chrome::NOTIFICATION_APP_TERMINATING, type);
 
   SetProfile(nullptr);  // Ensures launcher page web contents are torn down.
-
-  // SigninManagerFactory is not a leaky singleton (unlike this class), and
-  // its destructor will check that it has no remaining observers.
-  scoped_observer_.RemoveAll();
-  SigninManagerFactory::GetInstance()->RemoveObserver(this);
 }