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

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 2d36e244292abf1b8bedd8ee54dd81807e69ee1a..d1d60b6e2ad63ed935dabfa1659e954a75c26921 100644
--- a/chrome/browser/media/router/media_route.h
+++ b/chrome/browser/media/router/media_route.h
@@ -78,6 +78,14 @@ class MediaRoute {
 
   bool incognito() const { return incognito_; }
 
+  // Set to |true| if the presentation associated with this route is an
+  // offscreen presentation.
+  void set_is_offscreen_presentation(bool is_offscreen_presentation) {

[mark a. foltz, 2016/11/09 19:28:22]

Can this be set_offscreen_presentation()?

[zhaobin, 2016/11/11 18:41:51]

Done.

+    is_offscreen_presentation_ = is_offscreen_presentation;
+  }
+
+  bool is_offscreen_presentation() const { return is_offscreen_presentation_; }
+
   bool Equals(const MediaRoute& other) const;
 
  private:
@@ -89,6 +97,7 @@ class MediaRoute {
   std::string custom_controller_path_;
   bool for_display_;
   bool incognito_;

[mark a. foltz, 2016/11/09 19:28:22]

Side comment: Looking at the rest of this object, this should be is_incognito_
and the getter is_incognito(); my fault :)

Not necessary to fix in this patch at all.

[zhaobin, 2016/11/11 18:41:51]

Will do it later in seperate patch.

+  bool is_offscreen_presentation_;
 };
 
 }  // namespace media_router