Autocomplete should work with nameless fields

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"
@@ skipping 46 matching lines @@
     if (input_element->inputType() != WebCore::HTMLInputElement::TEXT)
       continue;
 
     // For each TEXT input field, store the name and value
     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;
+    std::wstring name = GetNameForInputElement(input_element);
+    if (name.length() == 0)
+      continue;  // If we have no name, there is nothing to store.
 
     result->elements.push_back(AutofillForm::Element(name, value));
   }
 
   return result;
 }
+
+// static
+std::wstring AutofillForm::GetNameForInputElement(WebCore::HTMLInputElement*
+    element) {
+  std::wstring name = webkit_glue::StringToStdWString(element->name());
+  std::wstring trimmed_name;
+  TrimWhitespace(name, TRIM_LEADING, &trimmed_name);
+  if (trimmed_name.length() > 0)
+    return trimmed_name;
+
+  name = webkit_glue::StringToStdWString(element->id());
+  TrimWhitespace(name, TRIM_LEADING, &trimmed_name);
+  if (trimmed_name.length() > 0)
+    return trimmed_name;
+
+  return std::wstring();
+}