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

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

Issue 2731043002: [Media Router] Add a presentation id to MediaRoute mapping in the MR (Closed)
Patch Set: resolve code review comments from Derek Created 3 years, 9 months 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_route.h
diff --git a/chrome/browser/media/router/media_route.h b/chrome/browser/media/router/media_route.h
index e2715846d60c73ef1d42b1bf941c67c182e7c64f..d21cec38a03ce9a819c2ef9a3d3ed61e60de9113 100644
--- a/chrome/browser/media/router/media_route.h
+++ b/chrome/browser/media/router/media_route.h
@@ -91,6 +91,13 @@ class MediaRoute {
}
bool is_offscreen_presentation() const { return is_offscreen_presentation_; }
+ void set_presentation_id(const std::string& presentation_id) {
+ presentation_id_ = presentation_id;
+ }
+ const base::Optional<std::string>& presentation_id() const {
+ return presentation_id_;
+ }
+
bool Equals(const MediaRoute& other) const;
private:
@@ -125,6 +132,9 @@ class MediaRoute {
// |true| if the presentation associated with this route is an offscreen
// presentation.
bool is_offscreen_presentation_ = false;
+
+ // Presentation id associated with this media route object.
+ base::Optional<std::string> presentation_id_;
};
} // namespace media_router
« no previous file with comments | « no previous file | chrome/browser/media/router/media_router.h » ('j') | chrome/browser/media/router/media_router_base.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698