login: Add more logging to container shutdown

In order to better understand the feedback reports, adding logging in key events of arc provisioning process.

BUG=b:162221783
TEST=deploy to dut, disable playstore, re-enable playstore, successfully
finish the authorization flow

Change-Id: I3f5cbfd0cb023c43982e15a5aba724901a4fe3e0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2377229
Commit-Queue: Muhammad Hasan Khan <mhasank@chromium.org>
Commit-Queue: Ryo Hashimoto <hashimoto@chromium.org>
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Reviewed-by: Hidehiko Abe <hidehiko@chromium.org>
Auto-Submit: Muhammad Hasan Khan <mhasank@chromium.org>
Cr-Commit-Position: refs/heads/master@{#804051}

chrome/browser/chromeos/arc/session/arc_session_manager.cc

@@ -531,7 +531,6 @@ void ArcSessionManager::OnSessionStopped(ArcStopReason reason,
     DCHECK_EQ(state_, State::ACTIVE);
     // If ARC is being restarted, here do nothing, and just wait for its
     // next run.
-    VLOG(1) << "ARC session is stopped, but being restarted: " << reason;
     return;
   }
 
@@ -1118,7 +1117,7 @@ void ArcSessionManager::RequestArcDataRemoval() {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   DCHECK(profile_);
   DCHECK(data_remover_);
-  VLOG(1) << "Removing user ARC data.";
+  VLOG(1) << "Scheduling ARC data removal.";
 
   // TODO(hidehiko): DCHECK the previous state. This is called for four cases;
   // 1) Supporting managed user initial disabled case (Please see also

components/arc/session/arc_data_remover.cc

@@ -40,7 +40,7 @@ bool ArcDataRemover::IsScheduledForTesting() const {
 void ArcDataRemover::Run(RunCallback callback) {
   DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
   if (!pref_.GetValue()) {
-    // Data removal is not scheduled.
+    VLOG(1) << "Data removal is not scheduled, skip.";
     std::move(callback).Run(base::nullopt);
     return;
   }

components/arc/session/arc_session_impl.cc

@@ -427,6 +427,8 @@ void ArcSessionImpl::RequestUpgrade(UpgradeParams params) {
   upgrade_requested_ = true;
   upgrade_params_ = std::move(params);
 
+  VLOG(1) << "Upgrade requested. state: " << state_;
+
   switch (state_) {
     case State::NOT_STARTED:
       NOTREACHED();
@@ -434,7 +436,6 @@ void ArcSessionImpl::RequestUpgrade(UpgradeParams params) {
     case State::WAITING_FOR_LCD_DENSITY:
     case State::WAITING_FOR_NUM_CORES:
     case State::STARTING_MINI_INSTANCE:
-      VLOG(2) << "Requested to upgrade a starting ARC mini instance";
       // OnMiniInstanceStarted() will restart a full instance.
       break;
     case State::RUNNING_MINI_INSTANCE: