Changeset 65327 in webkit

Timestamp:

Aug 13, 2010 9:49:30 AM (14 years ago)

Author:

commit-queue@webkit.org

Message:

2010-08-13 Satish Sampath <​satish@chromium.org>

Reviewed by Jeremy Orlow.

Support for multiple speech enabled elements in same page.
​https://bugs.webkit.org/show_bug.cgi?id=43922

• page/SpeechInput.cpp: Generates request ids as necessary when each speech enabled input element gets created/destroyed and multiplexes the listener callbacks to the appropriate input element based on the request id. (WebCore::SpeechInput::SpeechInput): (WebCore::SpeechInput::~SpeechInput): (WebCore::SpeechInput::registerListener): (WebCore::SpeechInput::unregisterListener): (WebCore::SpeechInput::didCompleteRecording): (WebCore::SpeechInput::didCompleteRecognition): (WebCore::SpeechInput::setRecognitionResult): (WebCore::SpeechInput::startRecognition): (WebCore::SpeechInput::stopRecording): (WebCore::SpeechInput::cancelRecognition):
• page/SpeechInput.h:
• page/SpeechInputClient.h: Now requires a one time set for the WebCore::SpeechInputListener and takes in a 'requestId' for all calls.
• page/SpeechInputListener.h: Now returns the above mentioned requestId in all the listener callbacks for identifying which input element the event goes to.
• platform/mock/SpeechInputClientMock.cpp: (WebCore::SpeechInputClientMock::SpeechInputClientMock): (WebCore::SpeechInputClientMock::setListener): (WebCore::SpeechInputClientMock::startRecognition): (WebCore::SpeechInputClientMock::stopRecording): (WebCore::SpeechInputClientMock::cancelRecognition): (WebCore::SpeechInputClientMock::timerFired):
• platform/mock/SpeechInputClientMock.h:
• rendering/TextControlInnerElements.cpp: (WebCore::InputFieldSpeechButtonElement::InputFieldSpeechButtonElement): (WebCore::InputFieldSpeechButtonElement::~InputFieldSpeechButtonElement): (WebCore::InputFieldSpeechButtonElement::defaultEventHandler): (WebCore::InputFieldSpeechButtonElement::speechInput): (WebCore::InputFieldSpeechButtonElement::didCompleteRecording): (WebCore::InputFieldSpeechButtonElement::didCompleteRecognition): (WebCore::InputFieldSpeechButtonElement::setRecognitionResult):
• rendering/TextControlInnerElements.h:

2010-08-13 Satish Sampath <​satish@chromium.org>

Reviewed by Jeremy Orlow.

Support for multiple speech enabled elements in same page.
​https://bugs.webkit.org/show_bug.cgi?id=43922

• public/WebSpeechInputController.h: Bubbles up the requestId given by webcore to the embedder. (WebKit::WebSpeechInputController::startRecognition): (WebKit::WebSpeechInputController::cancelRecognition): (WebKit::WebSpeechInputController::stopRecording):
• public/WebSpeechInputListener.h: Passes down the requestId given by embedder to webcore. (WebKit::WebSpeechInputListener::didCompleteRecording): (WebKit::WebSpeechInputListener::setRecognitionResult): (WebKit::WebSpeechInputListener::didCompleteRecognition):
• src/SpeechInputClientImpl.cpp: (WebKit::SpeechInputClientImpl::setListener): (WebKit::SpeechInputClientImpl::startRecognition): (WebKit::SpeechInputClientImpl::stopRecording): (WebKit::SpeechInputClientImpl::cancelRecognition): (WebKit::SpeechInputClientImpl::didCompleteRecording): (WebKit::SpeechInputClientImpl::didCompleteRecognition): (WebKit::SpeechInputClientImpl::setRecognitionResult):
• src/SpeechInputClientImpl.h:
• src/WebSpeechInputControllerMockImpl.cpp: (WebKit::WebSpeechInputControllerMockImpl::WebSpeechInputControllerMockImpl): (WebKit::WebSpeechInputControllerMockImpl::~WebSpeechInputControllerMockImpl): (WebKit::WebSpeechInputControllerMockImpl::didCompleteRecording): (WebKit::WebSpeechInputControllerMockImpl::didCompleteRecognition): (WebKit::WebSpeechInputControllerMockImpl::setRecognitionResult): (WebKit::WebSpeechInputControllerMockImpl::startRecognition): (WebKit::WebSpeechInputControllerMockImpl::cancelRecognition): (WebKit::WebSpeechInputControllerMockImpl::stopRecording):
• src/WebSpeechInputControllerMockImpl.h:

Location:

trunk

Files:

• WebCore/ChangeLog (modified) (1 diff)
• WebCore/page/SpeechInput.cpp (modified) (1 diff)
• WebCore/page/SpeechInput.h (modified) (2 diffs)
• WebCore/page/SpeechInputClient.h (modified) (1 diff)
• WebCore/page/SpeechInputListener.h (modified) (3 diffs)
• WebCore/platform/mock/SpeechInputClientMock.cpp (modified) (4 diffs)
• WebCore/platform/mock/SpeechInputClientMock.h (modified) (2 diffs)
• WebCore/rendering/TextControlInnerElements.cpp (modified) (3 diffs)
• WebCore/rendering/TextControlInnerElements.h (modified) (3 diffs)
• WebKit/chromium/ChangeLog (modified) (1 diff)
• WebKit/chromium/public/WebSpeechInputController.h (modified) (2 diffs)
• WebKit/chromium/public/WebSpeechInputListener.h (modified) (3 diffs)
• WebKit/chromium/src/SpeechInputClientImpl.cpp (modified) (1 diff)
• WebKit/chromium/src/SpeechInputClientImpl.h (modified) (1 diff)
• WebKit/chromium/src/WebSpeechInputControllerMockImpl.cpp (modified) (2 diffs)
• WebKit/chromium/src/WebSpeechInputControllerMockImpl.h (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-08-13  Satish Sampath  <satish@chromium.org>
+
+        Reviewed by Jeremy Orlow.
+
+        Support for multiple speech enabled elements in same page.
+        https://bugs.webkit.org/show_bug.cgi?id=43922
+
+        * page/SpeechInput.cpp: Generates request ids as necessary when each speech enabled input element gets
+        created/destroyed and multiplexes the listener callbacks to the appropriate input element based on the request id.
+        (WebCore::SpeechInput::SpeechInput):
+        (WebCore::SpeechInput::~SpeechInput):
+        (WebCore::SpeechInput::registerListener):
+        (WebCore::SpeechInput::unregisterListener):
+        (WebCore::SpeechInput::didCompleteRecording):
+        (WebCore::SpeechInput::didCompleteRecognition):
+        (WebCore::SpeechInput::setRecognitionResult):
+        (WebCore::SpeechInput::startRecognition):
+        (WebCore::SpeechInput::stopRecording):
+        (WebCore::SpeechInput::cancelRecognition):
+        * page/SpeechInput.h:
+        * page/SpeechInputClient.h: Now requires a one time set for the WebCore::SpeechInputListener and
+        takes in a 'requestId' for all calls.
+        * page/SpeechInputListener.h: Now returns the above mentioned requestId in all the listener callbacks for
+        identifying which input element the event goes to.
+        * platform/mock/SpeechInputClientMock.cpp:
+        (WebCore::SpeechInputClientMock::SpeechInputClientMock):
+        (WebCore::SpeechInputClientMock::setListener):
+        (WebCore::SpeechInputClientMock::startRecognition):
+        (WebCore::SpeechInputClientMock::stopRecording):
+        (WebCore::SpeechInputClientMock::cancelRecognition):
+        (WebCore::SpeechInputClientMock::timerFired):
+        * platform/mock/SpeechInputClientMock.h:
+        * rendering/TextControlInnerElements.cpp:
+        (WebCore::InputFieldSpeechButtonElement::InputFieldSpeechButtonElement):
+        (WebCore::InputFieldSpeechButtonElement::~InputFieldSpeechButtonElement):
+        (WebCore::InputFieldSpeechButtonElement::defaultEventHandler):
+        (WebCore::InputFieldSpeechButtonElement::speechInput):
+        (WebCore::InputFieldSpeechButtonElement::didCompleteRecording):
+        (WebCore::InputFieldSpeechButtonElement::didCompleteRecognition):
+        (WebCore::InputFieldSpeechButtonElement::setRecognitionResult):
+        * rendering/TextControlInnerElements.h:
+
 2010-08-13  Eric Carlson  <eric.carlson@apple.com>
 

trunk/WebCore/page/SpeechInput.cpp

@@ -42 +42 @@
 SpeechInput::SpeechInput(SpeechInputClient* client)
     : m_client(client)
-    , m_listener(0)
+    , m_nextListenerId(1)
 {
+    m_client->setListener(this);
 }
 
-void SpeechInput::didCompleteRecording()
+SpeechInput::~SpeechInput()
 {
-    ASSERT(m_listener);
-    m_listener->didCompleteRecording();
+    m_client->setListener(0);
 }
 
-void SpeechInput::didCompleteRecognition()
+int SpeechInput::registerListener(SpeechInputListener* listener)
 {
-    ASSERT(m_listener);
-    m_listener->didCompleteRecognition();
-    m_listener = 0;
+#if defined(DEBUG)
+    // Check if already present.
+    for (HashMap<int, SpeechInputListener*>::iterator it = m_listeners.begin(); it != m_listeners.end(); ++it)
+      ASSERT(it->second != listener);
+#endif
+
+    m_listeners.add(m_nextListenerId, listener);
+    return m_nextListenerId++;
 }
 
-void SpeechInput::setRecognitionResult(const String& result)
+void SpeechInput::unregisterListener(int listenerId)
 {
-    ASSERT(m_listener);
-    m_listener->setRecognitionResult(result);
+    if (m_listeners.contains(listenerId))
+        m_listeners.remove(listenerId);
 }
 
-bool SpeechInput::startRecognition(SpeechInputListener* listener)
+void SpeechInput::didCompleteRecording(int listenerId)
 {
-    // Cancel any ongoing recognition first.
-    if (m_listener) {
-        m_listener->didCompleteRecognition();
-        m_listener = 0;
-        m_client->cancelRecognition();
-    }
-
-    m_listener = listener;
-    return m_client->startRecognition(this);
+    // Don't assert if not present as the element might have been removed by the page while
+    // this event was on the way.
+    if (m_listeners.contains(listenerId))
+        m_listeners.get(listenerId)->didCompleteRecording(listenerId);
 }
 
-void SpeechInput::stopRecording()
+void SpeechInput::didCompleteRecognition(int listenerId)
 {
-    ASSERT(m_listener);
-    m_client->stopRecording();
+    // Don't assert if not present as the element might have been removed by the page while
+    // this event was on the way.
+    if (m_listeners.contains(listenerId))
+        m_listeners.get(listenerId)->didCompleteRecognition(listenerId);
+}
+
+void SpeechInput::setRecognitionResult(int listenerId, const String& result)
+{
+    // Don't assert if not present as the element might have been removed by the page while
+    // this event was on the way.
+    if (m_listeners.contains(listenerId))
+        m_listeners.get(listenerId)->setRecognitionResult(listenerId, result);
+}
+
+bool SpeechInput::startRecognition(int listenerId)
+{
+    ASSERT(m_listeners.contains(listenerId));
+    return m_client->startRecognition(listenerId);
+}
+
+void SpeechInput::stopRecording(int listenerId)
+{
+    ASSERT(m_listeners.contains(listenerId));
+    m_client->stopRecording(listenerId);
+}
+
+void SpeechInput::cancelRecognition(int listenerId)
+{
+    ASSERT(m_listeners.contains(listenerId));
+    m_client->cancelRecognition(listenerId);
 }
 

trunk/WebCore/page/SpeechInput.h

@@ -36 +36 @@
 #include "SpeechInputListener.h"
 #include <wtf/Forward.h>
+#include <wtf/HashMap.h>
 #include <wtf/Noncopyable.h>
 
@@ -49 +50 @@
 public:
     SpeechInput(SpeechInputClient*);
-    virtual ~SpeechInput() { }
+    virtual ~SpeechInput();
+
+    // Generates a unique ID for the given listener to be used for speech requests.
+    // This should be the first call made by listeners before anything else.
+    int registerListener(SpeechInputListener*);
+
+    // Invoked when the listener is done with recording or getting destroyed.
+    // Failure to unregister may result in crashes if there were any pending speech events.
+    void unregisterListener(int);
 
     // Methods invoked by the input elements.
-    bool startRecognition(SpeechInputListener* listener);
-    void stopRecording();
+    bool startRecognition(int);
+    void stopRecording(int);
+    void cancelRecognition(int);
 
     // SpeechInputListener methods.
-    virtual void didCompleteRecording();
-    virtual void didCompleteRecognition();
-    virtual void setRecognitionResult(const String&);
+    virtual void didCompleteRecording(int);
+    virtual void didCompleteRecognition(int);
+    virtual void setRecognitionResult(int, const String&);
 
 private:
     SpeechInputClient* m_client;
-    SpeechInputListener* m_listener;
+    HashMap<int, SpeechInputListener*> m_listeners;
+    int m_nextListenerId;
 };
 

trunk/WebCore/page/SpeechInputClient.h

@@ -41 +41 @@
 class SpeechInputClient {
 public:
-    virtual bool startRecognition(SpeechInputListener* listener) = 0;
+    // This is the first call made by a listener, registering itself for future callbacks.
+    // When the listener no longer needs speech input (for e.g. when it gets destroyed),
+    // it should set a null listener to stop receiving callbacks.
+    // The client does not take ownership of the pointer.
+    virtual void setListener(SpeechInputListener*) = 0;
+
+    // Starts speech recognition and audio recording.
+    virtual bool startRecognition(int requestId) = 0;
 
     // Stops audio recording and performs recognition with the audio recorded until now
     // (does not discard audio).
-    virtual void stopRecording() = 0;
+    virtual void stopRecording(int requestId) = 0;
 
     // Cancels an ongoing recognition and discards any audio recorded so far. No partial
     // recognition results are returned to the listener.
-    virtual void cancelRecognition() = 0;
+    virtual void cancelRecognition(int requestId) = 0;
 
 protected:

trunk/WebCore/page/SpeechInputListener.h

@@ -42 +42 @@
 public:
     // Informs that audio recording has completed and recognition is underway.
-    virtual void didCompleteRecording() = 0;
+    virtual void didCompleteRecording(int requestId) = 0;
 
     // Informs that speech recognition has completed. This gets invoked irrespective of whether
@@ -48 +48 @@
     // handler typically frees up any temporary resources allocated and waits for the next speech
     // recognition request.
-    virtual void didCompleteRecognition() = 0;
+    virtual void didCompleteRecognition(int requestId) = 0;
 
     // Gives results from speech recognition, either partial or the final results.
@@ -54 +54 @@
     // available as the user keeps speaking. If the speech could not be recognized properly
     // or if there was any other errors in the process, this method may never be called.
-    virtual void setRecognitionResult(const String& result) = 0;
+    virtual void setRecognitionResult(int requestId, const String& result) = 0;
 
 protected:

trunk/WebCore/platform/mock/SpeechInputClientMock.cpp

@@ -42 +42 @@
     , m_timer(this, &SpeechInputClientMock::timerFired)
     , m_listener(0)
+    , m_requestId(0)
 {
 }
 
-bool SpeechInputClientMock::startRecognition(SpeechInputListener* listener)
+void SpeechInputClientMock::setListener(SpeechInputListener* listener)
+{
+    m_listener = listener;
+}
+
+bool SpeechInputClientMock::startRecognition(int requestId)
 {
     if (m_timer.isActive())
         return false;
-    m_listener = listener;
+    m_requestId = requestId;
     m_recording = true;
     m_timer.startOneShot(0);
@@ -55 +61 @@
 }
 
-void SpeechInputClientMock::stopRecording()
+void SpeechInputClientMock::stopRecording(int requestId)
 {
+    ASSERT(requestId == m_requestId);
     if (m_timer.isActive() && m_recording) {
         m_timer.stop();
@@ -63 +70 @@
 }
 
-void SpeechInputClientMock::cancelRecognition()
+void SpeechInputClientMock::cancelRecognition(int requestId)
 {
+    ASSERT(requestId == m_requestId);
     if (m_timer.isActive()) {
         m_timer.stop();
         m_recording = false;
-        m_listener->didCompleteRecognition();
-        m_listener = 0;
+        m_listener->didCompleteRecognition(m_requestId);
+        m_requestId = 0;
     }
 }
@@ -82 +90 @@
     if (m_recording) {
         m_recording = false;
-        m_listener->didCompleteRecording();
+        m_listener->didCompleteRecording(m_requestId);
         m_timer.startOneShot(0);
     } else {
-        m_listener->setRecognitionResult(m_recognitionResult);
-        m_listener->didCompleteRecognition();
-        m_listener = 0;
+        m_listener->setRecognitionResult(m_requestId, m_recognitionResult);
+        m_listener->didCompleteRecognition(m_requestId);
+        m_requestId = 0;
     }
 }

trunk/WebCore/platform/mock/SpeechInputClientMock.h

@@ -50 +50 @@
 
     // SpeechInputClient methods.
-    bool startRecognition(SpeechInputListener*);
-    void stopRecording();
-    void cancelRecognition();
+    void setListener(SpeechInputListener*);
+    bool startRecognition(int);
+    void stopRecording(int);
+    void cancelRecognition(int);
 
 private:
@@ -60 +61 @@
     Timer<SpeechInputClientMock> m_timer;
     SpeechInputListener* m_listener;
+    int m_requestId;
     String m_recognitionResult;
 };

