Changing tab closure handling logic in speech recognition code and cleaning bubble controller. (Spe…

content/public/browser/speech_recognition_manager.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_SPEECH_RECOGNITION_MANAGER_H_
 #define CONTENT_PUBLIC_BROWSER_SPEECH_RECOGNITION_MANAGER_H_
 #pragma once
 
 #include "base/string16.h"
 #include "base/callback.h"
@@ skipping 31 matching lines @@
   // premilinary check on the delegate (CheckRecognitionIsAllowed).
   virtual void StartSession(int session_id) = 0;
 
   // Aborts recognition for an existing session, without providing any result.
   virtual void AbortSession(int session_id) = 0;
 
   // Aborts all sessions for a given listener, without providing any result.
   virtual void AbortAllSessionsForListener(
       SpeechRecognitionEventListener* listener) = 0;
 
+  // Aborts all sessions for a given renderer, without providing any result.
+  virtual void AbortAllSessionsForRenderer(

[Satish, 2012/07/03 15:11:57]

should be 'ForTab' since you are also passing a render-view-id ?

[Primiano Tucci (use gerrit), 2012/07/03 16:54:35]

Wouldn't such rename exclude the background renderer of extensions?

+      int render_process_id, int render_view_id) = 0;
+
   // Stops audio capture for an existing session. The audio captured before the
   // call will be processed, possibly ending up with a result.
   virtual void StopAudioCaptureForSession(int session_id) = 0;
 
   // Retrieves the configuration of a session, as provided by the caller
   // upon CreateSession.
   virtual const SpeechRecognitionSessionConfig& GetSessionConfig(int session_id)
       const = 0;
 
   // Retrieves the context associated to a session.
@@ skipping 20 matching lines @@
   // via the BrowserThread::FILE thread.
   virtual void ShowAudioInputSettings() = 0;
 
  protected:
    virtual ~SpeechRecognitionManager() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_SPEECH_RECOGNITION_MANAGER_H_