Adding DumpAccessibilityTree Tests (7 of 20)

Adding DumpAccessibilityTree Tests (7 of 20)

HTML Tags: abbr, button, col, figure, meta, meter, input type(search, time)
Aria role: log, marquee
Aria state/property: aria-orientation(horizontal, vertical), aria-sort(Ascending, Descending, Other)

BUG=170579
Committed: https://crrev.com/235fd06a8b60d37606ac7c6f40e8ece9a0bc607a
Cr-Commit-Position: refs/heads/master@{#296157}

[shreeramk, 2014-09-16 08:13:41]

shreeram.k@samsung.com changed reviewers:
+ aboxhall@chromium.org, dmazzoni@chromium.org

[shreeramk, 2014-09-16 08:13:43]

Please review. Thanks!!

[shreeramk, 2014-09-19 06:40:53]

On 2014/09/16 08:13:43, shreeram.k wrote:
> Please review. Thanks!!

@dmazzoni: PTAL

[dmazzoni, 2014-09-22 21:16:36]

Looks great, I only see two small issues.

https://codereview.chromium.org/569203002/diff/60001/content/test/data/access...
File content/test/data/accessibility/aria-log.html (right):

https://codereview.chromium.org/569203002/diff/60001/content/test/data/access...
content/test/data/accessibility/aria-log.html:8: <div id="liveregion" role="log"
aria-live="polite">
You shouldn't have to add aria-live="polite" to either this, or marquee, or any
other live region. The live status is implicit.

https://codereview.chromium.org/569203002/diff/60001/content/test/data/access...
File content/test/data/accessibility/aria-orientation-expected-win.txt (right):

https://codereview.chromium.org/569203002/diff/60001/content/test/data/access...
content/test/data/accessibility/aria-orientation-expected-win.txt:2:
ROLE_SYSTEM_SCROLLBAR
It should have IA2_STATE_VERTICAL

[shreeramk, 2014-09-23 05:30:10]

PTAL!

I updated aria-log.html and aria-marquee.html.

https://codereview.chromium.org/569203002/diff/60001/content/test/data/access...
File content/test/data/accessibility/aria-orientation-expected-win.txt (right):

https://codereview.chromium.org/569203002/diff/60001/content/test/data/access...
content/test/data/accessibility/aria-orientation-expected-win.txt:2:
ROLE_SYSTEM_SCROLLBAR
On 2014/09/22 21:16:36, dmazzoni wrote:
> It should have IA2_STATE_VERTICAL

I checked the accessibility tree in chrome://accessibility url , its not showing
the proper IA2_STATE_VERTICAL or _HORIZONTAL.

I think it might be an issue on windows. I ll skip.

And I'd file a bug for this and try to fix it.

[dmazzoni, 2014-09-23 05:38:43]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2014-09-23 05:38:43]

lgtm

Thanks!

[commit-bot: I haz the power, 2014-09-23 05:39:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/569203002/80001

[commit-bot: I haz the power, 2014-09-23 05:43:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-23 05:43:22]

Try jobs failed on following builders:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/59349)
  ios_dbg_simulator on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[shreeramk, 2014-09-23 06:13:11]

The CQ bit was checked by shreeram.k@samsung.com

[commit-bot: I haz the power, 2014-09-23 06:13:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/569203002/100001

[commit-bot: I haz the power, 2014-09-23 08:30:38]

Committed patchset #6 (id:100001) as 001d1602f251c68fd0b09e5b153435e2d8380f26

[commit-bot: I haz the power, 2014-09-23 08:31:17]

Patchset 6 (id:??) landed as
https://crrev.com/235fd06a8b60d37606ac7c6f40e8ece9a0bc607a
Cr-Commit-Position: refs/heads/master@{#296157}