trunk/WebCore/rendering/TextControlInnerElements.cpp

@@ -347 +347 @@
     , m_capturing(false)
     , m_state(Idle)
-{
+    , m_listenerId(document()->page()->speechInput()->registerListener(this))
+{
+}
+
+InputFieldSpeechButtonElement::~InputFieldSpeechButtonElement()
+{
+    SpeechInput* speech = speechInput();
+    if (speech)  { // Could be null when page is unloading.
+        if (m_state != Idle)
+            speech->cancelRecognition(m_listenerId);
+        speech->unregisterListener(m_listenerId);
+    }
 }
 
@@ -387 +398 @@
         switch (m_state) {
         case Idle:
-            if (speechInput()->startRecognition(this))
+            if (speechInput()->startRecognition(m_listenerId))
                 setState(Recording);
             break;
         case Recording:
-            speechInput()->stopRecording();
+            speechInput()->stopRecording(m_listenerId);
             break;
         case Recognizing:
@@ -414 +425 @@
 SpeechInput* InputFieldSpeechButtonElement::speechInput()
 {
-    return document()->page()->speechInput();
-}
-
-void InputFieldSpeechButtonElement::didCompleteRecording()
+    return document()->page() ? document()->page()->speechInput() : 0;
+}
+
+void InputFieldSpeechButtonElement::didCompleteRecording(int)
 {
     setState(Recognizing);
 }
 
-void InputFieldSpeechButtonElement::didCompleteRecognition()
+void InputFieldSpeechButtonElement::didCompleteRecognition(int)
 {
     setState(Idle);
 }
 
-void InputFieldSpeechButtonElement::setRecognitionResult(const String& result)
+void InputFieldSpeechButtonElement::setRecognitionResult(int, const String& result)
 {
     HTMLInputElement* input = static_cast<HTMLInputElement*>(shadowAncestorNode());

trunk/WebCore/rendering/TextControlInnerElements.h

@@ -127 +127 @@
 
     static PassRefPtr<InputFieldSpeechButtonElement> create(Node*);
+    virtual ~InputFieldSpeechButtonElement();
 
     virtual void detach();
@@ -133 +134 @@
 
     // SpeechInputListener methods.
-    void didCompleteRecording();
-    void didCompleteRecognition();
-    void setRecognitionResult(const String& result);
+    void didCompleteRecording(int);
+    void didCompleteRecognition(int);
+    void setRecognitionResult(int, const String& result);
 
 private:
@@ -144 +145 @@
     bool m_capturing;
     SpeechInputState m_state;
+    int m_listenerId;
 };
 

trunk/WebKit/chromium/ChangeLog

@@ +1 @@
+2010-08-13  Satish Sampath  <satish@chromium.org>
+
+        Reviewed by Jeremy Orlow.
+
+        Support for multiple speech enabled elements in same page.
+        https://bugs.webkit.org/show_bug.cgi?id=43922
+
+        * public/WebSpeechInputController.h: Bubbles up the requestId given by webcore to the embedder.
+        (WebKit::WebSpeechInputController::startRecognition):
+        (WebKit::WebSpeechInputController::cancelRecognition):
+        (WebKit::WebSpeechInputController::stopRecording):
+        * public/WebSpeechInputListener.h: Passes down the requestId given by embedder to webcore.
+        (WebKit::WebSpeechInputListener::didCompleteRecording):
+        (WebKit::WebSpeechInputListener::setRecognitionResult):
+        (WebKit::WebSpeechInputListener::didCompleteRecognition):
+        * src/SpeechInputClientImpl.cpp:
+        (WebKit::SpeechInputClientImpl::setListener):
+        (WebKit::SpeechInputClientImpl::startRecognition):
+        (WebKit::SpeechInputClientImpl::stopRecording):
+        (WebKit::SpeechInputClientImpl::cancelRecognition):
+        (WebKit::SpeechInputClientImpl::didCompleteRecording):
+        (WebKit::SpeechInputClientImpl::didCompleteRecognition):
+        (WebKit::SpeechInputClientImpl::setRecognitionResult):
+        * src/SpeechInputClientImpl.h:
+        * src/WebSpeechInputControllerMockImpl.cpp:
+        (WebKit::WebSpeechInputControllerMockImpl::WebSpeechInputControllerMockImpl):
+        (WebKit::WebSpeechInputControllerMockImpl::~WebSpeechInputControllerMockImpl):
+        (WebKit::WebSpeechInputControllerMockImpl::didCompleteRecording):
+        (WebKit::WebSpeechInputControllerMockImpl::didCompleteRecognition):
+        (WebKit::WebSpeechInputControllerMockImpl::setRecognitionResult):
+        (WebKit::WebSpeechInputControllerMockImpl::startRecognition):
+        (WebKit::WebSpeechInputControllerMockImpl::cancelRecognition):
+        (WebKit::WebSpeechInputControllerMockImpl::stopRecording):
+        * src/WebSpeechInputControllerMockImpl.h:
+
 2010-08-12  Yury Semikhatsky  <yurys@chromium.org>
 

trunk/WebKit/chromium/public/WebSpeechInputController.h

@@ -42 +42 @@
     // runs to the limit or stopRecording is called. Progress indications and the recognized
     // text are returned via the listener interface.
+    virtual bool startRecognition(int)
+    {
+        return startRecognition();
+    }
+    // FIXME: Remove this once chromium has picked up this change.
     virtual bool startRecognition()
     {
-         WEBKIT_ASSERT_NOT_REACHED();
-         return false;
+        WEBKIT_ASSERT_NOT_REACHED();
+        return false;
     }
 
     // Cancels an ongoing recognition and discards any audio recorded so far. No partial
     // recognition results are returned to the listener.
+    virtual void cancelRecognition(int) { cancelRecognition(); }
+    // FIXME: Remove this once chromium has picked up this change.
     virtual void cancelRecognition() { WEBKIT_ASSERT_NOT_REACHED(); }
 
@@ -57 +64 @@
     // recording 'endpointer' should detect silence in the input and stop recording automatically.
     // Call startRecognition() to record audio and recognize speech again.
+    virtual void stopRecording(int) { stopRecording(); }
+    // FIXME: Remove this once chromium has picked up this change.
     virtual void stopRecording() { WEBKIT_ASSERT_NOT_REACHED(); }
 

trunk/WebKit/chromium/public/WebSpeechInputListener.h

@@ -48 +48 @@
     // Typically after this call the listener would update the UI to reflect that recognition is
     // in progress.
-    virtual void didCompleteRecording() = 0;
+    virtual void didCompleteRecording(int) = 0;
+    // FIXME: Remove this once chromium has picked up this change.
+    virtual void didCompleteRecording()
+    {
+        didCompleteRecording(1);
+    }
 
     // Gives results from speech recognition, either partial or the final results.
@@ -54 +59 @@
     // available as the user keeps speaking. If the speech could not be recognized properly
     // or if there was any other errors in the process, this method may never be called.
-    virtual void setRecognitionResult(const WebString&) = 0;
+    virtual void setRecognitionResult(int, const WebString&) = 0;
+    // FIXME: Remove this once chromium has picked up this change.
+    virtual void setRecognitionResult(const WebString& result)
+    {
+        setRecognitionResult(1, result);
+    }
 
     // Informs that speech recognition has completed. This gets invoked irrespective of whether
@@ -60 +70 @@
     // handler typically frees up any temporary resources allocated and waits for the next speech
     // recognition request.
-    virtual void didCompleteRecognition() = 0;
+    virtual void didCompleteRecognition(int) = 0;
+    // FIXME: Remove this once chromium has picked up this change.
+    virtual void didCompleteRecognition()
+    {
+        didCompleteRecognition(1);
+    }
 
 protected:

trunk/WebKit/chromium/src/SpeechInputClientImpl.cpp

@@ -52 +52 @@
 }
 
