"Redirecting URLs to Packaged Apps" implementation: revised

chrome/browser/apps/app_browsertest_util.h

Index: chrome/browser/apps/app_browsertest_util.h
diff --git a/chrome/browser/apps/app_browsertest_util.h b/chrome/browser/apps/app_browsertest_util.h
index aa4efcd158f2730b4615153a0125c404dbc81060..dd91d8775f431c77b5defaa7f6e8f332a08321a5 100644
--- a/chrome/browser/apps/app_browsertest_util.h
+++ b/chrome/browser/apps/app_browsertest_util.h
@@ -29,6 +29,9 @@ class PlatformAppBrowserTest : public ExtensionApiTest {
   // until it is launched.
   const Extension* LoadAndLaunchPlatformApp(const char* name);
 
+  // Installs the app named |name| out of the platform_apps subdirectory.
+  const Extension* InstallPlatformApp(const char* name);
+
   // Installs and runs the app named |name| out of the platform_apps
   // subdirectory. Waits until it is launched.
   const Extension* InstallAndLaunchPlatformApp(const char* name);
@@ -87,6 +90,22 @@ class ExperimentalPlatformAppBrowserTest : public PlatformAppBrowserTest {
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE;
 };
 
+class PlatformAppUrlRedirectorBrowserTest : public PlatformAppBrowserTest {

[benwells, 2013/09/09 05:48:39]

Is there a reason this class is in this module? It would normally go in the file
that defines the tests, unless it is used in multiple test files.

[sergeygs, 2013/09/09 09:55:36]

No real reason, it just seemed to make sense. Moved. 

+ public:
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE;
+
+ protected:
+  void TestNavigationInTab(const char* launcher_page_path,
+                           const char* handler_path,
+                           const char* handler_ack,
+                           size_t expected_shell_window_count);
+  void TestNavigationInApp(const char* launcher_path,
+                           const char* launcher_ack,
+                           const char* handler_path,
+                           const char* handler_ack,
+                           size_t expected_shell_window_count);
+};
+
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_APPS_APP_BROWSERTEST_UTIL_H_