Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5844)

Unified Diff: chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js

Issue 2563173002: Fix issues with sending tts queue lots of utterances (Closed)
Patch Set: Fix issues with sending tts queue lots of utterances Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js
index 29a9c6f869de44e532d78c1d28a6418b76302a67..7c0eb03eb6a4938f2f48b0cd88ffdfa7645bd46b 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js
@@ -134,6 +134,10 @@ LiveRegions.prototype = {
return;
}
+ // Alerts should be announced as a result of focus.
+ if (node.role == RoleType.alert)
+ return;
+
var range = cursors.Range.fromNode(node);
var output = new Output();
if (opt_prependFormatStr)

Powered by Google App Engine
This is Rietveld 408576698