The children of non-editable combo boxes, search boxes and text fields should not be hidden

The children of non-editable combo boxes, search boxes and text fields specified via e.g. aria-owns should not be hidden from platform trees.

Children of combo boxes, search boxes and text fields might not always be part of the shadow DOM, especially when no <input> or contentEditable . tags have been usedThey might be a list of combo box items specified via aria-owns.
data:text/html,<div role="combobox" aria-owns="list" onKeyDown="this.setAttribute('aria-activedescendant', 'item');">content</div><ul id="list"><li id="item">item</li></ul>
BUG=620432
R=dmazzoni@chromium.org
TESTED=manually with Jaws NVDA and Voiceover

Committed: https://crrev.com/734e21b735b81cc5c8ac04a98e330a9e7f51decd
Cr-Commit-Position: refs/heads/master@{#404478}

[nektarios, 2016-07-01 18:36:46]

Description was changed from

==========
The children of combo boxes, search boxes and text fields specified via
aria-owns should not be hidden from platform trees.

Children of combo boxes, search boxes and text fields might not always be part
of the shadow DOM. They might be a list of combo box items specified via
aria-owns.
As an example, the list of items associated with the following combo box was not
exposed in platform trees:
data:text/html,<div contentEditable="true" role="combobox" aria-owns="list"
onKeyDown="this.setAttribute('aria-activedescendant', 'item');">content</div><ul
id="list"><li id="item">item</li></ul>
BUG=620432
R=dmazzoni@chromium.org
TESTED=manually with Jaws NVDA and Voiceover
==========

to

==========
The children of non-editable combo boxes, search boxes and text fields specified
via e.g. aria-owns should not be hidden from platform trees.

Children of combo boxes, search boxes and text fields might not always be part
of the shadow DOM, especially when no <input> or contentEditable . tags have
been usedThey might be a list of combo box items specified via aria-owns.
data:text/html,<div role="combobox" aria-owns="list"
onKeyDown="this.setAttribute('aria-activedescendant', 'item');">content</div><ul
id="list"><li id="item">item</li></ul>
BUG=620432
R=dmazzoni@chromium.org
TESTED=manually with Jaws NVDA and Voiceover
==========

[dmazzoni, 2016-07-01 19:04:42]

lgtm

This might affect some tests but definitely looks like the right thing

[nektarios, 2016-07-01 19:46:02]

The CQ bit was checked by nektar@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-01 19:46:17]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

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

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

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

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[nektarios, 2016-07-01 21:38:07]

The CQ bit was checked by nektar@chromium.org

[nektarios, 2016-07-01 21:38:08]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2120523002/#ps60001
(title: "Fixed tests.")

[commit-bot: I haz the power, 2016-07-01 21:38:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-01 22:41:57]

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

[commit-bot: I haz the power, 2016-07-01 22:41:58]

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

[nektarios, 2016-07-01 22:53:52]

The CQ bit was checked by nektar@chromium.org

[nektarios, 2016-07-01 22:53:52]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2120523002/#ps80001
(title: "Replaced test expectations.")

[commit-bot: I haz the power, 2016-07-01 22:54:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-01 22:59:37]

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

[commit-bot: I haz the power, 2016-07-01 22:59:39]

Try jobs failed on following builders:
  ios-device-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)

[nektarios, 2016-07-01 23:17:49]

The CQ bit was checked by nektar@chromium.org

[commit-bot: I haz the power, 2016-07-01 23:18:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-02 00:10:51]

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

[commit-bot: I haz the power, 2016-07-02 00:10:52]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[nektarios, 2016-07-02 00:51:01]

The CQ bit was checked by nektar@chromium.org

[nektarios, 2016-07-02 00:51:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2120523002/#ps100001
(title: "Multi-line text fields should drop their children on Android.")

[commit-bot: I haz the power, 2016-07-02 00:51:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-02 01:49:53]

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

[commit-bot: I haz the power, 2016-07-02 01:49:54]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[nektarios, 2016-07-06 01:22:27]

The CQ bit was checked by nektar@chromium.org

[commit-bot: I haz the power, 2016-07-06 01:22:40]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-06 03:21:49]

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

