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

chrome/browser/ui/browser.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_BROWSER_H_
 #define CHROME_BROWSER_UI_BROWSER_H_
 #pragma once
 
 #include <map>
 #include <set>
@@ skipping 38 matching lines @@
 #include "content/public/common/page_transition_types.h"
 #include "content/public/common/page_zoom.h"
 #include "ui/base/ui_base_types.h"
 #include "ui/gfx/rect.h"
 
 class BrowserContentSettingBubbleModelDelegate;
 class BrowserSyncedWindowDelegate;
 class BrowserToolbarModelDelegate;
 class BrowserTabRestoreServiceDelegate;
 class BrowserWindow;
+class CommandLine;
 class Extension;
 class ExtensionWindowController;
 class FindBarController;
 class FullscreenController;
 class InstantController;
 class InstantUnloadHandler;
 class PrefService;
 class Profile;
 class SkBitmap;
 class StatusBubble;
@@ skipping 211 matching lines @@
   // Opens a new window with the tabs from |profile|'s TabRestoreService.
   static void OpenWindowWithRestoredTabs(Profile* profile);
 
   // Opens the specified URL in a new browser window in an incognito session.
   // If there is already an existing active incognito session for the specified
   // |profile|, that session is re-used.
   static void OpenURLOffTheRecord(Profile* profile, const GURL& url);
 
   // Open |extension| in |container|, using |disposition| if container type is
   // TAB. Returns the WebContents* that was created or NULL. If non-empty,
-  // |override_url| is used in place of the app launch url.
+  // |override_url| is used in place of the app launch url. Pass relevant
+  // information in |command_line| onto platform app as launch data.
+  // |command_line| can be NULL, indicating there is no launch data to pass on.
   static content::WebContents* OpenApplication(

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

This can be in a follow-up cleanup CL, but OpenApplication now seems to have too
many parameters, only some combinations of which are valid (e.g. platform apps
look at command_line but ignore container, override_url and disposition).

Time for a LaunchData struct that can be created for platform app vs.
hosted/packaged app launches?

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

Agreed. Have added TODO.

       Profile* profile,
       const Extension* extension,
       extension_misc::LaunchContainer container,
       const GURL& override_url,
-      WindowOpenDisposition disposition);
+      WindowOpenDisposition disposition,
+      const CommandLine* command_line);
 
 #if defined(USE_ASH)
   // Opens |url| in a new application panel window for the specified url.
   static content::WebContents* OpenApplicationPanel(
       Profile* profile,
       const Extension* extension,
       const GURL& url);
 #endif
 
   // Opens a new application window for the specified url. If |as_panel|
@@ skipping 1176 matching lines @@
   bool window_has_shown_;
 
   // Currently open color chooser. Non-NULL after OpenColorChooser is called and
   // before DidEndColorChooser is called.
   scoped_ptr<content::ColorChooser> color_chooser_;
 
   DISALLOW_COPY_AND_ASSIGN(Browser);
 };
 
 #endif  // CHROME_BROWSER_UI_BROWSER_H_