Commit 6b6ca1e4 authored by rbpotter's avatar rbpotter Committed by Chromium LUCI CQ

Print Preview plugin: Remove aria-live attribute

This largely seems to create a bunch of noise for screenreaders whenever
the preview is reloaded, unless another element is explicitly focused.
The message is also generally unhelpful, e.g. Chrome Vox reads out
the name of the underlying WebContent, iframe, and the names of all the
plugin buttons, which seems unlikely to be useful.

Bug: 987370
Change-Id: I97b7d25b859e50bd51b82ffa6760571ae8319a73
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2573881Reviewed-by: default avatarJohn Lee <johntlee@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#833758}
parent 47a8e1b8
......@@ -115,8 +115,6 @@ export class PluginProxyImpl {
this.plugin_ = /** @type {PDFPlugin} */ (
PDFCreateOutOfProcessPlugin(srcUrl, 'chrome://print/pdf'));
this.plugin_.classList.add('preview-area-plugin');
this.plugin_.setAttribute('aria-live', 'polite');
this.plugin_.setAttribute('aria-atomic', 'true');
// NOTE: The plugin's 'id' field must be set to 'pdf-viewer' since
// chrome/renderer/printing/print_render_frame_helper.cc actually
// references it.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment