Detect change on video/viewport intersection when rendered remotely

media/blink/webmediaplayer_impl.h

Index: media/blink/webmediaplayer_impl.h
diff --git a/media/blink/webmediaplayer_impl.h b/media/blink/webmediaplayer_impl.h
index 6e01840d71c6e0e05129c139d91354fd04f2f4d9..41d20f7e1f3da43c47c3d7a78af1cb735216476c 100644
--- a/media/blink/webmediaplayer_impl.h
+++ b/media/blink/webmediaplayer_impl.h
@@ -220,6 +220,9 @@ class MEDIA_BLINK_EXPORT WebMediaPlayerImpl
   // 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.)