rAc: fix validation logic when autofill completion sets country

rAc: fix validation logic when autofill completion sets country

BUG=314842
R=dbeam@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233430

[Dan Beam, 2013-11-05 00:01:32]

so the effect of this is that the user is able to ship/bill international
addresses but only validate based on US rules (or something)?

[Evan Stade, 2013-11-05 00:02:59]

On 2013/11/05 00:01:32, Dan Beam wrote:
> so the effect of this is that the user is able to ship/bill international
> addresses but only validate based on US rules (or something)?

good point. That wasn't the intention.

[Evan Stade, 2013-11-05 01:28:00]

updated. TAL?

[Dan Beam, 2013-11-05 02:54:42]

lgtm* l = &(*lgtm)[i];

https://codereview.chromium.org/58683002/diff/50001/chrome/browser/ui/autofil...
File chrome/browser/ui/autofill/autofill_dialog_controller_browsertest.cc
(right):

https://codereview.chromium.org/58683002/diff/50001/chrome/browser/ui/autofil...
chrome/browser/ui/autofill/autofill_dialog_controller_browsertest.cc:507:
string16 value = full_profile.GetRawInfo(triggering_input.type);
nit: base::string16 in new code (or using base::string16;)

https://codereview.chromium.org/58683002/diff/50001/chrome/browser/ui/autofil...
File chrome/browser/ui/autofill/data_model_wrapper.cc (right):

https://codereview.chromium.org/58683002/diff/50001/chrome/browser/ui/autofil...
chrome/browser/ui/autofill/data_model_wrapper.cc:30: for (size_t i = 0; i <
inputs->size(); ++i) {
nit: DetailInput* input = &(*inputs)[i];

[commit-bot: I haz the power, 2013-11-05 04:04:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/estade@chromium.org/58683002/290001

[commit-bot: I haz the power, 2013-11-05 04:38:30]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-11-05 04:55:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/estade@chromium.org/58683002/290001

[commit-bot: I haz the power, 2013-11-05 05:12:30]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-11-05 06:01:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/estade@chromium.org/58683002/290001

[commit-bot: I haz the power, 2013-11-05 10:07:08]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-11-05 18:01:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/estade@chromium.org/58683002/290001

[commit-bot: I haz the power, 2013-11-05 21:46:56]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-11-05 22:19:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/estade@chromium.org/58683002/290001

[commit-bot: I haz the power, 2013-11-06 03:27:54]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[Evan Stade, 2013-11-07 00:22:40]

Committed patchset #3 manually as r233430 (presubmit successful).