Include addresses with the getdetailsforsavecard Payments RPC. |
||||||||
Issue descriptionThese addresses will be used by Payments to determine the correct legal documents to show the user and to verify that the addresses are valid for their purposes (and thus won't fail on any subsequent save).
,
Sep 28 2016
,
Sep 28 2016
,
Sep 29 2016
Your change meets the bar and is auto-approved for M54 (branch: 2840)
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/be023346b6939f20fafbed8612f1bbf4bdf6ca52 commit be023346b6939f20fafbed8612f1bbf4bdf6ca52 Author: Justin Donnelly <jdonnelly@chromium.org> Date: Thu Sep 29 17:02:23 2016 Include addresses with the getdetailsforsavecard Payments RPC. These addresses will be used by Payments to determine the correct legal documents to show the user and to verify that the addresses are valid for their purposes (and thus won't fail on any subsequent save). Names and phone numbers are removed from the addresses before uploading since they are not useful for these purposes. BUG=535784 Committed: https://crrev.com/da7eb69a380d53eec42cc4e620e55b78e5d6fa82 Review-Url: https://codereview.chromium.org/2349033002 Cr-Original-Commit-Position: refs/heads/master@{#420409} Cr-Commit-Position: refs/heads/master@{#420893} (cherry picked from commit 005d6cd78d14cb1d68a92a0d8dd9a98595b54d80) Review URL: https://codereview.chromium.org/2376323003 . Cr-Commit-Position: refs/branch-heads/2840@{#580} Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607} [modify] https://crrev.com/be023346b6939f20fafbed8612f1bbf4bdf6ca52/components/autofill/content/browser/payments/payments_client_unittest.cc [modify] https://crrev.com/be023346b6939f20fafbed8612f1bbf4bdf6ca52/components/autofill/core/browser/autofill_manager.cc [modify] https://crrev.com/be023346b6939f20fafbed8612f1bbf4bdf6ca52/components/autofill/core/browser/autofill_manager_unittest.cc [modify] https://crrev.com/be023346b6939f20fafbed8612f1bbf4bdf6ca52/components/autofill/core/browser/payments/payments_client.cc [modify] https://crrev.com/be023346b6939f20fafbed8612f1bbf4bdf6ca52/components/autofill/core/browser/payments/payments_client.h
,
Sep 29 2016
,
Sep 29 2016
If there is no pending work, please remove Merge-Approved-54 label.
,
Sep 30 2016
,
Jun 27 2017
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by jdonnelly@chromium.org
, Sep 28 2016