Moves video frame callbacks from VideoFrameCompositor to Renderer.

media/base/renderer_client.h

Index: media/base/renderer_client.h
diff --git a/media/base/renderer_client.h b/media/base/renderer_client.h
index 56cc15eaa5be9bf1db3af6cb0e1bb977efe11834..775dc7802552875a3a317dbfada1ba7c30c3fc6d 100644
--- a/media/base/renderer_client.h
+++ b/media/base/renderer_client.h
@@ -5,6 +5,9 @@
 #ifndef MEDIA_BASE_RENDERER_CLIENT_H_
 #define MEDIA_BASE_RENDERER_CLIENT_H_
 
+#include "media/base/pipeline_status.h"
+#include "ui/gfx/geometry/size.h"
+
 namespace media {
 
 // Interface used by Renderer, AudioRenderer, and VideoRenderer implementations
@@ -25,6 +28,12 @@ class RendererClient {
 
   // Executed whenever the key needed to decrypt the stream is not available.
   virtual void OnWaitingForDecryptionKey() = 0;
+
+  // Executed for the first video frame and whenever natural size changes.
+  virtual void OnVideoNaturalSizeChange(const gfx::Size& size) = 0;
+
+  // Executed for the first video frame and whenever opacity changes.
+  virtual void OnVideoOpacityChange(bool opaque) = 0;

[xhwang, 2016/05/17 06:24:12]

This client is also used by AudioRenderer, where these two calls don't make
sense. Add a comment/TODO about improve that later?

[alokp, 2016/05/17 16:25:49]

Added comment. This is equivalent to DemuxerStream interface where a portion of
the API is specific to Audio and Video renderers. I do not think this warrants
separate Client interfaces for Audio and Video renderers.

I also already have checks in place to ensure that it does not get invoked by
AudioRenderer. See AudioRendererImplTest and RendererInternal.

 };
 
 }  // namespace media