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

Side by Side Diff: remoting/host/chromoting_host_unittest.cc

Issue 10829467: [Chromoting] Introducing refcount-based life time management of the message loops in the service (d… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: CR feedback and a unit test for AutoThreadTaskRunner. Created 8 years, 3 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/bind.h" 5 #include "base/bind.h"
6 #include "base/memory/scoped_ptr.h" 6 #include "base/memory/scoped_ptr.h"
7 #include "base/message_loop_proxy.h" 7 #include "base/message_loop_proxy.h"
8 #include "remoting/jingle_glue/mock_objects.h" 8 #include "remoting/jingle_glue/mock_objects.h"
9 #include "remoting/host/audio_capturer.h" 9 #include "remoting/host/audio_capturer.h"
10 #include "remoting/host/chromoting_host_context.h" 10 #include "remoting/host/chromoting_host_context.h"
(...skipping 348 matching lines...) Expand 10 before | Expand all | Expand 10 after
359 if (expect_host_status_change) { 359 if (expect_host_status_change) {
360 EXPECT_CALL(host_status_observer_, OnClientDisconnected(session_jid)) 360 EXPECT_CALL(host_status_observer_, OnClientDisconnected(session_jid))
361 .After(after) 361 .After(after)
362 .RetiresOnSaturation(); 362 .RetiresOnSaturation();
363 } 363 }
364 } 364 }
365 365
366 protected: 366 protected:
367 MessageLoop message_loop_; 367 MessageLoop message_loop_;
368 scoped_refptr<base::MessageLoopProxy> message_loop_proxy_; 368 scoped_refptr<base::MessageLoopProxy> message_loop_proxy_;
369 MockChromotingHostContext context_;
369 MockConnectionToClientEventHandler handler_; 370 MockConnectionToClientEventHandler handler_;
370 MockSignalStrategy signal_strategy_; 371 MockSignalStrategy signal_strategy_;
371 MockEventExecutor* event_executor_; 372 MockEventExecutor* event_executor_;
372 scoped_ptr<DesktopEnvironment> desktop_environment_; 373 scoped_ptr<DesktopEnvironment> desktop_environment_;
373 scoped_ptr<It2MeHostUserInterface> it2me_host_user_interface_; 374 scoped_ptr<It2MeHostUserInterface> it2me_host_user_interface_;
374 scoped_refptr<ChromotingHost> host_; 375 scoped_refptr<ChromotingHost> host_;
375 MockHostStatusObserver host_status_observer_; 376 MockHostStatusObserver host_status_observer_;
376 MockChromotingHostContext context_;
377 protocol::MockSessionManager* session_manager_; 377 protocol::MockSessionManager* session_manager_;
378 MockConnectionToClient* connection1_; 378 MockConnectionToClient* connection1_;
379 scoped_ptr<MockConnectionToClient> owned_connection1_; 379 scoped_ptr<MockConnectionToClient> owned_connection1_;
380 ClientSession* client1_; 380 ClientSession* client1_;
381 std::string session_jid1_; 381 std::string session_jid1_;
382 MockSession* session1_; // Owned by |connection_|. 382 MockSession* session1_; // Owned by |connection_|.
383 SessionConfig session_config1_; 383 SessionConfig session_config1_;
384 MockVideoStub video_stub1_; 384 MockVideoStub video_stub1_;
385 MockClientStub client_stub1_; 385 MockClientStub client_stub1_;
386 MockHostStub host_stub1_; 386 MockHostStub host_stub1_;
(...skipping 242 matching lines...) Expand 10 before | Expand all | Expand 10 after
629 ExpectClientDisconnected(0, true, video_packet_sent, 629 ExpectClientDisconnected(0, true, video_packet_sent,
630 InvokeWithoutArgs(this, &ChromotingHostTest::ShutdownHost)); 630 InvokeWithoutArgs(this, &ChromotingHostTest::ShutdownHost));
631 EXPECT_CALL(host_status_observer_, OnShutdown()); 631 EXPECT_CALL(host_status_observer_, OnShutdown());
632 632
633 host_->Start(); 633 host_->Start();
634 SimulateClientConnection(0, true, false); 634 SimulateClientConnection(0, true, false);
635 message_loop_.Run(); 635 message_loop_.Run();
636 } 636 }
637 637
638 } // namespace remoting 638 } // namespace remoting
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698