-bool SpeechInputClientImpl::startRecognition(WebCore::SpeechInputListener* listener)
+void SpeechInputClientImpl::setListener(WebCore::SpeechInputListener* listener)
 {
     m_listener = listener;
-    return m_controller->startRecognition();
 }
 
-void SpeechInputClientImpl::stopRecording()
+bool SpeechInputClientImpl::startRecognition(int requestId)
 {
     ASSERT(m_listener);
-    m_controller->stopRecording();
+    return m_controller->startRecognition(requestId);
 }
 
-void SpeechInputClientImpl::cancelRecognition()
+void SpeechInputClientImpl::stopRecording(int requestId)
 {
     ASSERT(m_listener);
-    m_controller->cancelRecognition();
+    m_controller->stopRecording(requestId);
 }
 
-void SpeechInputClientImpl::didCompleteRecording()
+void SpeechInputClientImpl::cancelRecognition(int requestId)
 {
     ASSERT(m_listener);
-    m_listener->didCompleteRecording();
+    m_controller->cancelRecognition(requestId);
 }
 
-void SpeechInputClientImpl::didCompleteRecognition()
+void SpeechInputClientImpl::didCompleteRecording(int requestId)
 {
     ASSERT(m_listener);
-    m_listener->didCompleteRecognition();
-    m_listener = 0;
+    m_listener->didCompleteRecording(requestId);
 }
 
