rAc: When filling partial autofill data, don't fill across dialog sections.

rAc: When filling partial autofill data, don't fill across dialog sections.

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

[groby-ooo-7-16, 2014-06-30 22:01:31]

LGTM % comment nit

https://codereview.chromium.org/365463002/diff/1/chrome/browser/ui/autofill/a...
File chrome/browser/ui/autofill/autofill_dialog_controller_browsertest.cc
(right):

https://codereview.chromium.org/365463002/diff/1/chrome/browser/ui/autofill/a...
chrome/browser/ui/autofill/autofill_dialog_controller_browsertest.cc:675: //
Adding a new billing address.
Who is adding that address? Not sure I get what the comment refers to.

[Evan Stade, 2014-06-30 22:16:47]

https://codereview.chromium.org/365463002/diff/1/chrome/browser/ui/autofill/a...
File chrome/browser/ui/autofill/autofill_dialog_controller_browsertest.cc
(right):

https://codereview.chromium.org/365463002/diff/1/chrome/browser/ui/autofill/a...
chrome/browser/ui/autofill/autofill_dialog_controller_browsertest.cc:675: //
Adding a new billing address.
On 2014/06/30 22:01:31, groby wrote:
> Who is adding that address? Not sure I get what the comment refers to.

sorry, should read: "Dialog is already asking for a new billing address."

[Evan Stade, 2014-07-07 22:29:14]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2014-07-07 22:29:19]

The CQ bit was unchecked by estade@chromium.org

[Evan Stade, 2014-07-07 22:30:56]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2014-07-07 22:32:53]

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

[commit-bot: I haz the power, 2014-07-08 01:07:24]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/9...)

[commit-bot: I haz the power, 2014-07-08 01:50:05]

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

[commit-bot: I haz the power, 2014-07-08 01:50:06]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/9...)

[Evan Stade, 2014-07-08 17:52:48]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2014-07-08 17:55:10]

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

[commit-bot: I haz the power, 2014-07-08 19:21:33]

Change committed as 281791