[commit-bot: I haz the power, 2016-07-06 03:21:50]

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

[nektarios, 2016-07-06 17:03:09]

The CQ bit was checked by nektar@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-06 17:03:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-06 18:38:24]

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

[commit-bot: I haz the power, 2016-07-06 18:38:25]

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

[nektarios, 2016-07-06 22:11:32]

The CQ bit was checked by nektar@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-06 22:12:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-06 23:35:52]

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

[commit-bot: I haz the power, 2016-07-06 23:35:53]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[nektarios, 2016-07-06 23:45:20]

The CQ bit was checked by nektar@chromium.org

[nektarios, 2016-07-06 23:45:21]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2120523002/#ps180001
(title: "Fixed datetime test on Windows.")

[commit-bot: I haz the power, 2016-07-06 23:46:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-07 01:32:15]

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

[commit-bot: I haz the power, 2016-07-07 01:32:17]

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

[nektarios, 2016-07-07 14:50:14]

The CQ bit was checked by nektar@chromium.org

[nektarios, 2016-07-07 14:50:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2120523002/#ps200001
(title: "Fixed combo box unit test on Windows.")

[commit-bot: I haz the power, 2016-07-07 14:50:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-07 16:12:13]

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

[commit-bot: I haz the power, 2016-07-07 16:12:14]

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

[nektarios, 2016-07-07 23:07:55]

The CQ bit was checked by nektar@chromium.org

[nektarios, 2016-07-07 23:07:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2120523002/#ps220001
(title: "Fixed Windows unit tests.")

[commit-bot: I haz the power, 2016-07-07 23:08:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

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

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

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

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

[nektarios, 2016-07-08 16:41:38]

The CQ bit was checked by nektar@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-08 16:41:55]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-08 20:05:29]

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

[commit-bot: I haz the power, 2016-07-08 20:05:31]

Dry run: This issue passed the CQ dry run.

[nektarios, 2016-07-08 20:12:39]

The CQ bit was checked by nektar@chromium.org

[nektarios, 2016-07-08 20:12:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2120523002/#ps240001
(title: "Fixed Windows browser test.")

[commit-bot: I haz the power, 2016-07-08 20:13:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-08 20:18:02]

Committed patchset #13 (id:240001)

[commit-bot: I haz the power, 2016-07-08 20:18:05]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-08 20:19:16]

Description was changed from

==========
The children of non-editable combo boxes, search boxes and text fields specified
via e.g. aria-owns should not be hidden from platform trees.

Children of combo boxes, search boxes and text fields might not always be part
of the shadow DOM, especially when no <input> or contentEditable . tags have
been usedThey might be a list of combo box items specified via aria-owns.
data:text/html,<div role="combobox" aria-owns="list"
onKeyDown="this.setAttribute('aria-activedescendant', 'item');">content</div><ul
id="list"><li id="item">item</li></ul>
BUG=620432
R=dmazzoni@chromium.org
TESTED=manually with Jaws NVDA and Voiceover
==========

to

==========
The children of non-editable combo boxes, search boxes and text fields specified
via e.g. aria-owns should not be hidden from platform trees.

Children of combo boxes, search boxes and text fields might not always be part
of the shadow DOM, especially when no <input> or contentEditable . tags have
been usedThey might be a list of combo box items specified via aria-owns.
data:text/html,<div role="combobox" aria-owns="list"
onKeyDown="this.setAttribute('aria-activedescendant', 'item');">content</div><ul
id="list"><li id="item">item</li></ul>
BUG=620432
R=dmazzoni@chromium.org
TESTED=manually with Jaws NVDA and Voiceover

Committed: https://crrev.com/734e21b735b81cc5c8ac04a98e330a9e7f51decd
Cr-Commit-Position: refs/heads/master@{#404478}
==========

[commit-bot: I haz the power, 2016-07-08 20:19:18]

Patchset 13 (id:??) landed as
https://crrev.com/734e21b735b81cc5c8ac04a98e330a9e7f51decd
Cr-Commit-Position: refs/heads/master@{#404478}