inline alertNow into requestSpeech, see #37

phetsims · Feb 7, 2022 · d55f48a · d55f48a
1 parent 24b8543
commit d55f48a
Showing 1 changed file with 8 additions and 22 deletions.
diff --git a/js/accessibility/voicing/voicingManager.js b/js/accessibility/voicing/voicingManager.js
@@ -239,27 +239,6 @@ class VoicingManager extends Announcer {
     queue.splice( index, 1 );
   }
 
-  /**
-   * Ask the synth to speak an utterance now (synchronously).
-   * @private
-   *
-   * @param {SpeechSynthesisUtterance} speechSynthesisUtterance
-   */
-  alertNow( speechSynthesisUtterance ) {
-    const synth = voicingManager.getSynth();
-    if ( synth ) {
-
-      // Signify to the utterance-queue that we cannot speak yet until this utterance has finished
-      this.readyToSpeak = false;
-
-      synth.speak( speechSynthesisUtterance );
-
-      if ( !this.hasSpoken ) {
-        this.hasSpoken = true;
-      }
-    }
-  }
-
   /**
    * @override
    * @private
@@ -434,7 +413,14 @@ class VoicingManager extends Announcer {
     // but the error event does. In this case signify that speaking has ended.
     speechSynthUtterance.addEventListener( 'error', endListener );
 
-    this.alertNow( speechSynthUtterance );
+    // Signify to the utterance-queue that we cannot speak yet until this utterance has finished
+    this.readyToSpeak = false;
+
+    this.getSynth().speak( speechSynthUtterance );
+
+    if ( !this.hasSpoken ) {
+      this.hasSpoken = true;
+    }
   }
 
   /**