Create MediaRouterActionController and MediaRouterUIService

chrome/browser/media/router/media_router_base.cc

 // Copyright 2015 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/media/router/media_router_base.h"
 
 #include "base/bind.h"
 #include "base/guid.h"
 #include "base/memory/ptr_util.h"
 #include "base/stl_util.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/profiles/profile.h"
+#if !defined(OS_ANDROID)
+#include "chrome/browser/ui/toolbar/media_router_action_controller.h"
+#endif
 #include "content/public/browser/browser_thread.h"
 
 namespace media_router {
 
 // A MediaRoutesObserver that maintains state about the current set of media
 // routes.
 class MediaRouterBase::InternalMediaRoutesObserver
     : public MediaRoutesObserver {
  public:
   explicit InternalMediaRoutesObserver(MediaRouter* router)
@@ skipping 42 matching lines @@
   }
 
   return callbacks->Add(callback);
 }
 
 void MediaRouterBase::OnIncognitoProfileShutdown() {
   for (const auto& route_id : internal_routes_observer_->incognito_route_ids)
     TerminateRoute(route_id);
 }
 
-MediaRouterBase::MediaRouterBase() : initialized_(false) {}
+#if !defined(OS_ANDROID)
+MediaRouterActionController* MediaRouterBase::GetActionController() {
+  return action_controller_.get();
+}
+#endif
+
+MediaRouterBase::MediaRouterBase(Profile* profile)
+    : initialized_(false),
+      profile_(profile) {}
 
 // static
 std::string MediaRouterBase::CreatePresentationId() {
   return "mr_" + base::GenerateGUID();
 }
 
 void MediaRouterBase::NotifyPresentationConnectionStateChange(
     const MediaRoute::Id& route_id,
     content::PresentationConnectionState state) {
   auto* callbacks = presentation_connection_state_callbacks_.get(route_id);
@@ skipping 17 matching lines @@
   info.message = message;
   callbacks->Notify(info);
 }
 
 bool MediaRouterBase::HasJoinableRoute() const {
   return internal_routes_observer_->has_route;
 }
 
 void MediaRouterBase::Initialize() {
   DCHECK(!initialized_);
-  // The observer calls virtual methods on MediaRouter; it must be created
-  // outside of the ctor
+  // |internal_routes_observer_| and |action_controller_| call virtual methods
+  // on MediaRouter; they must be created outside of the ctor.
   internal_routes_observer_.reset(new InternalMediaRoutesObserver(this));
+#if !defined(OS_ANDROID)
+  // TODO(takumif): Actually instantiate MediaRouterActionController here to
+  // enable ephemeral toolbar icon.
+  action_controller_.reset();

[mark a. foltz, 2016/09/02 22:19:09]

Can this TODO be done now?

[takumif, 2016/09/07 21:48:32]

We need to change MRAction to use MRActionController before we do that, which I
plan to do in another CL. I'll do this TODO there.

+#endif
+
   initialized_ = true;
 }
 
 void MediaRouterBase::OnPresentationConnectionStateCallbackRemoved(
     const MediaRoute::Id& route_id) {
   auto* callbacks = presentation_connection_state_callbacks_.get(route_id);
   if (callbacks && callbacks->empty())
     presentation_connection_state_callbacks_.erase(route_id);
 }
 
 void MediaRouterBase::Shutdown() {
-  // The observer calls virtual methods on MediaRouter; it must be destroyed
-  // outside of the dtor
+  // |internal_routes_observer_| and |action_controller_| call virtual methods
+  // on MediaRouter; they must be destroyed outside of the dtor.
   internal_routes_observer_.reset();
+#if !defined(OS_ANDROID)
+  action_controller_.reset();
+#endif
 }
 
 }  // namespace media_router