Moves video frame callbacks from VideoFrameCompositor to Renderer.

media/base/renderer_client.h

 // Copyright 2016 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_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
 // to notify their clients.
 class RendererClient {
  public:
   // Executed if any error was encountered after Renderer initialization.
   virtual void OnError(PipelineStatus status) = 0;
 
   // Executed when rendering has reached the end of stream.
   virtual void OnEnded() = 0;
 
   // Executed periodically with rendering statistics.
   virtual void OnStatisticsUpdate(const PipelineStatistics& stats) = 0;
 
   // Executed when buffering state is changed.
   virtual void OnBufferingStateChange(BufferingState state) = 0;
 
   // 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
 
 #endif  // MEDIA_BASE_RENDERER_CLIENT_H_