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

Unified Diff: remoting/host/chromoting_host_unittest.cc

Issue 8495024: Access ChromotingHost::clients_ only on network thread. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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: remoting/host/chromoting_host_unittest.cc
diff --git a/remoting/host/chromoting_host_unittest.cc b/remoting/host/chromoting_host_unittest.cc
index 2889a3ef5cef09317d60b070b104afb255a3851f..2838437eeb37f06689c9041a2b0328f7ea9073d8 100644
--- a/remoting/host/chromoting_host_unittest.cc
+++ b/remoting/host/chromoting_host_unittest.cc
@@ -398,8 +398,10 @@ TEST_F(ChromotingHostTest, CurtainModeFailSecond) {
InSequence s;
EXPECT_CALL(*curtain_, EnableCurtainMode(true))
.WillOnce(QuitMainMessageLoop(&message_loop_));
- EXPECT_CALL(*disconnect_window_, Show(_, _))
- .Times(0);
+ EXPECT_CALL(*local_input_monitor_, Start(_))
+ .Times(1);
+ EXPECT_CALL(*disconnect_window_, Show(_, "user@domain"))
+ .Times(1);
EXPECT_CALL(video_stub_, ProcessVideoPacket(_, _))
.WillOnce(DoAll(
InvokeWithoutArgs(

Powered by Google App Engine
This is Rietveld 408576698