Context menu for "Voice recognition options"

content/browser/speech/speech_input_manager.h

 // Copyright (c) 2010 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_BROWSER_SPEECH_SPEECH_INPUT_MANAGER_H_
 #define CONTENT_BROWSER_SPEECH_SPEECH_INPUT_MANAGER_H_
 
 #include "base/basictypes.h"
 #include "content/common/speech_input_result.h"
 #include "ui/gfx/rect.h"
@@ skipping 46 matching lines @@
                                 int render_process_id,
                                 int render_view_id,
                                 const gfx::Rect& element_rect,
                                 const std::string& language,
                                 const std::string& grammar,
                                 const std::string& origin_url)  = 0;
   virtual void CancelRecognition(int caller_id) = 0;
   virtual void StopRecording(int caller_id) = 0;
 
   virtual void CancelAllRequestsWithDelegate(Delegate* delegate) = 0;
+
+  void set_censor_results(bool censor) { censor_results_ = censor; }
+
+  bool censor_results() { return censor_results_; }
+ private:
+  bool censor_results_;

[Satish, 2011/07/20 15:23:13]

Sorry I didn't notice this earlier. This class is used as an interface and
doesn't have an implementation by itself. SpeechInputManagerImpl implements it.
So this member variable ideally doesn't belong in here but in the implementation
class. It should also be initialised to an appropriate value (true?) in the
class's constructor.

However moving this variable to the Impl class means the 2 new methods would
have to be virtual, making it a bit ugly to have them as get_xxx and set_xxx. So
I think it is ok to leave this here, but just add a default constructor to this
class for initializing the variable.

[gshires, 2011/07/20 17:16:10]

Done.

 };
 
 // This typedef is to workaround the issue with certain versions of
 // Visual Studio where it gets confused between multiple Delegate
 // classes and gives a C2500 error. (I saw this error on the try bots -
 // the workaround was not needed for my machine).
 typedef SpeechInputManager::Delegate SpeechInputManagerDelegate;
 
 }  // namespace speech_input
 
 #endif  // CONTENT_BROWSER_SPEECH_SPEECH_INPUT_MANAGER_H_