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

Unified Diff: remoting/host/host_status_logger_unittest.cc

Issue 1113493005: [remoting] Replace MessageLoopProxy usage with ThreadTaskRunnerHandle (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Ready for review with finetune changes Created 5 years, 8 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: remoting/host/host_status_logger_unittest.cc
diff --git a/remoting/host/host_status_logger_unittest.cc b/remoting/host/host_status_logger_unittest.cc
index 0ce904ae3f3e25cb4df808ac2069bd6077dadc57..7cbbd390d9e8c1ab555fbe9221e5e87c47070303 100644
--- a/remoting/host/host_status_logger_unittest.cc
+++ b/remoting/host/host_status_logger_unittest.cc
@@ -5,7 +5,8 @@
#include "remoting/host/host_status_logger.h"
#include "base/message_loop/message_loop.h"
-#include "base/message_loop/message_loop_proxy.h"
+#include "base/single_thread_task_runner.h"
+#include "base/thread_task_runner_handle.h"
#include "remoting/host/fake_host_status_monitor.h"
#include "remoting/signaling/mock_signal_strategy.h"
#include "testing/gmock/include/gmock/gmock.h"
@@ -126,7 +127,7 @@ class HostStatusLoggerTest : public testing::Test {
public:
HostStatusLoggerTest() {}
void SetUp() override {
- message_loop_proxy_ = base::MessageLoopProxy::current();
+ message_loop_proxy_ = base::ThreadTaskRunnerHandle::Get();
EXPECT_CALL(signal_strategy_, AddListener(_));
host_status_logger_.reset(
new HostStatusLogger(host_status_monitor_.AsWeakPtr(),
@@ -138,7 +139,7 @@ class HostStatusLoggerTest : public testing::Test {
protected:
base::MessageLoop message_loop_;
- scoped_refptr<base::MessageLoopProxy> message_loop_proxy_;
+ scoped_refptr<base::SingleThreadTaskRunner> message_loop_proxy_;
Sergey Ulanov 2015/05/04 20:49:58 This is not used anywhere and can be removed.
anujsharma 2015/05/05 10:59:04 Done.
MockSignalStrategy signal_strategy_;
scoped_ptr<HostStatusLogger> host_status_logger_;
FakeHostStatusMonitor host_status_monitor_;

Powered by Google App Engine
This is Rietveld 408576698