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

Unified Diff: media/blink/video_frame_compositor.cc

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: Cleanup. 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
Index: media/blink/video_frame_compositor.cc
diff --git a/media/blink/video_frame_compositor.cc b/media/blink/video_frame_compositor.cc
index 36392a34b0e2bc6f4a4f0586490dd0f671fcf236..3cf2a1cb22d85b54c9f9e6859097e5ac3693c4c7 100644
--- a/media/blink/video_frame_compositor.cc
+++ b/media/blink/video_frame_compositor.cc
@@ -131,13 +131,12 @@ void VideoFrameCompositor::Stop() {
base::Unretained(this), false));
}
-void VideoFrameCompositor::PaintFrameUsingOldRenderingPath(
+void VideoFrameCompositor::PaintSingleFrame(
const scoped_refptr<VideoFrame>& frame) {
if (!compositor_task_runner_->BelongsToCurrentThread()) {
compositor_task_runner_->PostTask(
- FROM_HERE,
- base::Bind(&VideoFrameCompositor::PaintFrameUsingOldRenderingPath,
- base::Unretained(this), frame));
+ FROM_HERE, base::Bind(&VideoFrameCompositor::PaintSingleFrame,
+ base::Unretained(this), frame));
return;
}
@@ -172,7 +171,7 @@ base::TimeDelta VideoFrameCompositor::GetCurrentFrameTimestamp() const {
// When the VFC is stopped, |callback_| is cleared; this synchronously
// prevents CallRender() from invoking ProcessNewFrame(), and so
// |current_frame_| won't change again until after Start(). (Assuming that
- // PaintFrameUsingOldRenderingPath() is not also called while stopped.)
+ // PaintSingleFrame() is not also called while stopped.)
if (!current_frame_)
return base::TimeDelta();
return current_frame_->timestamp();

Powered by Google App Engine
This is Rietveld 408576698