[Merge M-53] [Payments] Use the correct index when checking addresses

[Merge M-53] [Payments] Use the correct index when checking addresses

We have an off-by-one error now that the mOptionLayout can have multiple children in it above the actual options.

* Use the correct index when indexing into the options layout.

* Add a test that checks for the crash and additionally checks that the "merchant doesn't accept address" string is
correctly displayed.

* Plumbs a new callback that indicates that the client has finished
checking the selection.

BUG=630348
Review-Url: https://codereview.chromium.org/2170603006
Cr-Commit-Position: refs/heads/master@{#407197}
(cherry picked from commit 80155b30015fca0ea8774117ee2a9b8d0d8da91d)

Committed: https://chromium.googlesource.com/chromium/src/+/aeb73e31bf78992a0303c8325c4f1e3dcb26dc1b

[please use gerrit instead, 2016-07-25 01:21:03]

Description was changed from

==========
[Merge M-53] [Payments] Use the correct index when checking addresses

We have an off-by-one error now that the mOptionLayout can have multiple
children in it above the actual options.

* Use the correct index when indexing into the options layout.

* Add a test that checks for the crash and additionally checks that the
"merchant doesn't accept address" string is
correctly displayed.

* Plumbs a new callback that indicates that the client has finished
checking the selection.

BUG=630348

Review-Url: https://codereview.chromium.org/2170603006
Cr-Commit-Position: refs/heads/master@{#407197}
(cherry picked from commit 80155b30015fca0ea8774117ee2a9b8d0d8da91d)
==========

to

==========
[Merge M-53] [Payments] Use the correct index when checking addresses

We have an off-by-one error now that the mOptionLayout can have multiple
children in it above the actual options.

* Use the correct index when indexing into the options layout.

* Add a test that checks for the crash and additionally checks that the
"merchant doesn't accept address" string is
correctly displayed.

* Plumbs a new callback that indicates that the client has finished
checking the selection.

BUG=630348

Review-Url: https://codereview.chromium.org/2170603006
Cr-Commit-Position: refs/heads/master@{#407197}
(cherry picked from commit 80155b30015fca0ea8774117ee2a9b8d0d8da91d)

Committed:
https://chromium.googlesource.com/chromium/src/+/aeb73e31bf78992a0303c8325c4f...
==========

[please use gerrit instead, 2016-07-25 01:21:04]

Committed patchset #1 (id:1) manually as
aeb73e31bf78992a0303c8325c4f1e3dcb26dc1b.