Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5712)

Unified Diff: chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc

Issue 2705213002: Show different error message if ARC is disabled. (Closed)
Patch Set: Rebase Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc
diff --git a/chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc b/chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc
index 62973cf22bb61c07c82731a9746023c71f506fec..9c09c5311f310f87b7eff31a13582c6a753a0c97 100644
--- a/chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc
+++ b/chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc
@@ -182,7 +182,10 @@ void ArcBackgroundAuthCodeFetcher::ReportResult(
const std::string& auth_code,
OptInSilentAuthCode uma_status) {
UpdateSilentAuthCodeUMA(uma_status);
- base::ResetAndReturn(&callback_).Run(auth_code);
+ base::ResetAndReturn(&callback_)
+ .Run(auth_code.empty() ? ArcAuthInfoFetcher::Status::FAILURE
+ : ArcAuthInfoFetcher::Status::SUCCESS,
+ auth_code);
}
} // namespace arc

Powered by Google App Engine
This is Rietveld 408576698