Remove FullCardRequest's dependency on AutofillClient.

Remove FullCardRequest's dependency on AutofillClient.

BUG=706511
Review-Url: https://codereview.chromium.org/2794613002
Cr-Commit-Position: refs/heads/master@{#461209}
Committed: https://chromium.googlesource.com/chromium/src/+/ddc6c499fc6f05937be269de5c0daf54b0d298fb

[anthonyvd, 2017-03-31 18:51:47]

The CQ bit was checked by anthonyvd@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-31 18:52:07]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[anthonyvd, 2017-03-31 19:00:18]

anthonyvd@chromium.org changed reviewers:
+ mathp@chromium.org

[anthonyvd, 2017-03-31 19:00:19]

Hi Math, can you PTAL? Thanks!

[Mathieu, 2017-03-31 19:04:04]

Thanks for doing this! lgtm

https://codereview.chromium.org/2794613002/diff/20001/components/autofill/cor...
File components/autofill/core/browser/payments/full_card_request.h (right):

https://codereview.chromium.org/2794613002/diff/20001/components/autofill/cor...
components/autofill/core/browser/payments/full_card_request.h:86: // Responsible
for showing the UI that prompts the user for the CVC and/or the
let's update this comment

https://codereview.chromium.org/2794613002/diff/20001/components/autofill/cor...
File components/autofill/core/browser/risk_data_loader.h (right):

https://codereview.chromium.org/2794613002/diff/20001/components/autofill/cor...
components/autofill/core/browser/risk_data_loader.h:16: virtual
~RiskDataLoader() {}
I think this can be protected?

[anthonyvd, 2017-03-31 19:15:06]

The CQ bit was checked by anthonyvd@chromium.org

[anthonyvd, 2017-03-31 19:15:06]

The patchset sent to the CQ was uploaded after l-g-t-m from mathp@chromium.org
Link to the patchset: https://codereview.chromium.org/2794613002/#ps40001
(title: "Address comments.")

[commit-bot: I haz the power, 2017-03-31 19:15:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-31 20:26:35]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490987706067680,
"parent_rev": "7ca06a2f0b0cd9ade0dd6a47f4fa3c6605479af0", "commit_rev":
"ddc6c499fc6f05937be269de5c0daf54b0d298fb"}

[commit-bot: I haz the power, 2017-03-31 20:27:24]

Description was changed from

==========
Remove FullCardRequest's dependency on AutofillClient.

BUG=706511
==========

to

==========
Remove FullCardRequest's dependency on AutofillClient.

BUG=706511

Review-Url: https://codereview.chromium.org/2794613002
Cr-Commit-Position: refs/heads/master@{#461209}
Committed:
https://chromium.googlesource.com/chromium/src/+/ddc6c499fc6f05937be269de5c0d...
==========

[commit-bot: I haz the power, 2017-03-31 20:27:25]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/ddc6c499fc6f05937be269de5c0d...

[anthonyvd, 2017-04-03 15:01:20]

https://codereview.chromium.org/2794613002/diff/20001/components/autofill/cor...
File components/autofill/core/browser/risk_data_loader.h (right):

https://codereview.chromium.org/2794613002/diff/20001/components/autofill/cor...
components/autofill/core/browser/risk_data_loader.h:16: virtual
~RiskDataLoader() {}
On 2017/03/31 at 19:04:03, Mathieu wrote:
> I think this can be protected?

Done.