Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1270)

Unified Diff: chrome/browser/media/router/media_source.h

Issue 2547703002: [Media Router] Handle multiple Presentation URLs when creating routes (Closed)
Patch Set: resolve code review comments from Derek and Mark Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/media/router/media_source.h
diff --git a/chrome/browser/media/router/media_source.h b/chrome/browser/media/router/media_source.h
index 279edf0c49ff3c23dd29763ea4e5395cf1c88e03..ac5da7148ec28f700e8f624645b20a195ede1040 100644
--- a/chrome/browser/media/router/media_source.h
+++ b/chrome/browser/media/router/media_source.h
@@ -11,8 +11,7 @@
#include <string>
#include "base/hash.h"
-
-class GURL;
+#include "url/gurl.h"
// TODO(mfoltz): Right now this is a wrapper for std::string. Factor methods
// from media_source_helper here so this object becomes useful; and don't just
@@ -30,6 +29,10 @@ class MediaSource {
// Gets the ID of the media source.
MediaSource::Id id() const;
+ // If MediaSource is created from a URL, return the URL; otherwise return an
+ // empty GURL.
+ GURL url() const;
+
// Returns true if two MediaSource objects use the same media ID.
bool operator==(const MediaSource& other) const;
@@ -45,6 +48,7 @@ class MediaSource {
private:
MediaSource::Id id_;
+ GURL url_;
};
} // namespace media_router

Powered by Google App Engine
This is Rietveld 408576698