fix autofill shadowing vars warning

fix autofill shadowing vars warning

found by the vc /analyze bot

BUG=440500
Committed: https://crrev.com/d722c2ce033811a1dcb33d7c2fb7d1cfc7041e78
Cr-Commit-Position: refs/heads/master@{#313433}

[cpu_(ooo_6.6-7.5), 2015-01-28 00:44:21]

cpu@chromium.org changed reviewers:
+ isherman@chromium.org

[cpu_(ooo_6.6-7.5), 2015-01-28 00:44:38]

these are the warnings

e:\b\build\slave\chromium_windows_analyze\build\src\third_party\skia\include\core\skmath.h(82)
: warning C6102: Using 'index' from failed function call at line '81'.: Lines:
79, 80, 81, 82
e:\b\build\slave\chromium_windows_analyze\build\src\chrome\browser\ui\autofill\autofill_dialog_controller_unittest.cc(746)
: warning C6246: Local declaration of 'i' hides declaration of the same name in
outer scope. For additional information, see previous declaration at line '735'
of
'e:\b\build\slave\chromium_windows_analyze\build\src\chrome\browser\ui\autofill\autofill_dialog_controller_unittest.cc'.:
Lines: 735
e:\b\build\slave\chromium_windows_analyze\build\src\chrome\browser\ui\autofill\autofill_dialog_controller_unittest.cc(2976)
: warning C6246: Local declaration of 'inputs' hides declaration of the same
name in outer scope. For additional information, see previous declaration at
line '2969' of
'e:\b\build\slave\chromium_windows_analyze\build\src\chrome\browser\ui\autofill\autofill_dialog_controller_unittest.cc'.:
Lines: 2969
e:\b\build\slave\chromium_windows_analyze\build\src\chrome\browser\ui\autofill\autofill_dialog_controller_unittest.cc(2992)
: warning C6246: Local declaration of 'inputs' hides declaration of the same
name in outer scope. For additional information, see previous declaration at
line '2969' of
'e:\b\build\slave\chromium_windows_analyze\build\src\chrome\browser\ui\autofill\autofill_dialog_controller_unittest.cc'.:
Lines: 2969
e:\b\build\slave\chromium_windows_analyze\build\src\chrome\browser\ui\autofill\autofill_dialog_controller_unittest.cc(3009)
: warning C6246: Local declaration of 'inputs' hides declaration of the same
name in outer scope. For additional information, see previous declaration at
line '2969' of
'e:\b\build\slave\chromium_windows_analyze\build\src\chrome\browser\ui\autofill\autofill_dialog_controller_unittest.cc'.:
Lines: 2969

[Ilya Sherman, 2015-01-28 01:35:49]

LGTM % a nit.  Thanks.

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

https://codereview.chromium.org/876343004/diff/1/chrome/browser/ui/autofill/a...
chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc:746: for
(size_t ix = 0; ix < inputs.size(); ++ix) {
nit: Please use 'j' instead of 'ix'

[cpu_(ooo_6.6-7.5), 2015-01-28 02:02:34]

The CQ bit was checked by cpu@chromium.org

[commit-bot: I haz the power, 2015-01-28 02:03:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/876343004/20001

[commit-bot: I haz the power, 2015-01-28 04:14:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-28 04:16:06]

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