Detect change on video/viewport intersection when rendered remotely

media/blink/webmediaplayer_impl.h

 // Copyright 2013 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 MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
 #define MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 202 matching lines @@
 
   // Called from WebMediaPlayerCast.
   // TODO(hubbe): WMPI_CAST make private.
   void OnPipelineSeeked(bool time_updated);
 
   // Restart the player/pipeline as soon as possible. This will destroy the
   // current renderer, if any, and create a new one via the RendererFactory; and
   // then seek to resume playback at the current position.
   void ScheduleRestart();
 
+  // Called when media starts/stops being rendered remotely.
+  void OnRemoteRenderingChanged(bool is_displayed_remotely);

[xhwang, 2016/12/14 18:11:41]

Do rendering and displayed mean the same thing?

Does it make sense to add an enum to make it more readable?

enum class RenderingState {
  LOCAL,
  REMOTE
};

onRenderingStateChange(RenderingState rendering_state);

[xjz, 2016/12/15 00:01:41]

This name seems confusing. This is not actually telling whether we are using
local or remoting renderer. This is actually used to activate whether to monitor
the changes on viewport intersection. Renamed.

+
   // Distinct states that |delegate_| can be in.
   // TODO(sandersd): This should move into WebMediaPlayerDelegate.
   // (Public for testing.)
   enum class DelegateState {
     GONE,
     PLAYING,
     PAUSED,
     ENDED,
   };
 
@@ skipping 345 matching lines @@
 
   // Whether the player is currently in autoplay muted state.
   bool autoplay_muted_ = false;
 
   DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_