AXMenuListOption should respect role attribute.

This patch makes AXMenuListOption respect role attribute.

Because AXMenuListOption is a type of AXMockObject,
it can't have determineAccessibilityRole() from AXNodeObject
and role attribute is checked at roleValue().

BUG=467434
Committed: https://crrev.com/b592cbe206c36811da9c6f9af13f2c896ac30384
Cr-Commit-Position: refs/heads/master@{#370336}

[je_julie(Not used), 2016-01-05 13:13:51]

Description was changed from

==========
role attr

BUG=
==========

to

==========
This patch makes AXMenuListOption respect role attribute.

Because AXMenuListOption is a type of AXMockObject,
it can't have determineAccessibilityRole() from AXNodeObject
and role attribute is checked at roleValue().

BUG=467434
==========

[je_julie(Not used), 2016-01-05 14:10:15]

je_julie.kim@samsung.com changed reviewers:
+ dmazzoni@chromium.org

[je_julie(Not used), 2016-01-05 14:10:15]

Hi Dominic,
Happy New Year!

Normally we decide role with determineAccessibilityRole()
but AXMenuListOption is a type of AXMockObject and it can't have
determineAccessibilityRole().
So, I make roleValue() check role attribute for AXMenuListOption.
PTAL.

[je_julie(Not used), 2016-01-20 01:24:42]

On 2016/01/05 14:10:15, je_julie wrote:
> Hi Dominic,
> Happy New Year!
> 
> Normally we decide role with determineAccessibilityRole()
> but AXMenuListOption is a type of AXMockObject and it can't have
> determineAccessibilityRole().
> So, I make roleValue() check role attribute for AXMenuListOption.
> PTAL.

ping?

[dmazzoni, 2016-01-20 06:03:17]

lgtm

Hi! Sorry I missed this earlier, looks good!

[je_julie(Not used), 2016-01-20 07:11:52]

The CQ bit was checked by je_julie.kim@samsung.com

[commit-bot: I haz the power, 2016-01-20 07:12:07]

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

[commit-bot: I haz the power, 2016-01-20 08:43:35]

Description was changed from

==========
This patch makes AXMenuListOption respect role attribute.

Because AXMenuListOption is a type of AXMockObject,
it can't have determineAccessibilityRole() from AXNodeObject
and role attribute is checked at roleValue().

BUG=467434
==========

to

==========
This patch makes AXMenuListOption respect role attribute.

Because AXMenuListOption is a type of AXMockObject,
it can't have determineAccessibilityRole() from AXNodeObject
and role attribute is checked at roleValue().

BUG=467434
==========

[commit-bot: I haz the power, 2016-01-20 08:43:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-20 08:45:07]

Description was changed from

==========
This patch makes AXMenuListOption respect role attribute.

Because AXMenuListOption is a type of AXMockObject,
it can't have determineAccessibilityRole() from AXNodeObject
and role attribute is checked at roleValue().

BUG=467434
==========

to

==========
This patch makes AXMenuListOption respect role attribute.

Because AXMenuListOption is a type of AXMockObject,
it can't have determineAccessibilityRole() from AXNodeObject
and role attribute is checked at roleValue().

BUG=467434

Committed: https://crrev.com/b592cbe206c36811da9c6f9af13f2c896ac30384
Cr-Commit-Position: refs/heads/master@{#370336}
==========

[commit-bot: I haz the power, 2016-01-20 08:45:07]

Patchset 1 (id:??) landed as
https://crrev.com/b592cbe206c36811da9c6f9af13f2c896ac30384
Cr-Commit-Position: refs/heads/master@{#370336}