[Presentation API] 1-UA presentation support + presenter APIs.

chrome/browser/media/router/media_route.h

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 b3056892f087b5138652036d1cc008acb0be66bd..10e2e4416a34ebc64ec525aa939ebddd7be235ef 100644
--- a/chrome/browser/media/router/media_route.h
+++ b/chrome/browser/media/router/media_route.h
@@ -36,13 +36,16 @@ class MediaRoute {
   //                      provider. empty otherwise.
   // |for_display|: Set to true if this route should be displayed for
   //                |media_sink_id| in UI.
+  // |is_offscreen_presentation|: Set to true if the route corresponds to an
+  //                              offscreen presentation.
   MediaRoute(const MediaRoute::Id& media_route_id,

[miu, 2015/09/27 00:22:08]

8 arguments?!?  IMO, you badly need to make this 3 arguments or less and
initialize the other data members to common default values.  Then, introduce
setter methods and pass const MediaRoutes around if you're worried about other
code trying to change these objects.

[imcheng, 2015/09/30 01:13:41]

I have removed the arg from the ctor and added a setter. For longer term I want
to see if we can do a builder pattern so tests don't have to specify the other
common args. The only place in production code that uses it actually already
have all the args.

              const MediaSource& media_source,
              const MediaSink::Id& media_sink_id,
              const std::string& description,
              bool is_local,
              const std::string& custom_controller_path,
-             bool for_display);
+             bool for_display,
+             bool is_offscreen_presentation);
   ~MediaRoute();
 
   // The media route identifier.
@@ -72,6 +75,8 @@ class MediaRoute {
 
   bool for_display() const { return for_display_; }
 
+  bool is_offscreen_presentation() const { return is_offscreen_presentation_; }
+
   bool Equals(const MediaRoute& other) const;
 
  private:
@@ -82,6 +87,7 @@ class MediaRoute {
   bool is_local_;
   std::string custom_controller_path_;
   bool for_display_;
+  bool is_offscreen_presentation_;
 };
 
 }  // namespace media_router