Trim username_value to passwordForm

Trim username_value to passwordForm

username_element having input type="text" allows space character
and it can make duplicated suggestion items to users.
One is a normal id and the other is the id including space character.
Usually when user input id with predictive text of IME,
it include space character at the end.
So need trim username_value.

BUG=598543

[yeonju.ann, 2016-03-29 07:18:10]

Description was changed from

==========
Trim username_value to passwordForm

username_element having input type="text" allows space character
and it can make duplicated suggestion items to users.
One is normal id and the other is id including space character.
Usually when user input id with predictive test of IME,
it include space character at the end.
So need trim username_value.

BUG=598543
==========

to

==========
Trim username_value to passwordForm

username_element having input type="text" allows space character
and it can make duplicated suggestion items to users.
One is a normal id and the other is the id including space character.
Usually when user input id with predictive text of IME,
it include space character at the end.
So need trim username_value.

BUG=598543
==========

[ramya.v, 2016-03-31 04:10:06]

ramya.v@samsung.com changed reviewers:
+ ramya.v@samsung.com

[ramya.v, 2016-03-31 04:10:07]

https://codereview.chromium.org/1841663002/diff/20001/components/autofill/con...
File components/autofill/content/renderer/password_form_conversion_utils.cc
(right):

https://codereview.chromium.org/1841663002/diff/20001/components/autofill/con...
components/autofill/content/renderer/password_form_conversion_utils.cc:549:
base::TrimString(username_value, base::ASCIIToUTF16(" "), &username_value);
Can you add a test in password_form_conversion_utils_browsertest.cc to test this
change.

[vabr (Chromium), 2016-03-31 09:26:16]

Hello,

Thanks for the contribution, but please note that this is a somewhat
controversial change: http://crbug.com/598543#c5.

Cheers,
Vaclav