mash: Have chrome set itself as the app list presenter.

ash/mus/app_list_presenter_mus.cc

 // Copyright 2016 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 "ash/mus/app_list_presenter_mus.h"
 
-#include "content/public/common/service_names.mojom.h"
-#include "services/service_manager/public/cpp/connector.h"
+#include "ash/common/wm_shell.h"
+#include "ui/app_list/presenter/app_list.h"
 
 namespace ash {
 
-namespace {
-
-bool HasConnection(app_list::mojom::AppListPresenterPtr* interface_ptr) {
-  return interface_ptr->is_bound() && !interface_ptr->encountered_error();
-}
-
-}  // namespace
-
-AppListPresenterMus::AppListPresenterMus(service_manager::Connector* connector)
-    : connector_(connector) {}
+AppListPresenterMus::AppListPresenterMus() {}
 
 AppListPresenterMus::~AppListPresenterMus() {}
 
 void AppListPresenterMus::Show(int64_t display_id) {
-  ConnectIfNeeded();
-  presenter_->Show(display_id);
+  app_list::AppList* app_list = WmShell::Get()->app_list();

[mfomitchev, 2016/12/12 22:59:49]

nit: why not do 
app_list_presenter = WmShell::Get()->app_list()->GetAppListPresenter()
here and below

[msw, 2016/12/12 23:47:54]

Done.

+  if (app_list->GetAppListPresenter())
+    app_list->GetAppListPresenter()->Show(display_id);
 }
 
 void AppListPresenterMus::Dismiss() {
-  ConnectIfNeeded();
-  presenter_->Dismiss();
+  app_list::AppList* app_list = WmShell::Get()->app_list();
+  if (app_list->GetAppListPresenter())
+    app_list->GetAppListPresenter()->Dismiss();
 }
 
 void AppListPresenterMus::ToggleAppList(int64_t display_id) {
-  ConnectIfNeeded();
-  presenter_->ToggleAppList(display_id);
+  app_list::AppList* app_list = WmShell::Get()->app_list();
+  if (app_list->GetAppListPresenter())
+    app_list->GetAppListPresenter()->ToggleAppList(display_id);
 }
 
 bool AppListPresenterMus::IsVisible() const {
   return false;
 }
 
 bool AppListPresenterMus::GetTargetVisibility() const {
   // TODO(mfomitchev): we have GetTargetVisibility() in mojom, but this
   // shouldn't be a synchronous method. We should go through the call sites and
   // either teach them to use a callback, or perhaps use a visibility observer.
   //  NOTIMPLEMENTED();
   return false;
 }
 
-void AppListPresenterMus::ConnectIfNeeded() {
-  if (!connector_ || HasConnection(&presenter_))
-    return;
-  connector_->ConnectToInterface(content::mojom::kBrowserServiceName,
-                                 &presenter_);
-  CHECK(HasConnection(&presenter_))
-      << "Could not connect to app_list::mojom::AppListPresenter.";
-}
-
 }  // namespace ash