SpeechInputExtensionManager now interface with SpeechRecognitionManagerDelegate

chrome/browser/speech/chrome_speech_recognition_manager_delegate.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 CHROME_BROWSER_SPEECH_CHROME_SPEECH_RECOGNITION_MANAGER_DELEGATE_H_
 #define CHROME_BROWSER_SPEECH_CHROME_SPEECH_RECOGNITION_MANAGER_DELEGATE_H_
 #pragma once
 
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "chrome/browser/speech/speech_recognition_bubble_controller.h"
 #include "content/public/browser/speech_recognition_event_listener.h"
 #include "content/public/browser/speech_recognition_manager_delegate.h"
 #include "content/public/browser/speech_recognition_session_config.h"
 
+class SpeechRecognitionTrayIconController;
+
 namespace speech {
 
 // This is Chrome's implementation of the SpeechRecognitionManagerDelegate
 // interface.
 class ChromeSpeechRecognitionManagerDelegate
     : NON_EXPORTED_BASE(public content::SpeechRecognitionManagerDelegate),
       public content::SpeechRecognitionEventListener,
       public SpeechRecognitionBubbleControllerDelegate {
  public:
   ChromeSpeechRecognitionManagerDelegate();
@@ skipping 36 matching lines @@
   static void CheckRenderViewType(
       int session_id,
       base::Callback<void(int session_id, bool is_allowed)> callback,
       int render_process_id,
       int render_view_id);
 
   // Starts a new recognition session, using the config of the last one
   // (which is copied into |last_session_config_|). Used for "try again".
   void RestartLastSession();
 
+  // Lazy initializers for bubble and tray icon controller.
+  SpeechRecognitionBubbleController* BubbleController();

[Satish, 2012/05/20 21:40:26]

I'd suggest naming these as
  GetBubbleController()
because they are not simple getters which are of the form
  bubble_controller()

[Primiano Tucci (use gerrit), 2012/05/21 16:03:13]

Done.

+  SpeechRecognitionTrayIconController* TrayIconController();
+
   scoped_refptr<SpeechRecognitionBubbleController> bubble_controller_;
+  scoped_refptr<SpeechRecognitionTrayIconController> tray_icon_controller_;
   scoped_refptr<OptionalRequestInfo> optional_request_info_;
   scoped_ptr<content::SpeechRecognitionSessionConfig> last_session_config_;
 
   // TODO(primiano) this information should be kept into the bubble_controller_.
   int active_bubble_session_id_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeSpeechRecognitionManagerDelegate);
 };
 
 }  // namespace speech
 
 #endif  // CHROME_BROWSER_SPEECH_CHROME_SPEECH_RECOGNITION_MANAGER_DELEGATE_H_