Commit 4bd395e2 authored by mhasank's avatar mhasank Committed by Commit Bot

arc: Added a comment to explain why we shutdown ARC session

BUG=b/149874690
TEST=None

Change-Id: I8390efa1df473a1c3a79726c01b75498aeaab989
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087958
Commit-Queue: Muhammad Hasan Khan <mhasank@chromium.org>
Reviewed-by: default avatarYury Khmel <khmel@chromium.org>
Reviewed-by: default avatarHidehiko Abe <hidehiko@chromium.org>
Auto-Submit: Muhammad Hasan Khan <mhasank@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748866}
parent ed8831c2
......@@ -461,6 +461,10 @@ void ArcSessionManager::OnProvisioningFinished(ProvisioningResult result) {
break;
}
// When ARC provisioning fails due to Chrome failing to talk to server, we
// don't need to keep the ARC session running as the logs necessary to
// investigate are already present. ARC session will not provide any useful
// context.
if (result == ProvisioningResult::ARC_STOPPED ||
result == ProvisioningResult::CHROME_SERVER_COMMUNICATION_ERROR) {
if (profile_->GetPrefs()->HasPrefPath(prefs::kArcSignedIn))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment