Pass command line arguments onto platform apps which provide the right intent.

chrome/browser/extensions/platform_app_browsertest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/command_line.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/automation/automation_util.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_browsertest.h"
@@ skipping 56 matching lines @@
     ExtensionService* service = browser()->profile()->GetExtensionService();
     const Extension* extension = service->GetExtensionById(
         last_loaded_extension_id_, false);
     EXPECT_TRUE(extension);
 
     Browser::OpenApplication(
         browser()->profile(),
         extension,
         extension_misc::LAUNCH_NONE,
         GURL(),
-        NEW_WINDOW);
+        NEW_WINDOW,
+        NULL);
 
     app_loaded_observer.Wait();
   }
 
   // Gets the WebContents associated with the first shell window that is found
   // (most tests only deal with one platform app window, so this is good
   // enough).
   WebContents* GetFirstShellWindowWebContents() {
     ShellWindowRegistry* app_registry =
         ShellWindowRegistry::Get(browser()->profile());
@@ skipping 98 matching lines @@
       set_cookie_url,
       browser()->GetWebContentsAt(0),
       &cookie_size,
       &cookie_value);
   ASSERT_EQ("testCookie=1", cookie_value);
 
   // Let the platform app request the same URL, and make sure that it doesn't
   // see the cookie.
   ASSERT_TRUE(RunPlatformAppTest("platform_apps/isolation")) << message_;
 }
+
+// Tests that command line parameters get passed through to platform apps
+// via launchData correctly when launching with a file.
+IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, LaunchWithFile) {
+  CommandLine* command_line = CommandLine::ForCurrentProcess();
+  FilePath test_doc(test_data_dir_.AppendASCII(
+      "platform_apps/test_files/test.txt"));
+  // The command line already has arguments on it ("about:blank"). We want to

[Mihai Parparita -not on Chrome, 2012/05/15 00:39:28]

What is adding the about:blank?

[benwells, 2012/05/18 03:36:02]

Its added here
http://code.google.com/p/chromium/source/search?q=InProcessBrowserTest%3A%3AP...

I added a comment mentioning this.

+  // clear any existing arguments and replace with our test file.
+  CommandLine::StringVector args = command_line->GetArgs();
+  CommandLine::StringVector argv = command_line->argv();
+  for (size_t i=0; i<args.size(); i++)
+    argv.pop_back();

[benwells, 2012/05/14 23:50:33]

The removal of existing command line args should be extracted into a common
function.

[benwells, 2012/05/18 03:36:02]

Done.

+  argv.push_back(test_doc.value());
+  command_line->InitFromArgv(argv);
+  ASSERT_TRUE(RunPlatformAppTest("platform_apps/launch_file"))
+      << message_;
+}
+
+// Tests that no launch data is sent through if the platform app provides
+// an intent with the wrong action.
+IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, LaunchWithWrongIntent) {
+  CommandLine* command_line = CommandLine::ForCurrentProcess();
+  FilePath test_doc(test_data_dir_.AppendASCII(
+      "platform_apps/test_files/test.txt"));
+  // The command line already has arguments on it ("about:blank"). We want to
+  // clear any existing arguments and replace with our test file.
+  CommandLine::StringVector args = command_line->GetArgs();
+  CommandLine::StringVector argv = command_line->argv();
+  for (size_t i=0; i<args.size(); i++)
+    argv.pop_back();
+  argv.push_back(test_doc.value());
+  command_line->InitFromArgv(argv);
+  ASSERT_TRUE(RunPlatformAppTest("platform_apps/launch_wrong_intent"))
+      << message_;
+}
+
+// Tests that no launch data is sent through if the file is of the wrong MIME
+// type.
+IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, LaunchWithWrongType) {
+  CommandLine* command_line = CommandLine::ForCurrentProcess();
+  FilePath test_doc(test_data_dir_.AppendASCII(
+      "platform_apps/test_files/test.txt"));
+  // The command line already has arguments on it ("about:blank"). We want to
+  // clear any existing arguments and replace with our test file.
+  CommandLine::StringVector args = command_line->GetArgs();
+  CommandLine::StringVector argv = command_line->argv();
+  for (size_t i=0; i<args.size(); i++)
+    argv.pop_back();
+  argv.push_back(test_doc.value());
+  command_line->InitFromArgv(argv);
+  ASSERT_TRUE(RunPlatformAppTest("platform_apps/launch_wrong_type"))
+      << message_;
+}
+
+// Tests that no launch data is sent through if the platform app does not
+// provide an intent.
+IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, LaunchWithNoIntent) {
+  CommandLine* command_line = CommandLine::ForCurrentProcess();
+  FilePath test_doc(test_data_dir_.AppendASCII(
+      "platform_apps/test_files/test.txt"));
+  // The command line already has arguments on it ("about:blank"). We want to
+  // clear any existing arguments and replace with our test file.
+  CommandLine::StringVector args = command_line->GetArgs();
+  CommandLine::StringVector argv = command_line->argv();
+  for (size_t i=0; i<args.size(); i++)
+    argv.pop_back();
+  argv.push_back(test_doc.value());
+  command_line->InitFromArgv(argv);
+  ASSERT_TRUE(RunPlatformAppTest("platform_apps/launch_no_intent"))
+      << message_;
+}
+
+// Tests that no launch data is sent through if the file MIME type cannot
+// be read.
+IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, LaunchNoType) {
+  CommandLine* command_line = CommandLine::ForCurrentProcess();
+  FilePath test_doc(test_data_dir_.AppendASCII(
+      "platform_apps/test_files/test.unknownextension"));
+  // The command line already has arguments on it ("about:blank"). We want to
+  // clear any existing arguments and replace with our test file.
+  CommandLine::StringVector args = command_line->GetArgs();
+  CommandLine::StringVector argv = command_line->argv();
+  for (size_t i=0; i<args.size(); i++)
+    argv.pop_back();
+  argv.push_back(test_doc.value());
+  command_line->InitFromArgv(argv);
+  ASSERT_TRUE(RunPlatformAppTest("platform_apps/launch_no_type"))
+      << message_;
+}
+
+// Tests that no launch data is sent through if there are no arguments passed
+// on the command line
+IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, LaunchWithNothing) {
+  CommandLine* command_line = CommandLine::ForCurrentProcess();
+  // The command line already has arguments on it ("about:blank"). We want to
+  // clear all of these.
+  CommandLine::StringVector args = command_line->GetArgs();
+  CommandLine::StringVector argv = command_line->argv();
+  for (size_t i=0; i<args.size(); i++)
+    argv.pop_back();
+  command_line->InitFromArgv(argv);
+  ASSERT_TRUE(RunPlatformAppTest("platform_apps/launch_nothing"))
+      << message_;
+}