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

Unified Diff: content/browser/renderer_host/media/video_capture_host_unittest.cc

Issue 2393483002: Revert of VideoCapture: migrate VideoCapture renderer-->host messages to mojo, part 1 (Closed)
Patch Set: Created 4 years, 2 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 | « content/browser/renderer_host/media/video_capture_host.cc ('k') | content/common/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/media/video_capture_host_unittest.cc
diff --git a/content/browser/renderer_host/media/video_capture_host_unittest.cc b/content/browser/renderer_host/media/video_capture_host_unittest.cc
index d9c933599cbcf429b5c8308f22a76ad901081a53..2b98a1175e9519f14ba285c24b0a469a49f681ec 100644
--- a/content/browser/renderer_host/media/video_capture_host_unittest.cc
+++ b/content/browser/renderer_host/media/video_capture_host_unittest.cc
@@ -67,8 +67,8 @@
// Define to use a real video capture device.
// #define TEST_REAL_CAPTURE_DEVICE
-// Simple class used for dumping video to a file. This can be used for verifying
-// the output.
+// Simple class used for dumping video to a file. This can be used for
+// verifying the output.
class DumpVideo {
public:
DumpVideo() {}
@@ -291,7 +291,7 @@
// Verifies and removes the expectations on host_ and
// returns true iff successful.
Mock::VerifyAndClearExpectations(host_.get());
- EXPECT_TRUE(host_->controllers_.empty());
+ EXPECT_EQ(0u, host_->entries_.size());
CloseSession();
@@ -402,7 +402,7 @@
params.requested_format = media::VideoCaptureFormat(
gfx::Size(352, 288), 30, media::PIXEL_FORMAT_I420);
host_->OnStartCapture(kDeviceId, opened_session_id_, params);
- host_->StopCapture(kDeviceId);
+ host_->OnStopCapture(kDeviceId);
run_loop.RunUntilIdle();
WaitForVideoDeviceThread();
}
@@ -433,7 +433,7 @@
OnStateChanged(kDeviceId, VIDEO_CAPTURE_STATE_STOPPED))
.WillOnce(ExitMessageLoop(task_runner_, run_loop.QuitClosure()));
- host_->StopCapture(kDeviceId);
+ host_->OnStopCapture(kDeviceId);
host_->SetReturnReceivedDibs(true);
host_->ReturnReceivedDibs(kDeviceId);
@@ -441,7 +441,7 @@
host_->SetReturnReceivedDibs(false);
// Expect the VideoCaptureDevice has been stopped
- EXPECT_TRUE(host_->controllers_.empty());
+ EXPECT_EQ(0u, host_->entries_.size());
}
void NotifyPacketReady() {
« no previous file with comments | « content/browser/renderer_host/media/video_capture_host.cc ('k') | content/common/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698