Unreviewed, rolling out r184645.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 May 2015 20:12:39 +0000 (20:12 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 May 2015 20:12:39 +0000 (20:12 +0000)
https://bugs.webkit.org/show_bug.cgi?id=145220

Broken build (Requested by jacob_nielsen on #webkit).

Reverted changeset:

"AX: AutoFill button is not accessible with VoiceOver"
https://bugs.webkit.org/show_bug.cgi?id=145200
http://trac.webkit.org/changeset/184645

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@184650 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/accessibility/input-auto-fill-button-expected.txt [deleted file]
LayoutTests/accessibility/input-auto-fill-button.html [deleted file]
Source/WebCore/ChangeLog
Source/WebCore/English.lproj/Localizable.strings
Source/WebCore/html/TextFieldInputType.cpp
Source/WebCore/platform/LocalizedStrings.cpp
Source/WebCore/platform/LocalizedStrings.h
Source/WebCore/platform/efl/LocalizedStringsEfl.cpp
Source/WebCore/platform/gtk/LocalizedStringsGtk.cpp

index 9e0e61c..23acf48 100644 (file)
@@ -1,3 +1,16 @@
+2015-05-20  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r184645.
+        https://bugs.webkit.org/show_bug.cgi?id=145220
+
+        Broken build (Requested by jacob_nielsen on #webkit).
+
+        Reverted changeset:
+
+        "AX: AutoFill button is not accessible with VoiceOver"
+        https://bugs.webkit.org/show_bug.cgi?id=145200
+        http://trac.webkit.org/changeset/184645
+
 2015-05-20  Daniel Bates  <dabates@apple.com>
 
         AX: AutoFill button is not accessible with VoiceOver
diff --git a/LayoutTests/accessibility/input-auto-fill-button-expected.txt b/LayoutTests/accessibility/input-auto-fill-button-expected.txt
deleted file mode 100644 (file)
index 48c5d3a..0000000
+++ /dev/null
@@ -1,11 +0,0 @@
-This tests that there exists an accessibility element for the AutoFill button.
-
-On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
-
-
-PASS accessibilityController.accessibleElementById("password").childAtIndex(0).description is "AXDescription: autofill"
-PASS accessibilityController.accessibleElementById("password").childAtIndex(0).role is "AXRole: AXButton"
-PASS successfullyParsed is true
-
-TEST COMPLETE
-
diff --git a/LayoutTests/accessibility/input-auto-fill-button.html b/LayoutTests/accessibility/input-auto-fill-button.html
deleted file mode 100644 (file)
index 436a722..0000000
+++ /dev/null
@@ -1,22 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script src="../resources/js-test-pre.js"></script>
-</head>
-<body>
-<input type="password" id="password">
-<script>
-description("This tests that there exists an accessibility element for the AutoFill button.");
-
-if (!window.internals || !window.accessibilityController)
-    testFailed("You must run this test in the test tool and have implemented window.internals and window.accessibilityController.");
-else {
-    internals.setShowAutoFillButton(document.getElementById("password"), true);
-
-    shouldBeEqualToString('accessibilityController.accessibleElementById("password").childAtIndex(0).description', "AXDescription: autofill");
-    shouldBeEqualToString('accessibilityController.accessibleElementById("password").childAtIndex(0).role', "AXRole: AXButton");
-}
-</script>
-<script src="../resources/js-test-post.js"></script>
-</body>
-</html>
index e6c01f9..a29ca48 100644 (file)
@@ -1,3 +1,16 @@
+2015-05-20  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r184645.
+        https://bugs.webkit.org/show_bug.cgi?id=145220
+
+        Broken build (Requested by jacob_nielsen on #webkit).
+
+        Reverted changeset:
+
+        "AX: AutoFill button is not accessible with VoiceOver"
+        https://bugs.webkit.org/show_bug.cgi?id=145200
+        http://trac.webkit.org/changeset/184645
+
 2015-05-20  Daniel Bates  <dabates@apple.com>
 
         AX: AutoFill button is not accessible with VoiceOver
index 0281142..9da8c7d 100644 (file)
 /* accessibility description for a search field cancel button */
 "cancel" = "cancel";
 
-/* accessibility description for a password field AutoFill button */
-"autofill" = "autofill";
-
 /* Verb stating the action that will occur when an unchecked checkbox is clicked, as used by accessibility */
 "check" = "check";
 
index 6c71ae7..cbd48d9 100644 (file)
@@ -42,7 +42,6 @@
 #include "HTMLInputElement.h"
 #include "HTMLNames.h"
 #include "KeyboardEvent.h"
-#include "LocalizedStrings.h"
 #include "NodeRenderStyle.h"
 #include "Page.h"
 #include "PlatformKeyboardEvent.h"
@@ -603,8 +602,6 @@ void TextFieldInputType::createAutoFillButton()
 
     m_autoFillButton = AutoFillButtonElement::create(element().document(), *this);
     m_autoFillButton->setPseudo(AtomicString("-webkit-auto-fill-button", AtomicString::ConstructFromLiteral));
-    m_autoFillButton->setAttribute(aria_labelAttr, AtomicString(AXAutoFillButtonText()));
-    m_autoFillButton->setAttribute(roleAttr, AtomicString("button", AtomicString::ConstructFromLiteral));
     m_container->appendChild(m_autoFillButton, IGNORE_EXCEPTION);
 }
 
index 57f49c5..93d2e90 100644 (file)
@@ -604,11 +604,6 @@ String AXSearchFieldCancelButtonText()
     return WEB_UI_STRING("cancel", "accessibility description for a search field cancel button");
 }
 
-String AXAutoFillButtonText()
-{
-    return WEB_UI_STRING("autofill", "accessibility description for a password field AutoFill button");
-}
-
 String AXButtonActionVerb()
 {
     return WEB_UI_STRING("press", "Verb stating the action that will occur when a button is pressed, as used by accessibility");
index 4f71481..09a3800 100644 (file)
@@ -162,7 +162,7 @@ namespace WebCore {
     String AXFooterRoleDescriptionText();
     String AXFileUploadButtonText();
     String AXSearchFieldCancelButtonText();
-
+    
     String AXButtonActionVerb();
     String AXRadioButtonActionVerb();
     String AXTextFieldActionVerb();
@@ -174,8 +174,6 @@ namespace WebCore {
     String AXListItemActionVerb();
 #endif
 
-    String AXAutoFillButtonText();
-
 #if ENABLE(INPUT_TYPE_WEEK)
     // weekFormatInLDML() returns week and year format in LDML, Unicode
     // technical standard 35, Locale Data Markup Language, e.g. "'Week' ww, yyyy"
index 79009bf..3ea96d7 100644 (file)
@@ -408,11 +408,6 @@ String AXSearchFieldCancelButtonText()
     return String::fromUTF8("cancel");
 }
 
-String AXAutoFillButtonText()
-{
-    return String::fromUTF8("autofill");
-}
-
 String AXButtonActionVerb()
 {
     return String::fromUTF8("press");
index 464c947..412087b 100644 (file)
@@ -456,11 +456,6 @@ String AXSearchFieldCancelButtonText()
     return String::fromUTF8(_("cancel"));
 }
 
-String AXAutoFillButtonText()
-{
-    return String::fromUTF8(_("autofill"));
-}
-
 String AXButtonActionVerb()
 {
     return String::fromUTF8(_("press"));