aria-level is not exposed correctly for html header(h1...h6) tags.

aria-level is not exposed correctly for html header(h1...h6) tags.

This CL will match the implementation on Firefox.

Content:
<h1>Level 1</h1>
<h1 aria-level="2">Level 2</h1>
<h3 aria-level="4">Level 4</h3>
<h4 aria-level="2" role="heading">Level 2</h4>

AX Tree
ROLE_SYSTEM_DOCUMENT READONLY FOCUSABLE
    IA2_ROLE_HEADING name='Level 1' level:1
        ROLE_SYSTEM_STATICTEXT name='Level 1'
    IA2_ROLE_HEADING name='Level 2' level:2
        ROLE_SYSTEM_STATICTEXT name='Level 2'
    IA2_ROLE_HEADING name='Level 4' level:4
        ROLE_SYSTEM_STATICTEXT name='Level 4'
    IA2_ROLE_HEADING name='Level 2' level:2
        ROLE_SYSTEM_STATICTEXT name='Level 2'

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

[shreeramk, 2014-11-28 08:33:50]

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

[shreeramk, 2014-11-28 08:33:50]

Please review!

Thanks :)

[dmazzoni, 2014-11-29 07:44:36]

lgtm

Great. Please add a test for this in a follow-up change.

Do you think we should reject any value that's outside the range 1...6? Does the
spec allow aria-level=999?

[dmazzoni, 2014-11-29 07:44:39]

The CQ bit was checked by dmazzoni@chromium.org

[commit-bot: I haz the power, 2014-11-29 07:45:20]

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

[commit-bot: I haz the power, 2014-11-29 08:20:30]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=186200

[shreeramk, 2014-11-30 06:00:29]

On 2014/11/29 07:44:36, dmazzoni wrote:
> lgtm
> 
> Great. Please add a test for this in a follow-up change.
> 
> Do you think we should reject any value that's outside the range 1...6? Does
the
> spec allow aria-level=999?

The spec says for MSAA+ UIA:
When used on a heading, expose StyleId_Headingn where n is the heading level.
UIA supports up to heading level 9.

[dmazzoni, 2014-12-01 06:56:01]

On 2014/11/30 06:00:29, shreeramk wrote:
> The spec says for MSAA+ UIA:
> When used on a heading, expose StyleId_Headingn where n is the heading level.
> UIA supports up to heading level 9.

OK, I'd like a patch that limits the heading level to 1 - 9 if you want.

[shreeramk, 2014-12-01 06:58:03]

On 2014/12/01 06:56:01, dmazzoni wrote:
> On 2014/11/30 06:00:29, shreeramk wrote:
> > The spec says for MSAA+ UIA:
> > When used on a heading, expose StyleId_Headingn where n is the heading
level.
> > UIA supports up to heading level 9.
> 
> OK, I'd like a patch that limits the heading level to 1 - 9 if you want.

Sure, I ll submit a patch to limit aria heading level.