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

chrome/browser/media/router/media_route.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTE_H_
 #define CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTE_H_
 
 #include <string>
 
 #include "base/containers/small_map.h"
@@ skipping 18 matching lines @@
   // |media_route_id|: ID of the route. New route IDs should be created
   //                   by the RouteIdManager class.
   // |media_source|: Description of source of the route.
   // |media_sink|: The sink that is receiving the media.
   // |description|: Description of the route to be displayed.
   // |is_local|: true if the route was created from this browser.
   // |custom_controller_path|: custom controller path if it is given by route
   //                      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.
   const MediaRoute::Id& media_route_id() const { return media_route_id_; }
 
   // The media source being routed.
   const MediaSource& media_source() const { return media_source_; }
 
   // The ID of sink being routed to.
   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."
   // 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.)
   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.
   const std::string& custom_controller_path() const {
     return custom_controller_path_;
   }
 
   bool for_display() const { return for_display_; }
 
+  bool is_offscreen_presentation() const { return is_offscreen_presentation_; }
+
   bool Equals(const MediaRoute& other) const;
 
  private:
   MediaRoute::Id media_route_id_;
   MediaSource media_source_;
   MediaSink::Id media_sink_id_;
   std::string description_;
   bool is_local_;
   std::string custom_controller_path_;
   bool for_display_;
+  bool is_offscreen_presentation_;
 };
 
 }  // namespace media_router
 
 #endif  // CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTE_H_