-void SpeechInputClientImpl::setRecognitionResult(const WebString& result)
+void SpeechInputClientImpl::didCompleteRecognition(int requestId)
 {
     ASSERT(m_listener);
-    m_listener->setRecognitionResult(result);
+    m_listener->didCompleteRecognition(requestId);
+}
+
+void SpeechInputClientImpl::setRecognitionResult(int requestId, const WebString& result)
+{
+    ASSERT(m_listener);
+    m_listener->setRecognitionResult(requestId, result);
 }
 

trunk/WebKit/chromium/src/SpeechInputClientImpl.h

@@ -54 +54 @@
 
     // SpeechInputClient methods.
-    bool startRecognition(WebCore::SpeechInputListener*);
-    void stopRecording();
-    void cancelRecognition();
+    void setListener(WebCore::SpeechInputListener*);
+    bool startRecognition(int);
+    void stopRecording(int);
+    void cancelRecognition(int);
 
     // WebSpeechInputListener methods.
-    void didCompleteRecording();
-    void setRecognitionResult(const WebString&);
-    void didCompleteRecognition();
+    void didCompleteRecording(int);
+    void setRecognitionResult(int, const WebString&);
+    void didCompleteRecognition(int);
 
 private:
     WebSpeechInputController* m_controller; // To call into the embedder.
