Changeset 137111 in webkit

Timestamp:

Dec 9, 2012 11:28:02 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[GTK] lack of implemention of AccessibilityUIElementGtk::isSelectable and AccessibilityUIElementGtk::isMultiSelectable
​https://bugs.webkit.org/show_bug.cgi?id=104481

Patch by Alejandro Piñeiro <​apinheiro@igalia.com> on 2012-12-09
Reviewed by Martin Robinson.

• DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:

(AccessibilityUIElement::isSelectable): Implement this by checking
the current state of the AtkObject.
(AccessibilityUIElement::isMultiSelectable): Implement this by
checking the current state of the AtkObject.

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2012-12-09  Alejandro Piñeiro  <apinheiro@igalia.com>
+
+        [GTK] lack of implemention of AccessibilityUIElementGtk::isSelectable and AccessibilityUIElementGtk::isMultiSelectable
+        https://bugs.webkit.org/show_bug.cgi?id=104481
+
+        Reviewed by Martin Robinson.
+
+        * DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:
+        (AccessibilityUIElement::isSelectable): Implement this by checking
+        the current state of the AtkObject.
+        (AccessibilityUIElement::isMultiSelectable): Implement this by
+        checking the current state of the AtkObject.
+
 2012-12-09  Joanmarie Diggs  <jdiggs@igalia.com>
 

trunk/Tools/DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp

@@ -954 +954 @@
 bool AccessibilityUIElement::isSelectable() const
 {
-    // FIXME: implement
-    return false;
+    return checkElementState(m_element, ATK_STATE_SELECTABLE);
 }
 
 bool AccessibilityUIElement::isMultiSelectable() const
 {
-    // FIXME: implement
-    return false;
+    return checkElementState(m_element, ATK_STATE_MULTISELECTABLE);
 }