Don't store empty values for autofill

webkit/glue/autofill_form.cc

 // Copyright (c) 2006-2008 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 "config.h"
 #include "base/compiler_specific.h"
 
 MSVC_PUSH_WARNING_LEVEL(0);
 #include "Frame.h"
 #include "HTMLInputElement.h"
 #include "HTMLNames.h"
 MSVC_POP_WARNING();
 
 #undef LOG
 
 #include "base/basictypes.h"
 #include "base/logging.h"
+#include "base/string_util.h"
 #include "webkit/glue/autofill_form.h"
 #include "webkit/glue/glue_util.h"
 
 AutofillForm* AutofillForm::CreateAutofillForm(
     WebCore::HTMLFormElement* form) {
 
   DCHECK(form);
 
   WebCore::Frame* frame = form->document()->frame();
 
@@ skipping 22 matching lines @@
     WebCore::HTMLInputElement* input_element =
         static_cast<WebCore::HTMLInputElement*>(form_element);
     if (!input_element->isEnabled())
       continue;
 
     // Ignore all input types except TEXT.
     if (input_element->inputType() != WebCore::HTMLInputElement::TEXT)
       continue;
 
     // For each TEXT input field, store the name and value
-    std::wstring name = webkit_glue::StringToStdWString(input_element->name());
     std::wstring value = webkit_glue::StringToStdWString(
         input_element->value());
+    TrimWhitespace(value, TRIM_LEADING, &value);
+    if (value.length() == 0)
+      continue;
+
+    std::wstring name = webkit_glue::StringToStdWString(input_element->name());
+    // Test that the name is not blank.
+    std::wstring trimmed_name;
+    TrimWhitespace(name, TRIM_LEADING, &trimmed_name);
+    if (trimmed_name.length() == 0)
+      continue;
 
     result->elements.push_back(AutofillForm::Element(name, value));
   }
 
   return result;
 }