Autofill select field text should be captured not selection option value

chrome/renderer/form_manager.h

Index: chrome/renderer/form_manager.h
diff --git a/chrome/renderer/form_manager.h b/chrome/renderer/form_manager.h
index b50fc552adb31c9afd18f334aeb0903e199675c7..e05ad872803456e0507dfd1d4ebbad9e79f270dd 100644
--- a/chrome/renderer/form_manager.h
+++ b/chrome/renderer/form_manager.h
@@ -38,7 +38,9 @@ class FormManager {
   enum ExtractMask {
     EXTRACT_NONE    = 0,
     EXTRACT_VALUE   = 1 << 0,  // Extract value from WebFormControlElement.
-    EXTRACT_OPTIONS = 1 << 1,  // Extract options from WebFormControlElement.
+    EXTRACT_TEXT    = 1 << 1,  // Extract text from WebFormSelectElement.
+                               // Only valid when |EXTRACT_VALUE| is set.

[James Hawkins, 2010/11/16 18:03:12]

This line bothers me.  What text is it extracting?  Why do we need a new mask
for text?  At the very least, the name needs more, e.g. EXTRACT_OPTION_TEXT.

[dhollowa, 2010/11/16 19:25:57]

Done.

+    EXTRACT_OPTIONS = 1 << 2,  // Extract options from WebFormControlElement.
   };
 
   FormManager();