[Web Payments] Implement placeholders in the shipping options row

[Web Payments] Implement placeholders in the shipping options row

BUG=710977
Review-Url: https://codereview.chromium.org/2838193004
Cr-Commit-Position: refs/heads/master@{#467385}
Committed: https://chromium.googlesource.com/chromium/src/+/55cde5fb732692857b3535076e5ac9e08188033e

[anthonyvd, 2017-04-26 15:05:48]

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

[commit-bot: I haz the power, 2017-04-26 15:06:06]

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, 2017-04-26 16:06:24]

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

[commit-bot: I haz the power, 2017-04-26 16:06:25]

Dry run: This issue passed the CQ dry run.

[anthonyvd, 2017-04-26 17:02:15]

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

[commit-bot: I haz the power, 2017-04-26 17:02:56]

Dry run: CQ is trying da patch.

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

[anthonyvd, 2017-04-26 17:07:22]

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

[anthonyvd, 2017-04-26 17:07:23]

Hi Math, PTAL! :)

This change also refactors the row creation code that was getting pretty
unmanageable. Hopefully this approach is saner.

Thanks!

[Mathieu, 2017-04-26 17:29:09]

bellissimo! (or, lgtm)

[commit-bot: I haz the power, 2017-04-26 17:34:22]

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

[commit-bot: I haz the power, 2017-04-26 17:34:23]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[anthonyvd, 2017-04-26 17:36:18]

The CQ bit was checked by anthonyvd@chromium.org

[anthonyvd, 2017-04-26 17:36:19]

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/2838193004/#ps40001
(title: "Remove unused variable")

[commit-bot: I haz the power, 2017-04-26 17:37:01]

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-04-26 18:27:18]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493228178695000,
"parent_rev": "52939ed422f481e2970edfa57a8bf24ca3dbacb8", "commit_rev":
"55cde5fb732692857b3535076e5ac9e08188033e"}

[commit-bot: I haz the power, 2017-04-26 18:27:37]

Description was changed from

==========
[Web Payments] Implement placeholders in the shipping options row

BUG=710977
==========

to

==========
[Web Payments] Implement placeholders in the shipping options row

BUG=710977

Review-Url: https://codereview.chromium.org/2838193004
Cr-Commit-Position: refs/heads/master@{#467385}
Committed:
https://chromium.googlesource.com/chromium/src/+/55cde5fb732692857b3535076e5a...
==========

[commit-bot: I haz the power, 2017-04-26 18:27:38]

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