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

Unified Diff: chrome/browser/autofill/android/personal_data_manager_android.cc

Issue 950003002: Show Autofill/Wallet options only when signed in. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: pdm Created 5 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/autofill/android/personal_data_manager_android.cc
diff --git a/chrome/browser/autofill/android/personal_data_manager_android.cc b/chrome/browser/autofill/android/personal_data_manager_android.cc
index 9059f5db2b82cb69ea9392169a6ae7a653d9a63d..68e6ac1d9117ec4fc865692e3d81e8ba761cfe63 100644
--- a/chrome/browser/autofill/android/personal_data_manager_android.cc
+++ b/chrome/browser/autofill/android/personal_data_manager_android.cc
@@ -342,7 +342,13 @@ static jboolean IsAutofillManaged(JNIEnv* env, jclass clazz) {
// Returns whether the Wallet import feature is available.
static jboolean IsWalletImportFeatureAvailable(JNIEnv* env, jclass clazz) {
- return false;
+ // TODO(estade): what to do in the IsManaged case?
+ ProfileSyncService* service =
+ ProfileSyncServiceFactory::GetInstance()->GetForProfile(GetProfile());
+ PersonalDataManager* pdm = PersonalDataManagerFactory::GetForProfile(
+ GetProfile());
+ return service->IsSyncEnabledAndLoggedIn() &&
+ pdm->IsExperimentalWalletIntegrationEnabled();
}
// Returns whether the Wallet import feature is enabled.

Powered by Google App Engine
This is Rietveld 408576698