Browser Test: Autocomplete confused about direction if inline style and inherited dir attribute…

chrome/renderer/autofill/form_autofill_browsertest.cc

 // Copyright (c) 2012 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 <vector>
 
 #include "base/format_macros.h"
 #include "base/metrics/field_trial.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_util.h"
@@ skipping 663 matching lines @@
     if (test_cases[i].form_control_type == "text")
       expected.max_length = WebInputElement::defaultMaxLength();
     else
       expected.max_length = 0;
 
     SCOPED_TRACE(test_cases[i].element_id);
     EXPECT_FORM_FIELD_DATA_EQUALS(expected, result);
   }
 }
 
+// Both text input and autocomplete should have same direction: test_case#1

[Ilya Sherman, 2014/08/07 19:53:01]

Rather than just numbering the test cases, please describe what situation each
test case is testing.  For example, you might name this test

WebFormControlElementToFormField_TextDirectionDetectedFromDirectStyle.

Admittedly, that name is very long, mostly because the method name is very long.
 I personally think the suffix is the more important part; so just naming the
test something like "TextDirectionDetectedFromDirectStyle" would be fine.  (If
you can find a way to fit both the method name and a descriptive test name, even
better!)

[Sunil Ratnu, 2014/08/08 04:56:32]

Done.

+TEST_F(FormAutofillTest,
+       WebFormControlElementToFormFieldAutocompleteDirectionTest1) {
+  LoadHTML("<STYLE>input{direction:rtl}</STYLE>"
+           "<FORM autocomplete='on'>"

[Ilya Sherman, 2014/08/07 19:53:01]

nit: You shouldn't need to explicitly specify autocomplete='on' -- the default
state should be equally good, and is less code.  In tests, the less superfluous
code you have, the easier it is to see at a glance exactly what is being tested
:)

[Sunil Ratnu, 2014/08/08 04:56:32]

Done.

+           "  <INPUT type='text' id='element' value='RTL'/>"

[Ilya Sherman, 2014/08/07 19:53:00]

nit: Would this test be equally valid without a value specified on the input
element?  If so, please omit the value.

[Sunil Ratnu, 2014/08/08 04:56:32]

Done.

+           "</FORM>");
+
+  WebFrame* frame = GetMainFrame();
+  ASSERT_NE(static_cast<WebFrame*>(NULL), frame);
+
+  WebElement web_element = frame->document().getElementById("element");
+  WebFormControlElement element = web_element.to<WebFormControlElement>();
+
+  FormFieldData result;
+  WebFormControlElementToFormField(element, autofill::EXTRACT_VALUE, &result);
+  EXPECT_EQ(base::i18n::RIGHT_TO_LEFT, result.text_direction);
+}
+
+// Both text input and autocomplete should have same direction: test_case#2
+TEST_F(FormAutofillTest,
+       WebFormControlElementToFormFieldAutocompleteDirectionTest2) {
+  LoadHTML("<STYLE>span{direction:rtl}</STYLE>"
+           "<FORM autocomplete='on'>"
+           "  <SPAN>"
+           "    <INPUT type='text' id='element' value='RTL'/>"
+           "  </SPAN>"
+           "</FORM>");
+
+  WebFrame* frame = GetMainFrame();
+  ASSERT_NE(static_cast<WebFrame*>(NULL), frame);
+
+  WebElement web_element = frame->document().getElementById("element");
+  WebFormControlElement element = web_element.to<WebFormControlElement>();
+
+  FormFieldData result;
+  WebFormControlElementToFormField(element, autofill::EXTRACT_VALUE, &result);
+  EXPECT_EQ(base::i18n::RIGHT_TO_LEFT, result.text_direction);
+}
+
+// Both text input and autocomplete should have same direction: test_case#3
+TEST_F(FormAutofillTest,
+       WebFormControlElementToFormFieldAutocompleteDirectionTest3) {
+  LoadHTML("<FORM autocomplete='on'>"
+           "  <SPAN>"
+           "    <INPUT dir='rtl' type='text' id='element' value='RTL'/>"
+           "  </SPAN>"
+           "</FORM>");
+
+  WebFrame* frame = GetMainFrame();
+  ASSERT_NE(static_cast<WebFrame*>(NULL), frame);
+
+  WebElement web_element = frame->document().getElementById("element");
+  WebFormControlElement element = web_element.to<WebFormControlElement>();
+
+  FormFieldData result;
+  WebFormControlElementToFormField(element, autofill::EXTRACT_VALUE, &result);
+  EXPECT_EQ(base::i18n::RIGHT_TO_LEFT, result.text_direction);
+}
+
+// Both text input and autocomplete should have same direction: test_case#4
+TEST_F(FormAutofillTest,
+       WebFormControlElementToFormFieldAutocompleteDirectionTest4) {
+  LoadHTML("<STYLE>input{direction:ltr}</STYLE>"
+           "<FORM dir='rtl' autocomplete='on'>"
+           "  <INPUT type='text' id='element' value='LTR'/>"
+           "</FORM>");
+
+  WebFrame* frame = GetMainFrame();
+  ASSERT_NE(static_cast<WebFrame*>(NULL), frame);
+
+  WebElement web_element = frame->document().getElementById("element");
+  WebFormControlElement element = web_element.to<WebFormControlElement>();
+
+  FormFieldData result;
+  WebFormControlElementToFormField(element, autofill::EXTRACT_VALUE, &result);
+  EXPECT_EQ(base::i18n::LEFT_TO_RIGHT, result.text_direction);
+}
+
+// Both text input and autocomplete should have same direction: test_case#5
+TEST_F(FormAutofillTest,
+       WebFormControlElementToFormFieldAutocompleteDirectionTest5) {
+  LoadHTML("<STYLE>span{direction:ltr}</STYLE>"
+           "<FORM autocomplete='on'>"
+           "  <SPAN dir='rtl'>"
+           "    <INPUT type='text' id='element' value='LTR'/>"
+           "  </SPAN>"
+           "</FORM>");
+
+  WebFrame* frame = GetMainFrame();
+  ASSERT_NE(static_cast<WebFrame*>(NULL), frame);
+
+  WebElement web_element = frame->document().getElementById("element");
+  WebFormControlElement element = web_element.to<WebFormControlElement>();
+
+  FormFieldData result;
+  WebFormControlElementToFormField(element, autofill::EXTRACT_VALUE, &result);
+  EXPECT_EQ(base::i18n::LEFT_TO_RIGHT, result.text_direction);
+}
+
+// Both text input and autocomplete should have same direction: test_case#6
+TEST_F(FormAutofillTest,
+       WebFormControlElementToFormFieldAutocompleteDirectionTest6) {
+  LoadHTML("<BODY dir='rtl'>"
+           "<FORM style='direction:ltr' autocomplete='on'>"
+           "  <SPAN dir='rtl'>"
+           "    <INPUT dir='ltr' type='text' id='element' value='LTR'/>"
+           "  </SPAN>"
+           "</FORM>");
+
+  WebFrame* frame = GetMainFrame();
+  ASSERT_NE(static_cast<WebFrame*>(NULL), frame);
+
+  WebElement web_element = frame->document().getElementById("element");
+  WebFormControlElement element = web_element.to<WebFormControlElement>();
+
+  FormFieldData result;
+  WebFormControlElementToFormField(element, autofill::EXTRACT_VALUE, &result);
+  EXPECT_EQ(base::i18n::LEFT_TO_RIGHT, result.text_direction);
+}
+
 TEST_F(FormAutofillTest, WebFormElementToFormData) {
   LoadHTML("<FORM name=\"TestForm\" action=\"http://cnn.com\" method=\"post\">"
            " <LABEL for=\"firstname\">First name:</LABEL>"
            "  <INPUT type=\"text\" id=\"firstname\" value=\"John\"/>"
            " <LABEL for=\"lastname\">Last name:</LABEL>"
            "  <INPUT type=\"text\" id=\"lastname\" value=\"Smith\"/>"
            " <LABEL for=\"street-address\">Address:</LABEL>"
            "  <TEXTAREA id=\"street-address\">"
                "123 Fantasy Ln.&#10;"
                "Apt. 42"
@@ skipping 2868 matching lines @@
   expected.max_length = WebInputElement::defaultMaxLength();
   EXPECT_FORM_FIELD_DATA_EQUALS(expected, fields[1]);
 
   expected.name = ASCIIToUTF16("country");
   expected.value = ASCIIToUTF16("AL");
   expected.form_control_type = "select-one";
   expected.max_length = 0;
   EXPECT_FORM_FIELD_DATA_EQUALS(expected, fields[2]);
 }
 }  // namespace autofill