[Media Router] Add a presentation id to MediaRoute mapping in the MR

chrome/browser/media/router/media_router_base.cc

Index: chrome/browser/media/router/media_router_base.cc
diff --git a/chrome/browser/media/router/media_router_base.cc b/chrome/browser/media/router/media_router_base.cc
index 923afc4efc79b6e37ec7dc9f865c8d1f5b3f6c2b..61e3b7b142f862c3a6132d4dc0fff5ef3d94b0a3 100644
--- a/chrome/browser/media/router/media_router_base.cc
+++ b/chrome/browser/media/router/media_router_base.cc
@@ -38,6 +38,18 @@ class MediaRouterBase::InternalMediaRoutesObserver
     }
   }
 
+  const MediaRoute* GetPresentationRoute(
+      const std::string& presentation_id) const {
+    for (const auto& route : current_routes) {
+      if (route.presentation_id().has_value() &&

[mark a. foltz, 2017/03/10 00:56:28]

I think .has_value() is optional here.

[zhaobin, 2017/03/10 01:51:11]

Done.

+          (*route.presentation_id()) == presentation_id) {

[mark a. foltz, 2017/03/10 00:56:28]

In other code I have used .value() explicitly to retrieve the value (so I don't
have to remember what * does) but I don't feel strongly either way.

[zhaobin, 2017/03/10 01:51:11]

Done.

+        return &route;
+      }
+    }
+
+    return nullptr;
+  }
+
   bool has_route;
   std::vector<MediaRoute> current_routes;
   std::vector<MediaRoute::Id> incognito_route_ids;
@@ -76,6 +88,12 @@ std::vector<MediaRoute> MediaRouterBase::GetCurrentRoutes() const {
   return internal_routes_observer_->current_routes;
 }
 
+const MediaRoute* MediaRouterBase::GetPresentationRoute(
+    const std::string& presentation_id) const {
+  DCHECK(internal_routes_observer_);
+  return internal_routes_observer_->GetPresentationRoute(presentation_id);
+}
+
 MediaRouterBase::MediaRouterBase() : initialized_(false) {}
 
 // static