Clean up thread handling in Blimp browser tests.

blimp/engine/browser_tests/engine_browsertest.cc

Index: blimp/engine/browser_tests/engine_browsertest.cc
diff --git a/blimp/engine/browser_tests/engine_browsertest.cc b/blimp/engine/browser_tests/engine_browsertest.cc
index 43e04f190b35240f2739934b061a3e09ae52c5af..0128d9e9eeedb517e4454f85d62e9cfb540dd2bb 100644
--- a/blimp/engine/browser_tests/engine_browsertest.cc
+++ b/blimp/engine/browser_tests/engine_browsertest.cc
@@ -70,7 +70,7 @@ IN_PROC_BROWSER_TEST_F(EngineBrowserTest, LoadUrl) {
               OnTitleChanged(kDummyTabId, url.GetContent()));
   EXPECT_CALL(client_nav_feature_delegate_,
               OnTitleChanged(kDummyTabId, "hello"))
-      .WillOnce(InvokeWithoutArgs(this, &EngineBrowserTest::QuitRunLoop));
+      .WillOnce(InvokeWithoutArgs(this, &EngineBrowserTest::SignalCompletion));
 
   // Skip assigner. Engine info is already available.
   client_session_->ConnectWithAssignment(client::ASSIGNMENT_REQUEST_RESULT_OK,
@@ -81,7 +81,7 @@ IN_PROC_BROWSER_TEST_F(EngineBrowserTest, LoadUrl) {
   client_session_->GetNavigationFeature()->NavigateToUrlText(kDummyTabId,
                                                              url.spec());
 
-  RunUntilQuit();
+  RunUntilCompletion();
 }
 
 }  // namespace