Refactor remote_desktop_browsertest:

remoting/test/auth_browsertest.cc

Index: remoting/test/auth_browsertest.cc
diff --git a/remoting/test/auth_browsertest.cc b/remoting/test/auth_browsertest.cc
index 845a842f6e71b76ea589ad16f8214afcc3242774..51f464dc1b1ffae951a4fd76500bee6854584cd9 100644
--- a/remoting/test/auth_browsertest.cc
+++ b/remoting/test/auth_browsertest.cc
@@ -9,31 +9,14 @@ namespace remoting {
 IN_PROC_BROWSER_TEST_F(RemoteDesktopBrowserTest, MANUAL_Auth) {
   VerifyInternetAccess();
 
-  if (!NoInstall()) {
-    VerifyChromotingLoaded(false);
-    InstallChromotingApp();
-  }
-
-  VerifyChromotingLoaded(true);
+  Install();
 
   LaunchChromotingApp();
 
-  Authorize();
-
-  Authenticate();
-
-  Approve();
-
-  // TODO: Remove this hack by blocking on the appropriate notification.
-  // The browser may still be loading images embedded in the webapp. If we
-  // uinstall it now those load will fail. Navigating away to avoid the load
-  // failures.
-  ui_test_utils::NavigateToURL(browser(), GURL("about:blank"));
+  // Authorize, Authenticate, and Approve.
+  Auth();

[garykac, 2013/08/19 18:02:28]

Perhaps AuthAuthAppApp? ^_^

For Authorize, Authenticate and Approve Application.

(Not a serious suggestion)

 
-  if (!NoCleanup()) {
-    UninstallChromotingApp();
-    VerifyChromotingLoaded(false);
-  }
+  Cleanup();
 }
 
 }  // namespace remoting