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

Unified Diff: content/shell/renderer/layout_test/test_media_stream_video_renderer.cc

Issue 2005053006: Cleanup naming issue with content::VideoFrameProvider (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address 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
Index: content/shell/renderer/layout_test/test_media_stream_video_renderer.cc
diff --git a/content/shell/renderer/layout_test/test_video_frame_provider.cc b/content/shell/renderer/layout_test/test_media_stream_video_renderer.cc
similarity index 63%
rename from content/shell/renderer/layout_test/test_video_frame_provider.cc
rename to content/shell/renderer/layout_test/test_media_stream_video_renderer.cc
index 7a19a02975e197fcf7b00ce4d976559d3e83c45b..10cf4ea79523ec276b5ff45fe3355775e33bfda5 100644
--- a/content/shell/renderer/layout_test/test_video_frame_provider.cc
+++ b/content/shell/renderer/layout_test/test_media_stream_video_renderer.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "content/shell/renderer/layout_test/test_video_frame_provider.h"
+#include "content/shell/renderer/layout_test/test_media_stream_video_renderer.h"
#include "base/bind.h"
#include "base/location.h"
@@ -12,11 +12,11 @@
namespace content {
-TestVideoFrameProvider::TestVideoFrameProvider(
+TestMediaStreamVideoRenderer::TestMediaStreamVideoRenderer(
const gfx::Size& size,
const base::TimeDelta& frame_duration,
const base::Closure& error_cb,
- const VideoFrameProvider::RepaintCB& repaint_cb)
+ const MediaStreamVideoRenderer::RepaintCB& repaint_cb)
: task_runner_(base::ThreadTaskRunnerHandle::Get()),
size_(size),
state_(kStopped),
@@ -25,38 +25,39 @@ TestVideoFrameProvider::TestVideoFrameProvider(
repaint_cb_(repaint_cb) {
}
-TestVideoFrameProvider::~TestVideoFrameProvider() {}
+TestMediaStreamVideoRenderer::~TestMediaStreamVideoRenderer() {}
-void TestVideoFrameProvider::Start() {
- DVLOG(1) << "TestVideoFrameProvider::Start";
+void TestMediaStreamVideoRenderer::Start() {
+ DVLOG(1) << "TestMediaStreamVideoRenderer::Start";
DCHECK(task_runner_->BelongsToCurrentThread());
state_ = kStarted;
task_runner_->PostTask(
- FROM_HERE, base::Bind(&TestVideoFrameProvider::GenerateFrame, this));
+ FROM_HERE, base::Bind(&TestMediaStreamVideoRenderer::GenerateFrame,
+ this));
}
-void TestVideoFrameProvider::Stop() {
- DVLOG(1) << "TestVideoFrameProvider::Stop";
+void TestMediaStreamVideoRenderer::Stop() {
+ DVLOG(1) << "TestMediaStreamVideoRenderer::Stop";
DCHECK(task_runner_->BelongsToCurrentThread());
state_ = kStopped;
}
-void TestVideoFrameProvider::Play() {
- DVLOG(1) << "TestVideoFrameProvider::Play";
+void TestMediaStreamVideoRenderer::Resume() {
+ DVLOG(1) << "TestMediaStreamVideoRenderer::Resume";
DCHECK(task_runner_->BelongsToCurrentThread());
if (state_ == kPaused)
state_ = kStarted;
}
-void TestVideoFrameProvider::Pause() {
- DVLOG(1) << "TestVideoFrameProvider::Pause";
+void TestMediaStreamVideoRenderer::Pause() {
+ DVLOG(1) << "TestMediaStreamVideoRenderer::Pause";
DCHECK(task_runner_->BelongsToCurrentThread());
if (state_ == kStarted)
state_ = kPaused;
}
-void TestVideoFrameProvider::GenerateFrame() {
- DVLOG(1) << "TestVideoFrameProvider::GenerateFrame";
+void TestMediaStreamVideoRenderer::GenerateFrame() {
+ DVLOG(1) << "TestMediaStreamVideoRenderer::GenerateFrame";
DCHECK(task_runner_->BelongsToCurrentThread());
if (state_ == kStopped)
return;
@@ -75,7 +76,7 @@ void TestVideoFrameProvider::GenerateFrame() {
current_time_ += frame_duration_;
task_runner_->PostDelayedTask(
- FROM_HERE, base::Bind(&TestVideoFrameProvider::GenerateFrame, this),
+ FROM_HERE, base::Bind(&TestMediaStreamVideoRenderer::GenerateFrame, this),
frame_duration_);
}

Powered by Google App Engine
This is Rietveld 408576698