Create MediaRouterActionController and MediaRouterUIService

chrome/browser/media/router/media_router_ui_service_factory.cc

Index: chrome/browser/media/router/media_router_ui_service_factory.cc
diff --git a/chrome/browser/media/router/media_router_ui_service_factory.cc b/chrome/browser/media/router/media_router_ui_service_factory.cc
new file mode 100644
index 0000000000000000000000000000000000000000..fb4b16e75c37485bdd6a7dda4a1611d225353b0a
--- /dev/null
+++ b/chrome/browser/media/router/media_router_ui_service_factory.cc
@@ -0,0 +1,54 @@
+// 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 "chrome/browser/media/router/media_router_ui_service_factory.h"
+
+#include "chrome/browser/media/router/media_router_factory.h"
+#include "chrome/browser/media/router/media_router_ui_service.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/toolbar/toolbar_actions_model_factory.h"
+#include "components/keyed_service/content/browser_context_dependency_manager.h"
+
+using content::BrowserContext;
+
+namespace media_router {
+
+// static
+MediaRouterUIService* MediaRouterUIServiceFactory::GetForBrowserContext(
+    BrowserContext* context) {
+  DCHECK(context);
+  // GetServiceForBrowserContext returns a KeyedService hence the static_cast<>
+  // to return a pointer to MediaRouterUIService.
+  return static_cast<MediaRouterUIService*>(
+      GetInstance()->GetServiceForBrowserContext(context, true));
+}
+
+// static
+MediaRouterUIServiceFactory* MediaRouterUIServiceFactory::GetInstance() {
+  return base::Singleton<MediaRouterUIServiceFactory>::get();
+}
+
+MediaRouterUIServiceFactory::MediaRouterUIServiceFactory()
+    : BrowserContextKeyedServiceFactory(
+          "MediaRouterUIService",
+          BrowserContextDependencyManager::GetInstance()) {
+  DependsOn(MediaRouterFactory::GetInstance());
+  // MediaRouterUIService owns a MediaRouterActionController that depends on
+  // ToolbarActionsModel.
+  DependsOn(ToolbarActionsModelFactory::GetInstance());
+}
+
+MediaRouterUIServiceFactory::~MediaRouterUIServiceFactory() {}
+
+BrowserContext* MediaRouterUIServiceFactory::GetBrowserContextToUse(
+    BrowserContext* context) const {
+  return context;

[mark a. foltz, 2016/09/13 18:23:05]

This will mean a separate services gets instantiated for an incognito profile
spawned from a regular profile.  This seems fine as the media router (which is
the same across profiles) should advertise the identical set of routes to
observers.

I do have a question about pref behavior.

[takumif, 2016/09/14 04:00:52]

ToolbarActionsModel separates normal and incognito profiles, so it probably
makes sense to do so here as well.

+}
+
+KeyedService* MediaRouterUIServiceFactory::BuildServiceInstanceFor(
+    BrowserContext* context) const {
+  return new MediaRouterUIService(Profile::FromBrowserContext(context));
+}
+
+}  // namespace media_router