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

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 60 matching lines @@
     return custom_controller_path_;
   }
 
   bool for_display() const { return for_display_; }
 
   // Set this to true when the route was created by an incognito profile.
   void set_incognito(bool incognito) { incognito_ = incognito; }
 
   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:
   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 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
 
 #endif  // CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTE_H_