Enqueue new text if added while the engine is busy

Calling enqueue() should be equivalent to say() when the engine
is Ready, otherwise it should enqueue the text. The old
implementation only enqueued the text when the engine was
in Speaking state, overriding the current utterance when the
engine was already synthesizing, or paused.

Adjust test to enqueue the next text chunk as soon as the
engine transitions away from the Ready state.

Pick-to: 6.7 6.6
Fixes: QTBUG-122884
Change-Id: I19518a92d1ae73b01dc3de1d9ae6178f5f55b3ad
Reviewed-by: Axel Spoerl <axel.spoerl@qt.io>

1 files changed, 10 insertions, 3 deletions

diff --git a/src/tts/qtexttospeech.cpp b/src/tts/qtexttospeech.cpp
index c938771..8018a8f 100644
--- a/src/tts/qtexttospeech.cpp
+++ b/src/tts/qtexttospeech.cpp
@@ -826,11 +826,18 @@ qsizetype QTextToSpeech::enqueue(const QString &utterance)
     if (!d->m_engine || utterance.isEmpty())
         return -1;
 
-    if (d->m_engine->state() == QTextToSpeech::Speaking) {
-        d->m_pendingUtterances.enqueue(utterance);
-    } else {
+    switch (d->m_engine->state()) {
+    case QTextToSpeech::Error:
+        return -1;
+    case QTextToSpeech::Ready:
         emit aboutToSynthesize(0);
         d->m_engine->say(utterance);
+        break;
+    case QTextToSpeech::Speaking:
+    case QTextToSpeech::Synthesizing:
+    case QTextToSpeech::Paused:
+        d->m_pendingUtterances.enqueue(utterance);
+        break;
     }
 
     return d->m_utteranceCounter++;