[Payments] Don't suggest profiles with no street address in PR.

[Payments] Don't suggest profiles with no street address in PR.

BUG=658346
Committed: https://crrev.com/d6efc8643cafe5e3825ae422b4a12f2197633cf7
Cr-Commit-Position: refs/heads/master@{#426890}

[sebsg, 2016-10-21 18:32:20]

Patchset #1 (id:1) has been deleted

[sebsg, 2016-10-21 18:32:46]

sebsg@chromium.org changed reviewers:
+ rouslan@chromium.org

[sebsg, 2016-10-21 18:32:47]

Hi Rouslan, could you please take a look? Thanks!

[sebsg, 2016-10-21 18:35:29]

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

[commit-bot: I haz the power, 2016-10-21 18:36:12]

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

[please use gerrit instead, 2016-10-21 18:36:25]

lgtm % nit

https://codereview.chromium.org/2438343002/diff/20001/chrome/android/javatest...
File
chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestDynamicShippingMultipleAddressesTest.java
(right):

https://codereview.chromium.org/2438343002/diff/20001/chrome/android/javatest...
chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestDynamicShippingMultipleAddressesTest.java:31:
"Marge Simpson", "Acme Inc.", "123 Main", "California", "Los Angeles", "",
Why change the names? B<H<L<Mag<Mar were sorted alphabetically.

[sebsg, 2016-10-21 19:14:07]

Patchset #2 (id:40001) has been deleted

[sebsg, 2016-10-21 19:25:08]

Thanks, put the names back in order and added a test.

https://codereview.chromium.org/2438343002/diff/20001/chrome/android/javatest...
File
chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestDynamicShippingMultipleAddressesTest.java
(right):

https://codereview.chromium.org/2438343002/diff/20001/chrome/android/javatest...
chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestDynamicShippingMultipleAddressesTest.java:31:
"Marge Simpson", "Acme Inc.", "123 Main", "California", "Los Angeles", "",
On 2016/10/21 18:36:25, rouslan wrote:
> Why change the names? B<H<L<Mag<Mar were sorted alphabetically.

Right, I should change the test results instead.

[sebsg, 2016-10-21 19:25:22]

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

[commit-bot: I haz the power, 2016-10-21 19:26:18]

Dry run: 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, 2016-10-21 20:53:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-21 20:53:16]

Dry run: This issue passed the CQ dry run.

[sebsg, 2016-10-21 21:02:54]

The CQ bit was checked by sebsg@chromium.org

[sebsg, 2016-10-21 21:02:54]

The patchset sent to the CQ was uploaded after l-g-t-m from rouslan@chromium.org
Link to the patchset: https://codereview.chromium.org/2438343002/#ps60001
(title: "Added test")

[commit-bot: I haz the power, 2016-10-21 21:03:45]

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, 2016-10-21 21:20:37]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-10-21 21:22:23]

Description was changed from

==========
[Payments] Don't suggest profiles with no street address in PR.

BUG=658346
==========

to

==========
[Payments] Don't suggest profiles with no street address in PR.

BUG=658346

Committed: https://crrev.com/d6efc8643cafe5e3825ae422b4a12f2197633cf7
Cr-Commit-Position: refs/heads/master@{#426890}
==========

[commit-bot: I haz the power, 2016-10-21 21:22:24]

Patchset 2 (id:??) landed as
https://crrev.com/d6efc8643cafe5e3825ae422b4a12f2197633cf7
Cr-Commit-Position: refs/heads/master@{#426890}