Commit 438b4435 authored by Fabio Tirelo's avatar Fabio Tirelo Committed by Commit Bot

Revert "[Autofill] Bring back isAutofillEnabled and setAutofillEnabled"

This reverts commit 65103ba6.

Reason for revert: A proper fix was sent to clank downstream by jbudorick@ (https://chrome-internal-review.googlesource.com/c/clank/internal/apps/+/670069), so this is no longer needed.

Original change's description:
> [Autofill] Bring back isAutofillEnabled and setAutofillEnabled
> 
> The functions are still used by Clank downstream and will be removed
> once only the new preferences are used by it.
> 
> TBR=mathp@chromium.org
> 
> Bug: 879384
> Change-Id: Ibee61db92a2bd3cd8b4124b79fc1c0d747d401d1
> Reviewed-on: https://chromium-review.googlesource.com/1198453
> Reviewed-by: Fabio Tirelo <ftirelo@chromium.org>
> Commit-Queue: Fabio Tirelo <ftirelo@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#587912}

TBR=ftirelo@chromium.org

Change-Id: I415c94a269e97637774af17289f7699215ed9a4f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 879384
Reviewed-on: https://chromium-review.googlesource.com/1199564Reviewed-by: default avatarSebastien Seguin-Gagnon <sebsg@chromium.org>
Commit-Queue: Fabio Tirelo <ftirelo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#590502}
parent 0d8d0e26
...@@ -908,15 +908,6 @@ public class PersonalDataManager { ...@@ -908,15 +908,6 @@ public class PersonalDataManager {
return nativeHasCreditCards(mPersonalDataManagerAndroid); return nativeHasCreditCards(mPersonalDataManagerAndroid);
} }
/**
* TODO(crbug.com/879420): Remove this once Clank downstream uses isAutofillProfileEnabled and
* isAutofillCreditCardEnabled.
* @return Whether the Autofill feature is enabled.
*/
public static boolean isAutofillEnabled() {
return isAutofillProfileEnabled() && isAutofillCreditCardEnabled();
}
/** /**
* @return Whether the Autofill feature for Profiles (addresses) is enabled. * @return Whether the Autofill feature for Profiles (addresses) is enabled.
*/ */
...@@ -931,17 +922,6 @@ public class PersonalDataManager { ...@@ -931,17 +922,6 @@ public class PersonalDataManager {
return nativeGetPref(Pref.AUTOFILL_CREDIT_CARD_ENABLED); return nativeGetPref(Pref.AUTOFILL_CREDIT_CARD_ENABLED);
} }
/**
* Enables or disables the Autofill feature.
* TODO(crbug.com/879420): Remove this once Clank downstream uses setAutofillProfileEnabled and
* setAutofillCreditCardEnabled.
* @param enable True to disable Autofill, false otherwise.
*/
public static void setAutofillEnabled(boolean enable) {
setAutofillProfileEnabled(enable);
setAutofillCreditCardEnabled(enable);
}
/** /**
* Enables or disables the Autofill feature for Profiles. * Enables or disables the Autofill feature for Profiles.
* @param enable True to disable profile Autofill, false otherwise. * @param enable True to disable profile Autofill, false otherwise.
......
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