arc: Fix race conditon when Play Store is started too early.

chrome/browser/ui/app_list/arc/arc_app_utils.h

Index: chrome/browser/ui/app_list/arc/arc_app_utils.h
diff --git a/chrome/browser/ui/app_list/arc/arc_app_utils.h b/chrome/browser/ui/app_list/arc/arc_app_utils.h
index e0911730bbfed73048e44dbf63badcb64068755f..b575af7d9b26fefb559848b50a35b1ab6bc446c2 100644
--- a/chrome/browser/ui/app_list/arc/arc_app_utils.h
+++ b/chrome/browser/ui/app_list/arc/arc_app_utils.h
@@ -43,6 +43,8 @@ bool LaunchPlayStoreWithUrl(const std::string& url);
 
 // Launch an app with given layout and let the system decides how big and where
 // to place it.
+// TODO (khmel) Rename this method to reduce the chance of using another

[hidehiko, 2017/03/16 16:23:19]

Nice comment!

Or, maybe replacing "bool" for layout with enum class, so that implicit
conversion won't be triggered.

enum class Layout {
  PORTRAIT,
  LANDSCAPE,
};

bool LaunchApp(content::BrowserContext* context,
               const std::string& app_id,
               Layout layout,
               int event_flags);

Note: not necessary to address it in this CL, I think.

[khmel, 2017/03/16 16:38:47]

Nice alternative

+// LaunchApp with different signature mistakenly.
 bool LaunchApp(content::BrowserContext* context,
                const std::string& app_id,
                bool landscape_layout,