[Media Router] Custom Controls 2 - add MediaRouter::GetRouteController()

chrome/browser/media/router/media_router_base.h

Index: chrome/browser/media/router/media_router_base.h
diff --git a/chrome/browser/media/router/media_router_base.h b/chrome/browser/media/router/media_router_base.h
index e2710031b7627726cd609027c47c3cd1865a6082..cd045e6b52025f73a45b39d69101c1a8409e26a5 100644
--- a/chrome/browser/media/router/media_router_base.h
+++ b/chrome/browser/media/router/media_router_base.h
@@ -34,6 +34,9 @@ class MediaRouterBase : public MediaRouter {
 
   std::vector<MediaRoute> GetCurrentRoutes() const override;
 
+  scoped_refptr<MediaRouteController> GetRouteController(
+      const MediaRoute::Id& route_id) override;
+
  protected:
   FRIEND_TEST_ALL_PREFIXES(MediaRouterMojoImplTest,
                            PresentationConnectionStateChangedCallback);
@@ -59,6 +62,9 @@ class MediaRouterBase : public MediaRouter {
   // JoinRoute().
   bool HasJoinableRoute() const;
 
+  // Returns true if there is a route with the ID in the current list of routes.
+  bool IsRouteValid(const std::string& route_id) const;

[mark a. foltz, 2017/04/11 19:43:19]

This might be better named "IsRouteKnown".  It's checking whether the last route
update from the MRPM has the listed route.  It's not clear exactly what "valid"
means in this context.

[takumif, 2017/04/12 23:11:36]

Done.

+
   using PresentationConnectionStateChangedCallbacks = base::CallbackList<void(
       const content::PresentationConnectionStateChangeInfo&)>;
 
@@ -85,6 +91,10 @@ class MediaRouterBase : public MediaRouter {
   // KeyedService
   void Shutdown() override;
 
+  // MediaRouter
+  void DetachRouteController(const MediaRoute::Id& route_id,
+                             MediaRouteController* controller) override;
+
   std::unique_ptr<InternalMediaRoutesObserver> internal_routes_observer_;
   bool initialized_;