[MediaRouter] Add implementation of PresentationServiceDelegate

chrome/browser/media/router/media_source_helper.cc

Index: chrome/browser/media/router/media_source_helper.cc
diff --git a/chrome/browser/media/router/media_source_helper.cc b/chrome/browser/media/router/media_source_helper.cc
index 305a00fa3437dd83fa5b8eba3de0bdb61e65380c..921e7ce30fa249b44a9cfdd1e365bb089a724e47 100644
--- a/chrome/browser/media/router/media_source_helper.cc
+++ b/chrome/browser/media/router/media_source_helper.cc
@@ -26,7 +26,7 @@ MediaSource ForDesktopMediaSource() {
 }
 
 // TODO(mfoltz): Remove when the TODO in
-// MediaSourceManager::GetDefaultMediaSource is resolved.
+// PresentationServiceDelegateImpl::GetDefaultMediaSource is resolved.
 MediaSource ForCastAppMediaSource(const std::string& app_id) {
   return MediaSource(kCastUrnPrefix + app_id);
 }
@@ -49,4 +49,9 @@ bool IsValidMediaSource(const MediaSource& source) {
   return url.is_valid() && url.SchemeIsHTTPOrHTTPS();
 }
 
+std::string GetPresentationUrl(const MediaSource& source) {
+  DCHECK(IsValidPresentationUrl(source.id()));

[mark a. foltz, 2015/05/14 22:20:44]

What behavior is intended in release builds?

I would write this as:


return IsValidPresentationUrl(source.id()) ? source.id() : "";

[haibinlu, 2015/05/15 23:32:29]

Done.

+  return source.id();
+}
+
 }  // namespace media_router