diff options
| -rw-r--r-- | chrome/renderer/form_manager.cc | 4 | ||||
| -rw-r--r-- | chrome/renderer/form_manager_unittest.cc | 12 |
2 files changed, 11 insertions, 5 deletions
diff --git a/chrome/renderer/form_manager.cc b/chrome/renderer/form_manager.cc index 7c3fa73..7063e97 100644 --- a/chrome/renderer/form_manager.cc +++ b/chrome/renderer/form_manager.cc @@ -58,7 +58,9 @@ void FormManager::WebFormControlElementToFormField( const WebFormControlElement& element, FormField* field) { DCHECK(field); - field->set_label(LabelForElement(element)); + // TODO(jhawkins): LabelForElement. Returning an empty label temporarily to + // diagnose a perf issue. + field->set_label(string16()); field->set_name(element.nameForAutofill()); field->set_form_control_type(element.formControlType()); diff --git a/chrome/renderer/form_manager_unittest.cc b/chrome/renderer/form_manager_unittest.cc index 256f58f..feda04d 100644 --- a/chrome/renderer/form_manager_unittest.cc +++ b/chrome/renderer/form_manager_unittest.cc @@ -377,7 +377,8 @@ TEST_F(FormManagerTest, Reset) { ASSERT_EQ(0U, forms.size()); } -TEST_F(FormManagerTest, Labels) { +// http://crbug.com/40306 +TEST_F(FormManagerTest, DISABLED_Labels) { LoadHTML("<FORM name=\"TestForm\" action=\"http://cnn.com\" method=\"post\">" " <LABEL for=\"firstname\"> First name: </LABEL>" " <INPUT type=\"text\" id=\"firstname\" value=\"John\"/>" @@ -420,7 +421,8 @@ TEST_F(FormManagerTest, Labels) { fields[2]); } -TEST_F(FormManagerTest, LabelsInferredFromText) { +// http://crbug.com/40306 +TEST_F(FormManagerTest, DISABLED_LabelsInferredFromText) { LoadHTML("<FORM name=\"TestForm\" action=\"http://cnn.com\" method=\"post\">" " First name:" " <INPUT type=\"text\" id=\"firstname\" value=\"John\"/>" @@ -463,7 +465,8 @@ TEST_F(FormManagerTest, LabelsInferredFromText) { fields[2]); } -TEST_F(FormManagerTest, LabelsInferredFromParagraph) { +// http://crbug.com/40306 +TEST_F(FormManagerTest, DISABLED_LabelsInferredFromParagraph) { LoadHTML("<FORM name=\"TestForm\" action=\"http://cnn.com\" method=\"post\">" " <P>First name:</P><INPUT type=\"text\" " " id=\"firstname\" value=\"John\"/>" @@ -506,7 +509,8 @@ TEST_F(FormManagerTest, LabelsInferredFromParagraph) { fields[2]); } -TEST_F(FormManagerTest, LabelsInferredFromTableCell) { +// http://crbug.com/40306 +TEST_F(FormManagerTest, DISABLED_LabelsInferredFromTableCell) { LoadHTML("<FORM name=\"TestForm\" action=\"http://cnn.com\" method=\"post\">" "<TABLE>" " <TR>" |