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

chrome/browser/chromeos/arc/arc_session_manager.cc

 // Copyright 2016 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.
 
 #include "chrome/browser/chromeos/arc/arc_session_manager.h"
 
 #include <utility>
 
 #include "ash/common/shelf/shelf_delegate.h"
 #include "ash/common/wm_shell.h"
@@ skipping 240 matching lines @@
     //   kiosk app and device is not needed for opt-in;
     // * When ARC is managed and all OptIn preferences are managed too, because
     //   the whole OptIn flow should happen as seamless as possible for the
     //   user.
     const bool suppress_play_store_app =
         IsArcOptInVerificationDisabled() || IsArcKioskMode() ||
         (IsArcPlayStoreEnabledPreferenceManagedForProfile(profile_) &&
          AreArcAllOptInPreferencesManagedForProfile(profile_));
     if (!suppress_play_store_app) {
       playstore_launcher_.reset(
-          new ArcAppLauncher(profile_, kPlayStoreAppId, true));
+          new ArcAppLauncher(profile_, kPlayStoreAppId, true, false));

[xiyuan, 2017/03/15 23:01:47]

nit: Annotate the bool args since there are multiple ones now.

     }
 
     for (auto& observer : observer_list_)
       observer.OnArcInitialStart();
     return;
   }
 
   ArcSupportHost::Error error;
   VLOG(1) << "ARC provisioning failed: " << result << ".";
   switch (result) {
@@ skipping 702 matching lines @@
 
 #undef MAP_STATE
 
   // Some compilers report an error even if all values of an enum-class are
   // covered exhaustively in a switch statement.
   NOTREACHED() << "Invalid value " << static_cast<int>(state);
   return os;
 }
 
 }  // namespace arc