Preserve the selected shipping option index.

Preserve the selected shipping option index.

When the user selects a shipping option, the merchant can update the
list of line items and shipping options. If the merchant sends back
unchanged shipping options, then this update should not clear user's
shipping option selection.

BUG=607705
Committed: https://crrev.com/1222c2859b500ee61b3070f4c39db4a77a16be77
Cr-Commit-Position: refs/heads/master@{#392816}

[please use gerrit instead, 2016-05-09 01:11:36]

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

[please use gerrit instead, 2016-05-09 01:11:37]

dfalcantara@, ptal.

[please use gerrit instead, 2016-05-09 01:19:11]

Patchset #1 (id:1) has been deleted

[gone, 2016-05-09 16:59:38]

Is it possible that the shipping options get rearranged?  I imagine that the
user could go from something that is cheap to getting some expensive shipping
option instead.

[please use gerrit instead, 2016-05-09 17:02:40]

On 2016/05/09 16:59:38, dfalcantara wrote:
> Is it possible that the shipping options get rearranged?  I imagine that the
> user could go from something that is cheap to getting some expensive shipping
> option instead.

Sounds like we need to compare the shipping options and clear the selection if
they've changed. Will do.

[please use gerrit instead, 2016-05-11 00:59:46]

dfalcantara@, ptal patch 2.

[please use gerrit instead, 2016-05-11 01:01:02]

Description was changed from

==========
Preserve the selected shipping option index.

When the user selects a shipping option, the merchant can update the
list of line items and shipping options. This update should not clear
the shipping option selection.

BUG=607705
==========

to

==========
Preserve the selected shipping option index.

When the user selects a shipping option, the merchant can update the
list of line items and shipping options. If the merchant sends back
unchanged shipping options, then his update should not clear user's
shipping option selection.

BUG=607705
==========

[gone, 2016-05-11 01:08:06]

lgtm

[please use gerrit instead, 2016-05-11 01:12:26]

Description was changed from

==========
Preserve the selected shipping option index.

When the user selects a shipping option, the merchant can update the
list of line items and shipping options. If the merchant sends back
unchanged shipping options, then his update should not clear user's
shipping option selection.

BUG=607705
==========

to

==========
Preserve the selected shipping option index.

When the user selects a shipping option, the merchant can update the
list of line items and shipping options. If the merchant sends back
unchanged shipping options, then this update should not clear user's
shipping option selection.

BUG=607705
==========

[please use gerrit instead, 2016-05-11 01:13:46]

The CQ bit was checked by rouslan@chromium.org

[commit-bot: I haz the power, 2016-05-11 01:14:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1957973003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1957973003/40001

[commit-bot: I haz the power, 2016-05-11 01:51:56]

Description was changed from

==========
Preserve the selected shipping option index.

When the user selects a shipping option, the merchant can update the
list of line items and shipping options. If the merchant sends back
unchanged shipping options, then this update should not clear user's
shipping option selection.

BUG=607705
==========

to

==========
Preserve the selected shipping option index.

When the user selects a shipping option, the merchant can update the
list of line items and shipping options. If the merchant sends back
unchanged shipping options, then this update should not clear user's
shipping option selection.

BUG=607705
==========

[commit-bot: I haz the power, 2016-05-11 01:51:57]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-05-11 01:53:35]

Description was changed from

==========
Preserve the selected shipping option index.

When the user selects a shipping option, the merchant can update the
list of line items and shipping options. If the merchant sends back
unchanged shipping options, then this update should not clear user's
shipping option selection.

BUG=607705
==========

to

==========
Preserve the selected shipping option index.

When the user selects a shipping option, the merchant can update the
list of line items and shipping options. If the merchant sends back
unchanged shipping options, then this update should not clear user's
shipping option selection.

BUG=607705

Committed: https://crrev.com/1222c2859b500ee61b3070f4c39db4a77a16be77
Cr-Commit-Position: refs/heads/master@{#392816}
==========

[commit-bot: I haz the power, 2016-05-11 01:53:36]

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