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

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: Fix browsertests and add a new test for 904 Arc Disabled Created 3 years, 10 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..197dffc24616b5db1022cd064f0b0fb9c15d79c3 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() ? ArcAuthInfoFetchStatus::FAILURE
+ : ArcAuthInfoFetchStatus::SUCCESS,
+ auth_code);
}
} // namespace arc

Powered by Google App Engine
This is Rietveld 408576698