[DO NOT COMMIT] Initial MediaPlayerRenderer plumbing and basic features

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 49b098e70511d3ecc978794e3013c34e7ebd52b6..402c31b2e877db6a9a2b292392df282d9bb5efc9 100644
--- a/media/base/renderer_client.h
+++ b/media/base/renderer_client.h
@@ -5,13 +5,14 @@
 #ifndef MEDIA_BASE_RENDERER_CLIENT_H_
 #define MEDIA_BASE_RENDERER_CLIENT_H_
 
+#include "base/time/time.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.
+// Interface used by Renderer, AudioRenderer, VideoRenderer and
+// MediaPlayerRenderer implementations to notify their clients.
 class RendererClient {
  public:
   // Executed if any error was encountered after Renderer initialization.
@@ -36,6 +37,10 @@ class RendererClient {
   // Executed for the first video frame and whenever opacity changes.
   // Only used if media stream contains video track.
   virtual void OnVideoOpacityChange(bool opaque) = 0;
+
+  // Executed when video metadata is first read, and whenever it changes.
+  // Only used when we are using a URL demuxer (e.g. for MediaPlayerRenderer).
+  virtual void OnDurationChange(base::TimeDelta duration) = 0;

[xhwang, 2016/06/22 17:39:02]

As discussed offline, this change sgtm :) Now we have two sources of duration
(from Renderer and from Demuxer). Somewhere we need to document about what the
expectation is. For example, we shouldn't get duration change from both sources.

 };
 
 }  // namespace media