Pass command line arguments through with --load-and-launch-app.

apps/app_load_service.h

 // Copyright 2013 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 APPS_APP_LOAD_SERVICE_H_
 #define APPS_APP_LOAD_SERVICE_H_
 
 #include <map>
 #include <string>
 
+#include "base/command_line.h"
+#include "base/files/file_path.h"
 #include "components/browser_context_keyed_service/browser_context_keyed_service.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 
 class Profile;
 
 namespace apps {
 
 // Monitors apps being reloaded and performs app specific actions (like launch
 // or restart) on them. Also provides an interface to schedule these actions.
 class AppLoadService : public BrowserContextKeyedService,
                        public content::NotificationObserver {
  public:
+  enum PostReloadActionType {
+    RELOAD_LAUNCH,

[koz (OOO until 15th September), 2013/06/05 08:34:11]

nit: I think it's fine to remove the RELOAD_ prefix here, but up to you. Or if
you're into the whole verbosity thing the prefix could be POST_RELOAD_LAUNCH.
Just that the current names are sort of weird - RELOAD_RESTART has two words
that are alternatives to each other.

[benwells, 2013/06/06 00:22:42]

Done.

+    RELOAD_RESTART,
+    RELOAD_LAUNCH_WITH_COMMAND_LINE,
+  };
+
+  struct PostReloadAction {
+    PostReloadAction();
+
+    PostReloadActionType action_type;
+    CommandLine command_line;
+    base::FilePath current_dir;
+  };
+
   explicit AppLoadService(Profile* profile);
   virtual ~AppLoadService();
 
   // Reload the application with the given id and then send it the OnRestarted
   // event.
   void RestartApplication(const std::string& extension_id);
 
-  // Schedule a launch to happen for the extension with id |extension_id|
-  // when it loads next. This does not cause the extension to be reloaded.
-  void ScheduleLaunchOnLoad(const std::string& extension_id);
+  // Loads (or reloads) the app with |extension_path|, then launches it. Any
+  // command line parameters from |command_line| will be passed along via
+  // launch parameters. Returns true if loading the extension has begun
+  // successfully.
+  bool LoadAndLaunch(const base::FilePath& extension_path,
+                     const CommandLine& command_line,
+                     const base::FilePath& current_dir);
 
   static AppLoadService* Get(Profile* profile);
 
  private:
   // content::NotificationObserver.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
   // Map of extension id to reload action. Absence from the map implies
   // no action.
-  std::map<std::string, int> reload_actions_;
+  std::map<std::string, PostReloadAction> reload_actions_;

[koz (OOO until 15th September), 2013/06/05 08:34:11]

s/reload_actions_/post_reload_actions_/ ?

[benwells, 2013/06/06 00:22:42]

Done.

   content::NotificationRegistrar registrar_;
   Profile* profile_;
 
   DISALLOW_COPY_AND_ASSIGN(AppLoadService);
 };
 
 }  // namespace apps
 
 #endif  // APPS_APP_LOAD_SERVICE_H_