Add ARC++ specific fields to launch data for specific apps.

chrome/browser/ui/extensions/app_launch_params.h

Index: chrome/browser/ui/extensions/app_launch_params.h
diff --git a/chrome/browser/ui/extensions/app_launch_params.h b/chrome/browser/ui/extensions/app_launch_params.h
index 0aeabbabfce88c1187783de5646de0fd644564d6..db5518467801b81608af0609662ba336c506ab7f 100644
--- a/chrome/browser/ui/extensions/app_launch_params.h
+++ b/chrome/browser/ui/extensions/app_launch_params.h
@@ -10,11 +10,14 @@
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "chrome/common/extensions/extension_constants.h"
+#include "extensions/common/api/app_runtime.h"
 #include "extensions/common/constants.h"
 #include "ui/base/window_open_disposition.h"
 #include "ui/gfx/geometry/rect.h"
 #include "url/gurl.h"
 
+using extensions::api::app_runtime::PlayStoreStatus;
+
 class Profile;
 
 namespace extensions {
@@ -26,7 +29,8 @@ struct AppLaunchParams {
                   const extensions::Extension* extension,
                   extensions::LaunchContainer container,
                   WindowOpenDisposition disposition,
-                  extensions::AppLaunchSource source);
+                  extensions::AppLaunchSource source,
+                  bool set_playstore_status = false);

[sky, 2016/08/24 20:34:23]

Document what set_playstore_status means.

 
   AppLaunchParams(const AppLaunchParams& other);
 
@@ -62,6 +66,9 @@ struct AppLaunchParams {
   // Record where the app is launched from for tracking purpose.
   // Different app may have their own enumeration of sources.
   extensions::AppLaunchSource source;
+
+  // Status of ARC++ on this device.
+  PlayStoreStatus play_store_status;

[sky, 2016/08/24 16:20:39]

Isn't arc chromeos specific? In other words, shouldn't all the arc structs be
behind #if defined(OS_CHROMEOS)?

[rkc, 2016/08/24 20:02:36]

Whoops - yep. Put all of that code behind #ifdefs and compiled on multiple
platforms to confirm everything works.

I've left this field and other related code outside ifdefs since (a) we can't
not have this field in the IDL (we can't have platform specific
structures/enums) and (b) putting this behind an ifdef would introduce ifdefs
all over the code, which would result in much ugliness.

[sky, 2016/08/24 20:34:23]

As long as this doesn't result in pulling in any arc specific headers, ok.

 };
 
 // Helper to create AppLaunchParams using extensions::GetLaunchContainer with