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

Unified Diff: remoting/host/video_scheduler_unittest.cc

Issue 11821004: Remove unused VideoFrameCapturer::size_most_recent() method. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: CR feedback Created 7 years, 11 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 | « remoting/host/ipc_video_frame_capturer.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/video_scheduler_unittest.cc
diff --git a/remoting/host/video_scheduler_unittest.cc b/remoting/host/video_scheduler_unittest.cc
index 8335c5c5ed6d28a6baef48cc8e9f40857994ab79..747c8a1de4d308ea7ca867ea2df76688fc70a93f 100644
--- a/remoting/host/video_scheduler_unittest.cc
+++ b/remoting/host/video_scheduler_unittest.cc
@@ -73,7 +73,7 @@ MockVideoEncoder::~MockVideoEncoder() {}
class VideoSchedulerTest : public testing::Test {
public:
- VideoSchedulerTest() : size_(SkISize::Make(0, 0)) {
+ VideoSchedulerTest() {
}
virtual void SetUp() OVERRIDE {
@@ -104,7 +104,6 @@ class VideoSchedulerTest : public testing::Test {
// The following mock objects are owned by VideoScheduler.
MockVideoEncoder* encoder_;
- SkISize size_;
scoped_refptr<CaptureData> data_;
private:
@@ -125,15 +124,12 @@ void VideoSchedulerTest::GenerateOnCaptureCompleted() {
TEST_F(VideoSchedulerTest, StartAndStop) {
Expectation capturer_start = EXPECT_CALL(capturer_, Start(_));
- size_.set(kWidth, kHeight);
- data_ = new CaptureData(NULL, kWidth * CaptureData::kBytesPerPixel, size_);
+ data_ = new CaptureData(NULL, kWidth * CaptureData::kBytesPerPixel,
+ SkISize::Make(kWidth, kHeight));
// Create a RunLoop through which to drive |message_loop_|.
base::RunLoop run_loop;
- EXPECT_CALL(capturer_, size_most_recent())
- .WillRepeatedly(ReturnRef(size_));
-
// First the capturer is called.
Expectation capturer_capture = EXPECT_CALL(capturer_, CaptureFrame())
.After(capturer_start)
« no previous file with comments | « remoting/host/ipc_video_frame_capturer.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698