The name attribute can be from contents for listitem role(re-land)

The name attribute can be from contents for listitem role
according to http://www.w3.org/WAI/PF/aria/roles#listitem.

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

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

[je_julie(Not used), 2014-11-02 16:10:32]

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

[je_julie(Not used), 2014-11-02 16:17:21]

Dominic,
This patch makes litsItemRole have textUnderElement() for title.
Before this patch is landed, TCs related to ListItemRole should be disabled.
That is handled at https://codereview.chromium.org/696133002/.
The new test results are at https://codereview.chromium.org/685993005/. 
PTAL.

[dmazzoni, 2014-11-03 16:47:23]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2014-11-03 16:47:23]

lgtm

[commit-bot: I haz the power, 2014-11-03 16:48:30]

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

[commit-bot: I haz the power, 2014-11-03 19:02:49]

Committed patchset #1 (id:1) as 184795

[Hajime Morrita, 2014-11-03 21:52:34]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/703473002/ by morrita@chromium.org.

The reason for reverting is: This seems breaking AccessibilityAriaList in
content_browsertests on Mac.
http://build.chromium.org/p/chromium.webkit/builders/Mac10.6%20Tests/builds/2...
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#master=ch...
.

[je_julie(Not used), 2014-11-04 02:52:38]

On 2014/11/03 21:52:34, morrita wrote:
> A revert of this CL (patchset #1 id:1) has been created in
> https://codereview.chromium.org/703473002/ by mailto:morrita@chromium.org.
> 
> The reason for reverting is: This seems breaking AccessibilityAriaList in
> content_browsertests on Mac.
>
http://build.chromium.org/p/chromium.webkit/builders/Mac10.6%20Tests/builds/2...
>
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#master=ch...
> .

I re-opened this issue.

[je_julie(Not used), 2014-11-08 14:29:51]

Hi Dominic,

Chrome with this patch is same as working from FF.
I checked Screen Reader working with FF on Linux(orca) and Windows(NVDA) and
Chrome on Windows.
Scren Reader spoke content just once when I moved focus to item list.

For instance,
I loaded content/test/data/accessibility/contenteditable-descendants.html
,moved mouse to list item, "Editable list item" and then
Screen Reader spoke "Editable list item" just once.
I also moved mouse to the list item from upper line or from lower line and
Screen Reader spoke "Editable list item" just once.

I think Screen Reader checks that working and 
we should follow the spec.

*Before landing this patch, please check CL on
https://codereview.chromium.org/709243002/.
There is additionaly disabled TC.
After landing this patch, I'll enable them.

For CL,https://codereview.chromium.org/685993005/, related to expected test
results with this patch,
I'll request a review after landing this patch.

Could you take a look?

[dmazzoni, 2014-11-10 17:01:54]

lgtm

[je_julie(Not used), 2014-11-11 02:36:27]

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

[commit-bot: I haz the power, 2014-11-11 02:37:01]

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

[commit-bot: I haz the power, 2014-11-11 02:37:58]

Committed patchset #1 (id:1) as 185092