Make painting a single frame a permanent API on VideoRendererSink.

cc/layers/video_frame_provider.h

 // Copyright (c) 2012 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 CC_LAYERS_VIDEO_FRAME_PROVIDER_H_
 #define CC_LAYERS_VIDEO_FRAME_PROVIDER_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/time/time.h"
 #include "cc/base/cc_export.h"
@@ skipping 25 matching lines @@
     // UpdateCurrentFrame() calls to the provider. No further calls to
     // UpdateCurrentFrame() should be made once StopRendering() returns.
     //
     // Callers should use these methods to indicate when it expects and no
     // longer expects (respectively) to have new frames for the client. Clients
     // may use this information for power conservation.
     virtual void StartRendering() = 0;
     virtual void StopRendering() = 0;
 
     // Notifies the client that GetCurrentFrame() will return new data.
-    // TODO(dalecurtis): Nuke this once VideoFrameProviderClientImpl is using a
-    // BeginFrameObserver based approach. http://crbug.com/336733
     virtual void DidReceiveFrame() = 0;
 
    protected:
     virtual ~Client() {}
   };
 
   // May be called from any thread, but there must be some external guarantee
   // that the provider is not destroyed before this call returns.
   virtual void SetVideoFrameProviderClient(Client* client) = 0;
 
@@ skipping 10 matching lines @@
   // otherwise. Aside from thread locks, the state won't change.
   virtual bool HasCurrentFrame() = 0;
 
   // Returns the current frame, which may have been updated by a recent call to
   // UpdateCurrentFrame(). A call to this method does not ensure that the frame
   // will be rendered. A subsequent call to PutCurrentFrame() must be made if
   // the frame is expected to be rendered.
   //
   // Clients should call this in response to UpdateCurrentFrame() returning true
   // or in response to a DidReceiveFrame() call.
-  //
-  // TODO(dalecurtis): Remove text about DidReceiveFrame() once the old path
-  // has been removed. http://crbug.com/439548
   virtual scoped_refptr<media::VideoFrame> GetCurrentFrame() = 0;
 
   // Called in response to DidReceiveFrame() or a return value of true from
   // UpdateCurrentFrame() if the current frame was considered for rendering; the
   // frame may not been rendered for a variety of reasons (occlusion, etc).
   // Providers may use the absence of this call as a signal to detect when a new
   // frame missed its intended deadline.
   virtual void PutCurrentFrame() = 0;
 
  protected:
   virtual ~VideoFrameProvider() {}
 };
 
 }  // namespace cc
 
 #endif  // CC_LAYERS_VIDEO_FRAME_PROVIDER_H_