WebUI: Fix cr-policy-pref-indicator tooltip to pass accessibilty checks.

WebUI: Fix cr-policy-pref-indicator tooltip to pass accessibilty checks.

Per https://www.w3.org/TR/wai-aria/roles#tooltip the tooltip should be
always referenced by another element via an aria-describedby attribute.
This requires the tooltip itself to have an id.

The bug was uncovered by running MD Settings tests in Vulcanized mode, but was
always there.

BUG=673825
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2685933004
Cr-Commit-Position: refs/heads/master@{#449233}
Committed: https://chromium.googlesource.com/chromium/src/+/bf54c6ba3a28ace58768bd844cc01f1c6601ee25

[dpapad, 2017-02-09 03:00:02]

Description was changed from

==========
WebUI: Fix cr-policy-pref-indicator tooltip to pass accessibilty checks.


Per https://www.w3.org/TR/wai-aria/roles#tooltip the tooltip should be
always referenced by another element via an aria-describedby attribute.
This requires the tooltip itself to have an id.

The bug was uncovered by running MD Settings tests in Vulcanized mode, but was
always there.

BUG=673825
==========

to

==========
WebUI: Fix cr-policy-pref-indicator tooltip to pass accessibilty checks.


Per https://www.w3.org/TR/wai-aria/roles#tooltip the tooltip should be
always referenced by another element via an aria-describedby attribute.
This requires the tooltip itself to have an id.

The bug was uncovered by running MD Settings tests in Vulcanized mode, but was
always there.

BUG=673825
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2017-02-09 03:00:06]

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

[commit-bot: I haz the power, 2017-02-09 03:00:41]

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

[dpapad, 2017-02-09 03:05:22]

dpapad@chromium.org changed reviewers:
+ dbeam@chromium.org

[dpapad, 2017-02-09 03:05:23]

FYI this bug manifests in a non obvious way (see error below).

[4819:4819:0208/190433.788986:ERROR:web_ui_test_handler.cc(76)] Failed:
RUN_TEST_F("CrSettingsRouteTest","All")
TypeError: Cannot read property 'length' of null
    at axs.AuditRule.test [as test_] (axs_testing.js:2169:67)
    at axs.AuditRule.run (axs_testing.js:1706:10)
    at Object.axs.Audit.run (axs_testing.js:1843:17)
    at runAccessibilityAudit (test_api.js:965:34)
    at CrSettingsRouteTest.runAccessibilityAudit (test_api.js:371:12)
    at TestCase.runAccessibilityAudit (test_api.js:504:22)
    at test_api.js:1020:20
    at testDone (test_api.js:786:31)
    at Runner.<anonymous> (mocha_adapter.js:55:9)
    at Runner.EventEmitter.emit (mocha.js:363:20)

[dpapad, 2017-02-09 03:07:00]

Description was changed from

==========
WebUI: Fix cr-policy-pref-indicator tooltip to pass accessibilty checks.


Per https://www.w3.org/TR/wai-aria/roles#tooltip the tooltip should be
always referenced by another element via an aria-describedby attribute.
This requires the tooltip itself to have an id.

The bug was uncovered by running MD Settings tests in Vulcanized mode, but was
always there.

BUG=673825
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
WebUI: Fix cr-policy-pref-indicator tooltip to pass accessibilty checks.

Per https://www.w3.org/TR/wai-aria/roles#tooltip the tooltip should be
always referenced by another element via an aria-describedby attribute.
This requires the tooltip itself to have an id.

The bug was uncovered by running MD Settings tests in Vulcanized mode, but was
always there.

BUG=673825
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2017-02-09 03:07:00]

dpapad@chromium.org changed reviewers:
- dbeam@chromium.org

[Dan Beam, 2017-02-09 04:32:47]

dbeam@chromium.org changed reviewers:
+ dbeam@chromium.org

[Dan Beam, 2017-02-09 04:32:47]

lgtm

[commit-bot: I haz the power, 2017-02-09 05:57:03]

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

[commit-bot: I haz the power, 2017-02-09 05:57:05]

Dry run: This issue passed the CQ dry run.

[dpapad, 2017-02-09 08:01:00]

The CQ bit was checked by dpapad@chromium.org

[commit-bot: I haz the power, 2017-02-09 08:01: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, 2017-02-09 08:31:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486627260386390, "parent_rev":
"2193fbd54c6970038fbfb920b7521bbe0a5acc15", "commit_rev":
"bf54c6ba3a28ace58768bd844cc01f1c6601ee25"}

[commit-bot: I haz the power, 2017-02-09 08:41:13]

Description was changed from

==========
WebUI: Fix cr-policy-pref-indicator tooltip to pass accessibilty checks.

Per https://www.w3.org/TR/wai-aria/roles#tooltip the tooltip should be
always referenced by another element via an aria-describedby attribute.
This requires the tooltip itself to have an id.

The bug was uncovered by running MD Settings tests in Vulcanized mode, but was
always there.

BUG=673825
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
WebUI: Fix cr-policy-pref-indicator tooltip to pass accessibilty checks.

Per https://www.w3.org/TR/wai-aria/roles#tooltip the tooltip should be
always referenced by another element via an aria-describedby attribute.
This requires the tooltip itself to have an id.

The bug was uncovered by running MD Settings tests in Vulcanized mode, but was
always there.

BUG=673825
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2685933004
Cr-Commit-Position: refs/heads/master@{#449233}
Committed:
https://chromium.googlesource.com/chromium/src/+/bf54c6ba3a28ace58768bd844cc0...
==========

[commit-bot: I haz the power, 2017-02-09 08:41:14]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/bf54c6ba3a28ace58768bd844cc0...