Bug 133519 - [GTK] Layout Test accessibility/table-fallback-roles-expose-element-attributes.html is failing
Summary: [GTK] Layout Test accessibility/table-fallback-roles-expose-element-attribute...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
: 133508 (view as bug list)
Depends on:
Blocks:
 
Reported: 2014-06-04 11:18 PDT by Carlos Alberto Lopez Perez
Modified: 2014-07-22 06:18 PDT (History)
12 users (show)

See Also:


Attachments
Path proposal (5.88 KB, patch)
2014-06-05 03:58 PDT, Mario Sanchez Prada
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-02 for mac-mountainlion (503.91 KB, application/zip)
2014-06-05 05:25 PDT, Build Bot
no flags Details
Path proposal (5.87 KB, patch)
2014-06-05 05:35 PDT, Mario Sanchez Prada
no flags Details | Formatted Diff | Diff
Patch proposal (6.76 KB, patch)
2014-06-06 05:33 PDT, Mario Sanchez Prada
cfleizach: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2014-06-04 11:18:09 PDT
The following layout test added on r169427 <http://trac.webkit.org/r169427> is failing on GTK since it was added.

accessibility/table-fallback-roles-expose-element-attributes.html

The diff is the following:

--- /home/clopez/webkit/webkit/layout-test-results/accessibility/table-fallback-roles-expose-element-attributes-expected.txt
+++ /home/clopez/webkit/webkit/layout-test-results/accessibility/table-fallback-roles-expose-element-attributes-actual.txt
@@ -6,11 +6,11 @@
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
-cellChild: AXRole: AXStaticText, AXValue: item1
+cellChild: , 
 
-cellChild: AXRole: AXStaticText, AXValue: item1
+cellChild: , 
 
-cellChild: AXRole: AXStaticText, AXValue: item1
+cellChild: , 
 
 PASS successfullyParsed is true
Comment 1 Mario Sanchez Prada 2014-06-05 02:53:50 PDT
The problem here is that we don't expose the text in a cell as a child element for ATK, so that's why we can't anything printed when we navigate down to that level.

In other words, in the Mac the hiearchy is like this

  Grid
    |--Cell
    |    |--text
    |--Cell
    |    |--text
   ...  ...

In ATK, however, it's like this:

  Grid
    |--Cell
    |--Cell
   ...  ...

...and you get the text in the cell by quering the AX object for that cell (which implements AtkText).

Because of that reason, the following code can't output anything reasonable for ATK:

  if (window.accessibilityController) {
      for (var k = 1; k < 4; k++) {
         var grid = accessibilityController.accessibleElementById("grid" + k);
         var cellChild = grid.cellForColumnAndRow(0, 0).childAtIndex(0);
         debug("cellChild: " + cellChild.role + ", " + cellChild.stringValue + "\n");
      }
  }

I think an improvement could be to do something like this:

  if (window.accessibilityController) {
      for (var k = 1; k < 4; k++) {
          var grid = accessibilityController.accessibleElementById("grid" + k);
          var cell = grid.cellForColumnAndRow(0, 0);
          debug("cell: " + cell.role + ", " + cell.stringValue + "\n");
          if (accessibilityController.platformName != "atk") {
              var cellChild = cell.childAtIndex(0);
              debug("cellChild: " + cellChild.role + ", " + cellChild.stringValue + "\n");
          }
      }
  }

...and then update the expectations accordingly (probably with the help of the EWS, to know what the Mac would print there)
Comment 2 Mario Sanchez Prada 2014-06-05 03:58:39 PDT
Created attachment 232535 [details]
Path proposal

Attaching patch proposal but not asking for review yet since I'm mostly interested on seeing what the Mac EWS bots have to say about my guessed output
Comment 3 Build Bot 2014-06-05 05:24:59 PDT
Comment on attachment 232535 [details]
Path proposal

Attachment 232535 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/4519381602140160

New failing tests:
accessibility/table-fallback-roles-expose-element-attributes.html
Comment 4 Build Bot 2014-06-05 05:25:03 PDT
Created attachment 232538 [details]
Archive of layout-test-results from webkit-ews-02 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-02  Port: mac-mountainlion  Platform: Mac OS X 10.8.5
Comment 5 Mario Sanchez Prada 2014-06-05 05:35:10 PDT
Created attachment 232539 [details]
Path proposal

Now making the Mac bots happy
Comment 6 Mario Sanchez Prada 2014-06-06 05:33:40 PDT
Created attachment 232613 [details]
Patch proposal

Forgot to remove the failure test expectation before, so this should be the final patch

Please review, thanks
Comment 7 chris fleizach 2014-06-06 08:45:33 PDT
Comment on attachment 232613 [details]
Patch proposal

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

> LayoutTests/ChangeLog:12
> +        Updated text to print information of cells fro every platform but

cells fro -> cells from
Comment 8 Mario Sanchez Prada 2014-06-06 09:14:28 PDT
(In reply to comment #7)
> (From update of attachment 232613 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=232613&action=review
> 
> > LayoutTests/ChangeLog:12
> > +        Updated text to print information of cells fro every platform but
> 
> cells fro -> cells from

Will change before landing.

Thanks
Comment 9 Mario Sanchez Prada 2014-06-06 09:20:29 PDT
Committed r169646: <http://trac.webkit.org/changeset/169646>
Comment 10 Michal Pakula vel Rutka 2014-07-22 06:18:26 PDT
*** Bug 133508 has been marked as a duplicate of this bug. ***