Remove incorrect assert in PaymentRequest address editor UI.

Remove incorrect assert in PaymentRequest address editor UI.

The address editor in PaymentRequest UI needs to set the value of the
country dropdown when editing or create a new shipping address. This
change to the dropdown should not fire a callback that re-arranges the
address input fields. This is accomplished via the
EditorFieldModel.setValue() call. Therefore, asserting that setValue()
is not used for dropdowns is incorrect.

The mCountryField.setValue() call has been moved below the
mCountryField.setDropdownCallback() to make it clear to the reader that
the callback is not fired intentionally, instead of accidentally.

BUG=624653
TBR=dfalcantara

Committed: https://crrev.com/e5478255cd31bffa2867db84946f9512cba5a3fd
Cr-Commit-Position: refs/heads/master@{#403085}

[please use gerrit instead, 2016-06-30 03:01:00]

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

[please use gerrit instead, 2016-06-30 03:01:01]

dfalcantara@, ptal.

horo@, fyi.

[please use gerrit instead, 2016-06-30 03:53:30]

Since this change is trivial and fixes android bots, I'm going to TBR it.

[please use gerrit instead, 2016-06-30 03:53:58]

Description was changed from

==========
Remove incorrect assert in PaymentRequest address editor UI.

The address editor in PaymentRequest UI needs to set the value of the
country dropdown when editing or create a new shipping address. This
change to the dropdown should not fire a callback that re-arranges the
address input fields. This is accomplished via the
EditorFieldModel.setValue() call. Therefore, asserting that setValue()
is not used for dropdowns is incorrect.

The mCountryField.setValue() call has been moved below the
mCountryField.setDropdownCallback() to make it clear to the reader that
the callback is not fired intentionally, instead of accidentally.

BUG=624653
==========

to

==========
Remove incorrect assert in PaymentRequest address editor UI.

The address editor in PaymentRequest UI needs to set the value of the
country dropdown when editing or create a new shipping address. This
change to the dropdown should not fire a callback that re-arranges the
address input fields. This is accomplished via the
EditorFieldModel.setValue() call. Therefore, asserting that setValue()
is not used for dropdowns is incorrect.

The mCountryField.setValue() call has been moved below the
mCountryField.setDropdownCallback() to make it clear to the reader that
the callback is not fired intentionally, instead of accidentally.

BUG=624653
TBR=dfalcantara
==========

[please use gerrit instead, 2016-06-30 03:54:05]

The CQ bit was checked by rouslan@chromium.org

[commit-bot: I haz the power, 2016-06-30 03:54:25]

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-06-30 03:58:40]

Description was changed from

==========
Remove incorrect assert in PaymentRequest address editor UI.

The address editor in PaymentRequest UI needs to set the value of the
country dropdown when editing or create a new shipping address. This
change to the dropdown should not fire a callback that re-arranges the
address input fields. This is accomplished via the
EditorFieldModel.setValue() call. Therefore, asserting that setValue()
is not used for dropdowns is incorrect.

The mCountryField.setValue() call has been moved below the
mCountryField.setDropdownCallback() to make it clear to the reader that
the callback is not fired intentionally, instead of accidentally.

BUG=624653
TBR=dfalcantara
==========

to

==========
Remove incorrect assert in PaymentRequest address editor UI.

The address editor in PaymentRequest UI needs to set the value of the
country dropdown when editing or create a new shipping address. This
change to the dropdown should not fire a callback that re-arranges the
address input fields. This is accomplished via the
EditorFieldModel.setValue() call. Therefore, asserting that setValue()
is not used for dropdowns is incorrect.

The mCountryField.setValue() call has been moved below the
mCountryField.setDropdownCallback() to make it clear to the reader that
the callback is not fired intentionally, instead of accidentally.

BUG=624653
TBR=dfalcantara
==========

[commit-bot: I haz the power, 2016-06-30 03:58:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-30 03:58:45]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 04:00:30]

Description was changed from

==========
Remove incorrect assert in PaymentRequest address editor UI.

The address editor in PaymentRequest UI needs to set the value of the
country dropdown when editing or create a new shipping address. This
change to the dropdown should not fire a callback that re-arranges the
address input fields. This is accomplished via the
EditorFieldModel.setValue() call. Therefore, asserting that setValue()
is not used for dropdowns is incorrect.

The mCountryField.setValue() call has been moved below the
mCountryField.setDropdownCallback() to make it clear to the reader that
the callback is not fired intentionally, instead of accidentally.

BUG=624653
TBR=dfalcantara
==========

to

==========
Remove incorrect assert in PaymentRequest address editor UI.

The address editor in PaymentRequest UI needs to set the value of the
country dropdown when editing or create a new shipping address. This
change to the dropdown should not fire a callback that re-arranges the
address input fields. This is accomplished via the
EditorFieldModel.setValue() call. Therefore, asserting that setValue()
is not used for dropdowns is incorrect.

The mCountryField.setValue() call has been moved below the
mCountryField.setDropdownCallback() to make it clear to the reader that
the callback is not fired intentionally, instead of accidentally.

BUG=624653
TBR=dfalcantara

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

[commit-bot: I haz the power, 2016-06-30 04:00:30]

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