-    WebCore::SpeechInputListener* m_listener; // Valid when recognition is in progress.
+    WebCore::SpeechInputListener* m_listener;
 };
 

trunk/WebKit/chromium/src/WebSpeechInputControllerMockImpl.cpp

@@ -47 +47 @@
     , m_listener(listener)
 {
+    m_webcoreMock->setListener(this);
+}
+
+WebSpeechInputControllerMockImpl::~WebSpeechInputControllerMockImpl()
+{
+    m_webcoreMock->setListener(0);
 }
 
@@ -54 +60 @@
 }
 
-void WebSpeechInputControllerMockImpl::didCompleteRecording()
+void WebSpeechInputControllerMockImpl::didCompleteRecording(int requestId)
 {
-    m_listener->didCompleteRecording();
+    m_listener->didCompleteRecording(requestId);
 }
 
-void WebSpeechInputControllerMockImpl::didCompleteRecognition()
+void WebSpeechInputControllerMockImpl::didCompleteRecognition(int requestId)
 {
-    m_listener->didCompleteRecognition();
+    m_listener->didCompleteRecognition(requestId);
 }
 
-void WebSpeechInputControllerMockImpl::setRecognitionResult(const WTF::String& result)
+void WebSpeechInputControllerMockImpl::setRecognitionResult(int requestId, const WTF::String& result)
 {
-    m_listener->setRecognitionResult(result);
+    m_listener->setRecognitionResult(requestId, result);
 }
 
