Implement app launch changes for app runtime extension proposal.

apps/launcher.h

Index: apps/launcher.h
diff --git a/apps/launcher.h b/apps/launcher.h
index fa6ea1582d27dee47fee5d9b51d3a946f83c2931..15f9d35b0f9290142de2bad8831f4d2a8831ce39 100644
--- a/apps/launcher.h
+++ b/apps/launcher.h
@@ -41,6 +41,14 @@ void LaunchPlatformAppWithPath(Profile* profile,
                                const extensions::Extension* extension,
                                const base::FilePath& file_path);
 
+// Launches the platform app |extension| with the specific |action_data|.
+// |file_path| is an optional argument and if present contains any files that

[Daniel Erat, 2016/08/10 22:19:24]

nit: s/files/file/ (unless you meant to make this a vector of FilePaths)

[jdufault, 2016/08/10 22:33:35]

Done.

+// the app should open w.r.t. the given action.
+void LaunchPlatformAppWithAction(Profile* profile,
+                                 const extensions::Extension* extension,
+                                 extensions::ActionData action_data,

[Daniel Erat, 2016/08/10 22:19:24]

should this be a const reference instead? i'm not sure of the type of the code
that's generated from the idl (is it a struct?) and am still struggling with the
details of base::Optional and std::optional.

[jdufault, 2016/08/10 22:33:35]

Done.

+                                 const base::FilePath& file_path);
+
 // Launches the platform app |extension|. |source| tells us how the app is
 // launched.
 void LaunchPlatformApp(Profile* profile,