Fix menu list and list box screen reader accessibility.

Fix menu list and list box screen reader accessibility.

The most important change here is getting rid of code in
renderer_accessibility.cc that recreated the subtree whenever
AXSelectedChildrenChanged was fired. This is no longer
necessary because Blink now fires the right events on each
individual option in a menu list or list box.

The rest of the changes just make sure the exact attributes
and events match the spec and match what Firefox and
Safari do when interacting with these elements, including
adding some good tests for those.

BUG=323462
Committed: https://crrev.com/9dbb3741802d2a5a3f6370fef9d712b077fa4b86
Cr-Commit-Position: refs/heads/master@{#326755}

[dmazzoni, 2015-04-06 07:09:58]

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

[dmazzoni, 2015-04-06 07:09:58]

Depends on this Blink change in order for the tests to pass:

https://codereview.chromium.org/1059503002/

[je_julie(Not used), 2015-04-06 10:31:49]

lgtm

https://codereview.chromium.org/1057083002/diff/40001/content/browser/accessi...
File content/browser/accessibility/browser_accessibility_manager_mac.mm (right):

https://codereview.chromium.org/1057083002/diff/40001/content/browser/accessi...
content/browser/accessibility/browser_accessibility_manager_mac.mm:66:
node->GetParent()->GetRole() == ui::AX_ROLE_LIST_BOX) {
For this condition, how about extending IsListBoxOptionOrMenuListOption() in
browser_accessibility_win.h with 'event_type == ui::AX_EVENT_FOCUS' as moving it
to browser_accessibility.h?

https://codereview.chromium.org/1057083002/diff/40001/content/browser/accessi...
File content/browser/accessibility/browser_accessibility_win.cc (right):

https://codereview.chromium.org/1057083002/diff/40001/content/browser/accessi...
content/browser/accessibility/browser_accessibility_win.cc:3309: if
(multiselect) {
Do we need this variable? it doesn't seem to be used after condition check.

[dmazzoni, 2015-04-23 18:19:00]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2015-04-23 18:19:00]

The patchset sent to the CQ was uploaded after l-g-t-m from
je_julie.kim@samsung.com
Link to the patchset: https://codereview.chromium.org/1057083002/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2015-04-23 18:22:32]

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

[commit-bot: I haz the power, 2015-04-23 19:51:17]

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

[commit-bot: I haz the power, 2015-04-23 19:51:18]

Try jobs failed on following builders:
  win8_chromium_rel on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)

[dmazzoni, 2015-04-23 22:59:10]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2015-04-23 22:59:10]

The patchset sent to the CQ was uploaded after l-g-t-m from
je_julie.kim@samsung.com
Link to the patchset: https://codereview.chromium.org/1057083002/#ps80001
(title: "Fix a few tests")

[commit-bot: I haz the power, 2015-04-23 22:59:49]

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

[commit-bot: I haz the power, 2015-04-24 02:36:32]

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

[commit-bot: I haz the power, 2015-04-24 02:36:33]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[dmazzoni, 2015-04-24 07:16:55]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2015-04-24 07:16:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
je_julie.kim@samsung.com
Link to the patchset: https://codereview.chromium.org/1057083002/#ps100001
(title: "Fix Mac test")

[commit-bot: I haz the power, 2015-04-24 07:17:11]

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

[commit-bot: I haz the power, 2015-04-24 07:23:54]

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

[commit-bot: I haz the power, 2015-04-24 07:23:55]

Try jobs failed on following builders:
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dmazzoni, 2015-04-24 07:51:26]

The CQ bit was checked by dmazzoni@chromium.org

[commit-bot: I haz the power, 2015-04-24 07:51:43]

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

[commit-bot: I haz the power, 2015-04-24 08:22:28]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-04-24 08:23:31]

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