-bool WebSpeechInputControllerMockImpl::startRecognition()
+bool WebSpeechInputControllerMockImpl::startRecognition(int requestId)
 {
-    return m_webcoreMock->startRecognition(this);
+    return m_webcoreMock->startRecognition(requestId);
 }
 
-void WebSpeechInputControllerMockImpl::cancelRecognition()
+void WebSpeechInputControllerMockImpl::cancelRecognition(int requestId)
 {
-    m_webcoreMock->cancelRecognition();
+    m_webcoreMock->cancelRecognition(requestId);
 }
 
-void WebSpeechInputControllerMockImpl::stopRecording()
+void WebSpeechInputControllerMockImpl::stopRecording(int requestId)
 {
-    m_webcoreMock->stopRecording();
+    m_webcoreMock->stopRecording(requestId);
 }
 

trunk/WebKit/chromium/src/WebSpeechInputControllerMockImpl.h

@@ -48 +48 @@
 public:
     WebSpeechInputControllerMockImpl(WebSpeechInputListener*);
+    virtual ~WebSpeechInputControllerMockImpl();
 
     // WebCore::SpeechInputListener methods.
-    void didCompleteRecording();
-    void didCompleteRecognition();
-    void setRecognitionResult(const WTF::String& result);
+    void didCompleteRecording(int requestId);
+    void didCompleteRecognition(int requestId);
+    void setRecognitionResult(int requestId, const WTF::String& result);
 
     // WebSpeechInputController methods.
-    bool startRecognition();
-    void cancelRecognition();
-    void stopRecording();
+    bool startRecognition(int requestId);
+    void cancelRecognition(int requestId);
+    void stopRecording(int requestId);
 
     // WebSpeechInputControllerMock methods.