Make painting a single frame a permanent API on VideoRendererSink.

media/base/video_renderer_sink.h

Index: media/base/video_renderer_sink.h
diff --git a/media/base/video_renderer_sink.h b/media/base/video_renderer_sink.h
index 9dd62d87e6af3e00c8c63c221856d66ea76e3a6a..8ef9c642c2386e2729e0b46bed37283ced990e6e 100644
--- a/media/base/video_renderer_sink.h
+++ b/media/base/video_renderer_sink.h
@@ -53,16 +53,11 @@ class MEDIA_EXPORT VideoRendererSink {
   // 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() {}
 };