Title: [137111] trunk/Tools
Revision
137111
Author
[email protected]
Date
2012-12-09 23:28:02 -0800 (Sun, 09 Dec 2012)

Log 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 <[email protected]> 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.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (137110 => 137111)


--- trunk/Tools/ChangeLog	2012-12-10 06:54:30 UTC (rev 137110)
+++ trunk/Tools/ChangeLog	2012-12-10 07:28:02 UTC (rev 137111)
@@ -1,3 +1,16 @@
+2012-12-09  Alejandro Piñeiro  <[email protected]>
+
+        [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  <[email protected]>
 
         [GTK] accessibility/placeholder.html is failing

Modified: trunk/Tools/DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp (137110 => 137111)


--- trunk/Tools/DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp	2012-12-10 06:54:30 UTC (rev 137110)
+++ trunk/Tools/DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp	2012-12-10 07:28:02 UTC (rev 137111)
@@ -953,14 +953,12 @@
 
 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);
 }
 
 bool AccessibilityUIElement::isSelectedOptionActive() const
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to