Bug 141960

Summary: [GTK] Layout Test accessibility/roles-exposed.html is failing
Product: WebKit Reporter: Marcos Chavarría Teijeiro (irc: chavaone) <chavarria1991>
Component: AccessibilityAssignee: Joanmarie Diggs <jdiggs>
Status: RESOLVED FIXED    
Severity: Normal CC: a.badowski, aboxhall, apinheiro, cfleizach, cgarcia, clopez, commit-queue, dmazzoni, gustavo, jcraig, jdiggs, mario, samuel_white, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch for landing none

Description Marcos Chavarría Teijeiro (irc: chavaone) 2015-02-24 01:03:29 PST 
The accessibility/roles-exposed.html layout test is failing since r179922 (https://trac.webkit.org/changeset/r179922). 

This revision adds a default color chooser implementation so we probably need could update the expected file but I'm not sure.

The diff is the following:

--- /home/ch01/wk-tools/layout-test-results/accessibility/roles-exposed-expected.txt
+++ /home/ch01/wk-tools/layout-test-results/accessibility/roles-exposed-actual.txt
@@ -137,7 +137,7 @@
       AXRole: AXButton

 input[type='color']
-      AXRole: AXTextField
+      AXRole: AXColorWell

 input[type='checkbox']
       AXRole: AXCheckBox
Comment 1 Radar WebKit Bug Importer 2015-02-24 01:04:30 PST 
<rdar://problem/19934954>
Comment 2 Radar WebKit Bug Importer 2015-02-24 01:05:05 PST 
<rdar://problem/19934955>
Comment 3 Joanmarie Diggs 2015-02-24 06:10:34 PST 
I'll take a look at this one. But my guess is that the role should be a push button if the widget is a push button
Comment 4 Joanmarie Diggs 2015-02-24 08:42:12 PST 
Created attachment 247235 [details]
Patch
Comment 5 Joanmarie Diggs 2015-02-24 08:52:06 PST 
Andrzej: Just an FYI as you seem to be actively working in this area, but for EFL (right?).

The EFL test expectations already have the mapping to ATK_ROLE_COLOR_CHOOSER. But as you'll see here [1], a color chooser is a dialog (i.e. the dialog that appears when you activate the button). I wasn't sure if that mapping was deliberate/desired for your port.

As a related aside, we plan to add subrole support to ATK hopefully before too long to handle cases such as this (i.e. being able to say something is a specialized type of button, namely one that results in a color chooser appearing).

[1] https://developer.gnome.org/atk/unstable/AtkObject.html#AtkRole
Comment 6 Martin Robinson 2015-02-24 08:58:03 PST 
Comment on attachment 247235 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=247235&action=review

> Source/WebCore/accessibility/atk/WebKitAccessibleWrapperAtk.cpp:569
> +        // ATK_ROLE_COLOR_CHOOSER is defined as a dialog (i.e. it's what appears when you push the button)

Nit: Missing a period at the end of this comment.
Comment 7 Joanmarie Diggs 2015-02-24 09:08:38 PST 
Created attachment 247236 [details]
Patch for landing
Comment 8 WebKit Commit Bot 2015-02-24 09:20:17 PST 
Comment on attachment 247236 [details]
Patch for landing

Clearing flags on attachment: 247236

Committed r180566: <http://trac.webkit.org/changeset/180566>
Comment 9 WebKit Commit Bot 2015-02-24 09:20:22 PST 
All reviewed patches have been landed.  Closing bug.