Revert of Expose IA and IA2 role and state for HTML input type attrib…

…utes related to date and time. (patchset #1 id:1 of https://codereview.chromium.org/612063002/) Reason for revert: This likely broke DumpAccessibilityTreeTest.AccessibilityInputTime in content_browsertest Original issue's description: > Expose IA and IA2 role and state for HTML input type attributes related to date and time. > > For input type date, datetime and datetime-local, time, month, week - IA and IA2 role and states > are not exposed properly. > > BUG=418486 > > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=182848 [email protected],[email protected],[email protected],[email protected] NOTREECHECKS=true NOTRY=true BUG=418486 Review URL: https://codereview.chromium.org/609403005 git-svn-id: svn://svn.chromium.org/blink/trunk@182863 bbb929c8-8fbe-4397-9dbb-9b2b20218538
yoavweiss · Sep 29, 2014 · 0c6b223 · 0c6b223
1 parent f3f5b34
commit 0c6b223
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 27 deletions.
diff --git a/Source/core/accessibility/AXNodeObject.cpp b/Source/core/accessibility/AXNodeObject.cpp
@@ -212,13 +212,6 @@ AccessibilityRole AXNodeObject::determineAccessibilityRole()
                 return CheckBoxMenuItemRole;
             return CheckBoxRole;
         }
-        if (type == InputTypeNames::date)
-            return DateRole;
-        if (type == InputTypeNames::datetime
-            || type == InputTypeNames::datetime_local
-            || type == InputTypeNames::month
-            || type == InputTypeNames::week)
-            return DateTimeRole;
         if (type == InputTypeNames::radio)
             return RadioButtonRole;
         if (input.isTextButton())
@@ -227,8 +220,6 @@ AccessibilityRole AXNodeObject::determineAccessibilityRole()
             return SliderRole;
         if (type == InputTypeNames::color)
             return ColorWellRole;
-        if (type == InputTypeNames::time)
-            return TimeRole;
         return TextFieldRole;
     }
     if (isHTMLSelectElement(*node())) {

diff --git a/Source/core/accessibility/AXObject.h b/Source/core/accessibility/AXObject.h
@@ -78,8 +78,6 @@ enum AccessibilityRole {
     ComboBoxRole,
     ComplementaryRole,
     ContentInfoRole,
-    DateRole,
-    DateTimeRole,
     DefinitionRole,
     DescriptionListDetailRole,
     DescriptionListTermRole,
@@ -168,7 +166,6 @@ enum AccessibilityRole {
     TableRole,
     TextAreaRole,
     TextFieldRole,
-    TimeRole,
     TimerRole,
     ToggleButtonRole,
     ToolbarRole,

diff --git a/Source/core/accessibility/AXRenderObject.cpp b/Source/core/accessibility/AXRenderObject.cpp
@@ -323,21 +323,12 @@ AccessibilityRole AXRenderObject::determineAccessibilityRole()
                 return CheckBoxMenuItemRole;
             return CheckBoxRole;
         }
-        if (type == InputTypeNames::date)
-            return DateRole;
-        if (type == InputTypeNames::datetime
-            || type == InputTypeNames::datetime_local
-            || type == InputTypeNames::month
-            || type == InputTypeNames::week)
-            return DateTimeRole;
         if (type == InputTypeNames::radio)
             return RadioButtonRole;
         if (input.isTextButton())
             return buttonRoleType();
         if (type == InputTypeNames::color)
             return ColorWellRole;
-        if (type == InputTypeNames::time)
-            return TimeRole;
     }
 
     if (isFileUploadButton())

diff --git a/Source/web/AssertMatchingEnums.cpp b/Source/web/AssertMatchingEnums.cpp
@@ -190,8 +190,6 @@ COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleColumn, ColumnRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleComboBox, ComboBoxRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleComplementary, ComplementaryRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleContentInfo, ContentInfoRole);
-COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleDate, DateRole);
-COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleDateTime, DateTimeRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleDefinition, DefinitionRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleDescriptionListDetail, DescriptionListDetailRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleDescriptionListTerm, DescriptionListTermRole);
@@ -279,7 +277,6 @@ COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleTableHeaderContainer, TableHeaderContainer
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleTable, TableRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleTextArea, TextAreaRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleTextField, TextFieldRole);
-COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleTime, TimeRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleTimer, TimerRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleToggleButton, ToggleButtonRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAXRoleToolbar, ToolbarRole);

diff --git a/public/web/WebAXEnums.h b/public/web/WebAXEnums.h
@@ -90,8 +90,6 @@ enum WebAXRole {
     WebAXRoleComboBox,
     WebAXRoleComplementary,
     WebAXRoleContentInfo,
-    WebAXRoleDate,
-    WebAXRoleDateTime,
     WebAXRoleDefinition,
     WebAXRoleDescriptionListDetail,
     WebAXRoleDescriptionListTerm,
@@ -180,7 +178,6 @@ enum WebAXRole {
     WebAXRoleTable,
     WebAXRoleTextArea,
     WebAXRoleTextField,
-    WebAXRoleTime,
     WebAXRoleTimer,
     WebAXRoleToggleButton,
     WebAXRoleToolbar,