Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(35)

Unified Diff: apps/launcher.h

Issue 657023008: Add a new field "source" in launchData of chrome.app.runtime.onLaunched() to trace launch source. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: apps/launcher.h
diff --git a/apps/launcher.h b/apps/launcher.h
index eb2d98fb265bb0bd33daa6a89aa9f63343f23c18..b1bcd9e4e095595a99b43f8847df4d6af50faee8 100644
--- a/apps/launcher.h
+++ b/apps/launcher.h
@@ -8,6 +8,8 @@
#include <string>
#include <vector>
+#include "extensions/common/constants.h"
+
class GURL;
class Profile;
@@ -29,7 +31,8 @@ namespace apps {
void LaunchPlatformAppWithCommandLine(Profile* profile,
const extensions::Extension* extension,
const base::CommandLine& command_line,
- const base::FilePath& current_directory);
+ const base::FilePath& current_directory,
+ extensions::AppLaunchSource source);
// Launches the platform app |extension| by issuing an onLaunched event
// with the contents of |file_path| available through the launch data.
@@ -41,6 +44,11 @@ void LaunchPlatformAppWithPath(Profile* profile,
void LaunchPlatformApp(Profile* profile,
benwells 2014/10/21 03:51:58 Can't you just add the source parameter to this fu
cylee1 2014/10/21 13:41:16 Done.
const extensions::Extension* extension);
+// Launches the platform app |extension| with no launch data.
+void LaunchPlatformAppWithSource(Profile* profile,
+ const extensions::Extension* extension,
+ extensions::AppLaunchSource source);
+
// Launches the platform app |extension| with |handler_id| and the contents of
// |file_paths| available through the launch data. |handler_id| corresponds to
// the id of the file_handlers item in the manifest that resulted in a match

Powered by Google App Engine
This is Rietveld 408576698