[Autofill] Add a unit test for the autofill_regexes code.

[Autofill] Add a unit test for the autofill_regexes code.

BUG=none
TBR=thakis@chromium.org


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=165043

[Ilya Sherman, 2012-10-30 01:09:24]

Noticed while debugging that this file had no test coverage.

[Albert Bodenhamer, 2012-10-30 20:28:40]

https://chromiumcodereview.appspot.com/11344028/diff/1003/chrome/browser/auto...
File chrome/browser/autofill/autofill_regexes.cc (right):

https://chromiumcodereview.appspot.com/11344028/diff/1003/chrome/browser/auto...
chrome/browser/autofill/autofill_regexes.cc:83: }  // namespace autofill
Other than whitespace changes I'm not seeing anything changed in this file.

[Ilya Sherman, 2012-10-30 20:34:09]

https://chromiumcodereview.appspot.com/11344028/diff/1003/chrome/browser/auto...
File chrome/browser/autofill/autofill_regexes.cc (right):

https://chromiumcodereview.appspot.com/11344028/diff/1003/chrome/browser/auto...
chrome/browser/autofill/autofill_regexes.cc:83: }  // namespace autofill
On 2012/10/30 20:28:41, Albert Bodenhamer wrote:
> Other than whitespace changes I'm not seeing anything changed in this file.

It's just a whitespace change in this file, yes.

[Albert Bodenhamer, 2012-10-30 20:36:49]

lgtm

[commit-bot: I haz the power, 2012-10-30 20:40:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/isherman@chromium.org/11344028/1003

[commit-bot: I haz the power, 2012-10-30 20:40:58]

Presubmit check for 11344028-1003 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome

Presubmit checks took 2.7s to calculate.

[Ilya Sherman, 2012-10-30 20:43:37]

TBR Nico for the gyp change

[commit-bot: I haz the power, 2012-10-30 20:43:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/isherman@chromium.org/11344028/1003

[Nico, 2012-10-30 20:45:19]

lgtm

[commit-bot: I haz the power, 2012-10-30 22:49:19]

Change committed as 165043