Commit aabe1570 authored by Katie Dektar's avatar Katie Dektar Committed by Commit Bot

Ensure focus ring is cleared in stopAll_ even if speech was not on.

This fixes a bug in which the focus ring could remain on the screen
if the user released the 'search' key before letting go of the
mouse button.

Bug: 770881
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I11dd444aaa03e7e7fff08d77ed6e18ac79c8b895
Reviewed-on: https://chromium-review.googlesource.com/696188
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: default avatarDominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506977}
parent baaed53a
...@@ -350,9 +350,13 @@ SelectToSpeak.prototype = { ...@@ -350,9 +350,13 @@ SelectToSpeak.prototype = {
* Stop speech. If speech was in-progress, the interruption * Stop speech. If speech was in-progress, the interruption
* event will be caught and clearFocusRingAndNode_ will be * event will be caught and clearFocusRingAndNode_ will be
* called, stopping visual feedback as well. * called, stopping visual feedback as well.
* If speech was not in progress, i.e. if the user was drawing
* a focus ring on the screen, this still clears the visual
* focus ring.
*/ */
stopAll_: function() { stopAll_: function() {
chrome.tts.stop(); chrome.tts.stop();
this.clearFocusRing_();
}, },
/** /**
......
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