Add OnDurationChange() to RenderClient

media/renderers/renderer_impl.cc

 // Copyright 2014 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.
 
 #include "media/renderers/renderer_impl.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 40 matching lines @@
     renderer_->OnWaitingForDecryptionKey();
   }
   void OnVideoNaturalSizeChange(const gfx::Size& size) override {
     DCHECK(type_ == DemuxerStream::VIDEO);
     renderer_->OnVideoNaturalSizeChange(size);
   }
   void OnVideoOpacityChange(bool opaque) override {
     DCHECK(type_ == DemuxerStream::VIDEO);
     renderer_->OnVideoOpacityChange(opaque);
   }
+  void OnDurationChange(base::TimeDelta duration) override {
+    // RendererClients should only be notified of duration changes in certain
+    // scenarios, none of which should arrise for RendererClientInternal.

[watk, 2016/08/12 00:49:06]

arise

[tguilbert, 2016/08/12 00:58:15]

Done.

+    // Duration changes should be sent to the pipeline by the DemuxerStream, via
+    // the DemuxerHost interface.
+    NOTREACHED();
+  }
 
  private:
   DemuxerStream::Type type_;
   RendererImpl* renderer_;
 };
 
 RendererImpl::RendererImpl(
     const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
     std::unique_ptr<AudioRenderer> audio_renderer,
     std::unique_ptr<VideoRenderer> video_renderer)
@@ skipping 771 matching lines @@
   DCHECK(task_runner_->BelongsToCurrentThread());
   client_->OnVideoNaturalSizeChange(size);
 }
 
 void RendererImpl::OnVideoOpacityChange(bool opaque) {
   DCHECK(task_runner_->BelongsToCurrentThread());
   client_->OnVideoOpacityChange(opaque);
 }
 
 }  // namespace media