[Presentation API] (1st) (1-UA) Add is_offscreen_presentation attribute to MediaRouter

chrome/browser/media/router/create_presentation_connection_request.h

Index: chrome/browser/media/router/create_presentation_connection_request.h
diff --git a/chrome/browser/media/router/create_presentation_connection_request.h b/chrome/browser/media/router/create_presentation_connection_request.h
index ebf4b080990e26a9c3eb8f76f9da4d58c239296b..9dd9c037e198600e3c1df1a357ef043f47736e60 100644
--- a/chrome/browser/media/router/create_presentation_connection_request.h
+++ b/chrome/browser/media/router/create_presentation_connection_request.h
@@ -36,7 +36,7 @@ class CreatePresentationConnectionRequest {
  public:
   using PresentationSessionSuccessCallback =
       base::Callback<void(const content::PresentationSessionInfo&,
-                          const MediaRoute::Id&)>;
+                          const MediaRoute&)>;
   using PresentationSessionErrorCallback =
       content::PresentationSessionErrorCallback;
   // |presentation_url|: The presentation URL of the request. Must be a valid
@@ -60,7 +60,7 @@ class CreatePresentationConnectionRequest {
   // These functions can only be invoked once per instance. It is an error
   // to invoke these functions more than once.
   void InvokeSuccessCallback(const std::string& presentation_id,
-                             const MediaRoute::Id& route_id);
+                             const MediaRoute& route);

[mark a. foltz, 2016/11/03 18:32:05]

Thanks for doing this, it makes a lot more sense to pass objects instead of IDs
and we should get rid of MediaRoute::Id in our APIs.

[zhaobin, 2016/11/03 23:00:46]

Will do it in a seperate CL. Create
https://bugs.chromium.org/p/chromium/issues/detail?id=662208# for tracking.

   void InvokeErrorCallback(const content::PresentationError& error);
 
   // Handle route creation/joining response by invoking the right callback.