Adding MeterRole in blink to support MSAA+IA2 and AX roles for meter tag in Chromium.

Adding MeterRole in blink to support MSAA+IA2 and AX roles for meter tag in Chromium.

According to latest editor's draft http://rawgit.com/w3c/html-api-map/master/index.html
meter tag should have MSAA+IA2 Role as ROLE_SYSTEM_PROGRESSBAR and
AX role as AXGroup and Role description as "group"

BUG=421376
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183638

[shreeramk, 2014-10-13 16:55:57]

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

[shreeramk, 2014-10-13 16:55:57]

Please review.

This would be followed up by Chromium side CL which will take care of exposing
meter tag with correct MSAA+IA2 and AX roles.

Thanks!

[dmazzoni, 2014-10-13 17:44:00]

I'm suspicious of the spec - why should it be AXGroup on Mac? That looks like a
mistake.

I think we should use the existing ProgressIndicatorRole and file a spec bug.

[shreeramk, 2014-10-13 18:03:42]

On 2014/10/13 17:44:00, dmazzoni wrote:
> I'm suspicious of the spec - why should it be AXGroup on Mac? That looks like
a
> mistake.
> 
> I think we should use the existing ProgressIndicatorRole and file a spec bug.

ARIA role progressbar - An element that displays the progress status for tasks
that take a long time.

I think <progress> tag(which represents the progress of a task) would correspond
with ARIA role progressbar. Meter tag generally doesn't show the progress,
rather it shows a scalar measurement.

[dmazzoni, 2014-10-13 21:42:59]

On 2014/10/13 18:03:42, shreeram.k wrote:
> I think <progress> tag(which represents the progress of a task) would
correspond
> with ARIA role progressbar. Meter tag generally doesn't show the progress,
> rather it shows a scalar measurement.

Fair enough - it's possible we may want to expose the difference, even though
there isn't a a way to now. Perhaps AXRoleDescription could be different, for
example, even though it's not in the spec.

Go ahead with this change, then - but please just copy the ARIA role progressbar
for the AX mappings on the Chromium side rather than AXGroup.

Thanks.

[dmazzoni, 2014-10-13 21:43:05]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2014-10-13 21:43:05]

lgtm

[commit-bot: I haz the power, 2014-10-13 21:44:19]

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

[commit-bot: I haz the power, 2014-10-13 23:58:30]

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

[commit-bot: I haz the power, 2014-10-13 23:58:31]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/31578)

[shreeramk, 2014-10-14 03:37:26]

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

[commit-bot: I haz the power, 2014-10-14 03:38:04]

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

[commit-bot: I haz the power, 2014-10-14 03:39:15]

Committed patchset #1 (id:1) as 183638