Trim username_value to passwordForm

components/autofill/content/renderer/password_form_conversion_utils.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/autofill/content/renderer/password_form_conversion_utils.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <string>
@@ skipping 528 matching lines @@
                 base::i18n::ToLower(username_value),
                 base::i18n::ToLower(typed_username_value),
                 base::CompareCase::SENSITIVE)) {
           // We check that |username_value| was not obtained by autofilling
           // |typed_username_value|. In case when it was, |typed_username_value|
           // is incomplete, so we should leave autofilled value.
           username_value = typed_username_value;
         }
       }
     }
+    base::TrimString(username_value, base::ASCIIToUTF16(" "), &username_value);

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

Can you add a test in password_form_conversion_utils_browsertest.cc to test this
change.

     password_form->username_value = username_value;
   }
 
   password_form->origin =
       form_util::GetCanonicalOriginForDocument(form.document);
   GURL::Replacements rep;
   rep.SetPathStr("");
   password_form->signon_realm =
       password_form->origin.ReplaceComponents(rep).spec();
   password_form->other_possible_usernames.swap(other_possible_usernames);
@@ skipping 128 matching lines @@
 }
 
 bool HasAutocompleteAttributeValue(const blink::WebInputElement& element,
                                    const char* value_in_lowercase) {
   return base::LowerCaseEqualsASCII(
       base::StringPiece16(element.getAttribute("autocomplete")),
       value_in_lowercase);
 }
 
 }  // namespace autofill