Closed Bug 1399382 Opened 3 years ago Closed 3 years ago

[Form Autofill] Add a pref to hide credit card autofill feature


(Toolkit :: Form Manager, enhancement, P1)




Tracking Status
firefox-esr52 --- unaffected
firefox55 --- unaffected
firefox56 --- unaffected
firefox57 --- fixed


(Reporter: lchang, Assigned: scottwu)


(Blocks 1 open bug)


(Whiteboard: [form autofill:MVP])


(1 file)

Before credit card autofill is ready to ship, we should have a pref to hide the credit card feature and only show it on nightly for testing.
Assignee: nobody → scwwu
Comment on attachment 8907523 [details]
Bug 1399382 - Add a pref to hide credit card autofill feature,
Attachment #8907523 - Flags: review+
It occurs to me that we should disable credit-card-related heuristics as well for better performance. I'm thinking we can strip those fields here [1].

We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 7aaeb1ef05b5 -d 9f1636ed17b6: rebasing 420007:7aaeb1ef05b5 "Bug 1399382 - Add a pref to hide credit card autofill feature, r=lchang" (tip)
merging browser/app/profile/firefox.js
merging browser/extensions/formautofill/FormAutofillPreferences.jsm
merging browser/extensions/formautofill/test/browser/browser_privacyPreferences.js
warning: conflicts while merging browser/extensions/formautofill/FormAutofillPreferences.jsm! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Priority: -- → P1
Pushed by
Add a pref to hide credit card autofill feature, r=lchang
Closed: 3 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Depends on: 1400395
Attachment #8907523 - Flags: review?(MattN+bmo)
Do we need this pref on 56? I'm double-checking since I believe we are shipping some part of autofill in 56.
Flags: needinfo?(scwwu)
Address autofill is shipping on 56, but credit card autofill is not, so we don't need this patch for 56. Thanks Liz.
Flags: needinfo?(scwwu)
Depends on: 1400147
You need to log in before you can comment on or make changes to this bug.