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

chrome/browser/ui/extensions/app_launch_params.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_EXTENSIONS_APP_LAUNCH_PARAMS_H_
 #define CHROME_BROWSER_UI_EXTENSIONS_APP_LAUNCH_PARAMS_H_
 
 #include <string>
 
 #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 {
 class Extension;
 }
 
 struct AppLaunchParams {
   AppLaunchParams(Profile* profile,
                   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);
 
   ~AppLaunchParams();
 
   // The profile to load the application from.
   Profile* profile;
 
   // The extension to load.
   std::string extension_id;
@@ skipping 15 matching lines @@
   // application.
   base::CommandLine command_line;
 
   // If non-empty, the current directory from which any relative paths on the
   // command line should be expanded from.
   base::FilePath current_directory;
 
   // 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
 // LAUNCH_TYPE_REGULAR to check for a user-configured container.
 AppLaunchParams CreateAppLaunchParamsUserContainer(
     Profile* profile,
     const extensions::Extension* extension,
     WindowOpenDisposition disposition,
     extensions::AppLaunchSource source);
 
 // Helper to create AppLaunchParams using event flags that allows user to
 // override the user-configured container using modifier keys, falling back to
 // extensions::GetLaunchContainer() with no modifiers.
 AppLaunchParams CreateAppLaunchParamsWithEventFlags(
     Profile* profile,
     const extensions::Extension* extension,
     int event_flags,
     extensions::AppLaunchSource source);
 
 #endif  // CHROME_BROWSER_UI_EXTENSIONS_APP_LAUNCH_PARAMS_H_