Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3325)

Unified Diff: cc/layers/video_frame_provider.h

Issue 1996763002: Make painting a single frame a permanent API on VideoRendererSink. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments. Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chromecast/renderer/media/cma_renderer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layers/video_frame_provider.h
diff --git a/cc/layers/video_frame_provider.h b/cc/layers/video_frame_provider.h
index e7619a3fb7a84cf48c2ece8be0bbe97d0d6983ef..a6789c11166f930c9110f54894bb07b7392ab2e8 100644
--- a/cc/layers/video_frame_provider.h
+++ b/cc/layers/video_frame_provider.h
@@ -43,8 +43,6 @@ class CC_EXPORT VideoFrameProvider {
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:
@@ -75,9 +73,6 @@ class CC_EXPORT VideoFrameProvider {
//
// 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
« no previous file with comments | « no previous file | chromecast/renderer/media/cma_renderer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698