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

Unified Diff: chrome/browser/signin/chrome_proximity_auth_client.cc

Issue 2863533003: [EasyUnlock] Serialize and store BeaconSeeds along as cryptohome key metadata. (Closed)
Patch Set: [EasyUnlock] Serialize and store BeaconSeeds along as cryptohome key metadata. Created 3 years, 7 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/signin/chrome_proximity_auth_client.cc
diff --git a/chrome/browser/signin/chrome_proximity_auth_client.cc b/chrome/browser/signin/chrome_proximity_auth_client.cc
index 94e33cfbe5b9a0bbeedc7f31fb6cbedb8a4fb12d..3002689a996127269fa6a81a38c0bcc87dee92da 100644
--- a/chrome/browser/signin/chrome_proximity_auth_client.cc
+++ b/chrome/browser/signin/chrome_proximity_auth_client.cc
@@ -97,7 +97,14 @@ PrefService* ChromeProximityAuthClient::GetPrefService() {
std::unique_ptr<cryptauth::SecureMessageDelegate>
ChromeProximityAuthClient::CreateSecureMessageDelegate() {
- return GetCryptAuthService()->CreateSecureMessageDelegate();
+// Note: Although CryptAuthService::CreateSecureMessageDelegate() exists, we
+// don't use it here because the CryptAuthService is not available on the
Ryan Hansberry 2017/05/09 02:44:08 nit: clarify that you "don't use it here *(as oppo
Tim Song 2017/05/10 18:23:26 Done.
+// ChromeOS login screen.
+#if defined(OS_CHROMEOS)
+ return base::MakeUnique<chromeos::SecureMessageDelegateChromeOS>();
+#else
+ return nullptr;
+#endif
}
std::unique_ptr<cryptauth::CryptAuthClientFactory>

Powered by Google App Engine
This is Rietveld 408576698