Bug 1747898 Comment 16 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

### Beta/Release Uplift Approval Request
* **User impact if declined**: Credit Card Capture doesn't work on fenix
* **Is this code covered by automated tests?**: Unknown
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: Save:
1. navigate to a web page that requires credit card information
2. fill in the credit card information and then submit the form
3. Check the save credit card prompt shows up and contain the correct information
Update:
1. navigate to a web page that requires credit card information
2. autofill the credit card information previously saved
3. Change one of the fields (name or expiration date) and then submit the form
4. Check the update credit card prompt shows up and contain the correct information
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: There are two part of this patch. The part 1 of this patch has already been verified before. We only restore the change back. The change itself is also only reading a default value when the pref doesn't exist.
The part 2 of this patch doesn't change the logic, it only fixes issue by accessing the correct member variable.
* **String changes made/needed**:
### Beta/Release Uplift Approval Request
* **User impact if declined**: Credit Card Capture doesn't work on fenix
* **Is this code covered by automated tests?**: Unknown. Not sure if mobile team has testcase covers this up.
* **Has the fix been verified in Nightly?**: Yes, with geckoview example app.
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: Save:
1. navigate to a web page that requires credit card information
2. fill in the credit card information and then submit the form
3. Check the save credit card prompt shows up and contain the correct information
Update:
1. navigate to a web page that requires credit card information
2. autofill the credit card information previously saved
3. Change one of the fields (name or expiration date) and then submit the form
4. Check the update credit card prompt shows up and contain the correct information
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: There are two part of this patch. The part 1 of this patch has already been verified before. We only restore the change back. The change itself is also only reading a default value when the pref doesn't exist.
The part 2 of this patch doesn't change the logic, it only fixes issue by accessing the correct member variable.
* **String changes made/needed**:

Back to Bug 1747898 Comment 16