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

Unified Diff: remoting/client/chromoting_client_runtime_unittest.cc

Issue 2745583008: Refactoring out the chromoting jni runtime class in favor of chromoting client runtime. (Closed)
Patch Set: Unit tests need to link with android. Created 3 years, 9 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/client/chromoting_client_runtime_unittest.cc
diff --git a/remoting/client/chromoting_client_runtime_unittest.cc b/remoting/client/chromoting_client_runtime_unittest.cc
index e56f7482790146e004a3b49a2ed57d278d463cee..09a327b0ba5b0d3674c9d25c5403e96d67f44ffe 100644
--- a/remoting/client/chromoting_client_runtime_unittest.cc
+++ b/remoting/client/chromoting_client_runtime_unittest.cc
@@ -15,14 +15,7 @@ namespace remoting {
// A simple test that starts and stop the runtime. This tests the runtime
// operates properly and all threads and message loops are valid.
TEST(ChromotingClientRuntimeTest, StartAndStop) {
- std::unique_ptr<base::MessageLoopForUI> ui_loop;
- ui_loop.reset(new base::MessageLoopForUI());
-#if defined(OS_IOS)
- ui_loop->Attach();
-#endif
-
- std::unique_ptr<ChromotingClientRuntime> runtime =
- ChromotingClientRuntime::Create(ui_loop.get());
+ ChromotingClientRuntime* runtime = ChromotingClientRuntime::GetInstance();
ASSERT_TRUE(runtime);
EXPECT_TRUE(runtime->network_task_runner().get());
@@ -30,6 +23,7 @@ TEST(ChromotingClientRuntimeTest, StartAndStop) {
EXPECT_TRUE(runtime->display_task_runner().get());
EXPECT_TRUE(runtime->file_task_runner().get());
EXPECT_TRUE(runtime->url_requester().get());
+ EXPECT_TRUE(runtime->GetLogWriter());
}
#endif

Powered by Google App Engine
This is Rietveld 408576698