Make painting a single frame a permanent API on VideoRendererSink.

media/base/video_renderer_sink.h

 // Copyright 2015 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_VIDEO_RENDERER_SINK_H_
 #define MEDIA_BASE_VIDEO_RENDERER_SINK_H_
 
 #include "base/logging.h"
 #include "base/memory/ref_counted.h"
 #include "base/time/time.h"
@@ skipping 35 matching lines @@
   // |callback| must be ready to receive callbacks before calling Start().
   virtual void Start(RenderCallback* callback) = 0;
 
   // Stops video rendering, waits for any outstanding calls to the |callback|
   // given to Start() to complete before returning.  No new calls to |callback|
   // will be issued after this method returns.  May be used as a means of power
   // conservation by the sink implementation, so clients should call this
   // liberally if no new frames are expected.
   virtual void Stop() = 0;
 
-  // Instead of using a callback driven rendering path, allow clients to paint
-  // frames as they see fit without regard for the compositor.
-  // TODO(dalecurtis): This should be nuked once experiments show the new path
-  // is amazing and the old path is not! http://crbug.com/439548
-  virtual void PaintFrameUsingOldRenderingPath(
-      const scoped_refptr<VideoFrame>& frame) = 0;
-
-  // TODO(dalecurtis): We may need OnSizeChanged() and OnOpacityChanged()
-  // methods on this interface if background rendering is handled inside of
-  // the media layer instead of by VideoFrameCompositor.
+  // Instead of using a callback driven rendering path, allow clients to paint a
+  // single frame as they see fit without regard for the compositor; this is
+  // useful for painting poster images or hole frames without having to issue a
+  // Start() -> Render() -> Stop().

[xhwang, 2016/05/19 22:47:28]

nit: Can we call both PSF and Start/Render/Stop?

[DaleCurtis, 2016/05/20 18:18:39]

Yes, I've updated the docs.

+  virtual void PaintSingleFrame(const scoped_refptr<VideoFrame>& frame) = 0;
 
   virtual ~VideoRendererSink() {}
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_RENDERER_SINK_H_