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

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

Issue 2666873006: [Media Router] Convert to use typemaps for media_router.mojom. (Closed)
Patch Set: Remove DCHECK since tests are hitting the code path Created 3 years, 10 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
« no previous file with comments | « chrome/browser/media/router/BUILD.gn ('k') | chrome/browser/media/router/media_route.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 e3aa5a3b6762ffd1bc8df463cc543e0931f18e00..e2715846d60c73ef1d42b1bf941c67c182e7c64f 100644
--- a/chrome/browser/media/router/media_route.h
+++ b/chrome/browser/media/router/media_route.h
@@ -44,60 +44,87 @@ class MediaRoute {
const std::string& custom_controller_path,
bool for_display);
MediaRoute(const MediaRoute& other);
+ MediaRoute();
+
~MediaRoute();
- // The media route identifier.
+ void set_media_route_id(const MediaRoute::Id& media_route_id) {
+ media_route_id_ = media_route_id;
+ }
const MediaRoute::Id& media_route_id() const { return media_route_id_; }
- // The media source being routed.
+ void set_media_source(const MediaSource& media_source) {
+ media_source_ = media_source;
+ }
const MediaSource& media_source() const { return media_source_; }
- // The ID of sink being routed to.
+ void set_media_sink_id(const MediaSink::Id& media_sink_id) {
+ media_sink_id_ = media_sink_id;
+ }
const MediaSink::Id& media_sink_id() const { return media_sink_id_; }
- // The description of the media route activity, for example
- // "Playing Foo Bar Music All Access."
+ void set_description(const std::string& description) {
+ description_ = description;
+ }
+
// TODO(kmarshall): Do we need to pass locale for bidi rendering?
const std::string& description() const { return description_; }
- // Returns |true| if the route is created locally (versus discovered
- // by a media route provider.)
+ void set_local(bool is_local) { is_local_ = is_local; }
bool is_local() const { return is_local_; }
- // The custom controller path. This allows route provider to have custom route
- // detail as well as its own route control features route control features in
- // the media router dialog.
+ void set_custom_controller_path(const std::string& custom_controller_path) {
+ custom_controller_path_ = custom_controller_path;
+ }
const std::string& custom_controller_path() const {
return custom_controller_path_;
}
+ void set_for_display(bool for_display) { for_display_ = for_display; }
bool for_display() const { return for_display_; }
- // Set this to true when the route was created by an incognito profile.
void set_incognito(bool is_incognito) { is_incognito_ = is_incognito; }
-
bool is_incognito() const { return is_incognito_; }
- // Set to |true| if the presentation associated with this route is an
- // offscreen presentation.
void set_offscreen_presentation(bool is_offscreen_presentation) {
is_offscreen_presentation_ = is_offscreen_presentation;
}
-
bool is_offscreen_presentation() const { return is_offscreen_presentation_; }
bool Equals(const MediaRoute& other) const;
private:
+ // The media route identifier.
MediaRoute::Id media_route_id_;
+
+ // The media source being routed.
MediaSource media_source_;
+
+ // The ID of sink being routed to.
MediaSink::Id media_sink_id_;
+
+ // The description of the media route activity, for example
+ // "Playing Foo Bar Music All Access."
std::string description_;
- bool is_local_;
+
+ // |true| if the route is created locally (versus discovered by a media route
+ // provider.)
+ bool is_local_ = false;
+
+ // The custom controller path. This allows route provider to have custom route
+ // detail as well as its own route control features route control features in
+ // the media router dialog.
std::string custom_controller_path_;
- bool for_display_;
- bool is_incognito_;
- bool is_offscreen_presentation_;
+
+ // |true| if the route can be displayed in the UI.
+ bool for_display_ = false;
+
+ // |true| if the route was created by an incognito profile.
+ bool is_incognito_ = false;
+
+ // |true| if the presentation associated with this route is an offscreen
+ // presentation.
+ bool is_offscreen_presentation_ = false;
};
} // namespace media_router
« no previous file with comments | « chrome/browser/media/router/BUILD.gn ('k') | chrome/browser/media/router/media_route.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698