Context menu for "Voice recognition options"

content/browser/speech/speech_input_manager.h

Index: content/browser/speech/speech_input_manager.h
diff --git a/content/browser/speech/speech_input_manager.h b/content/browser/speech/speech_input_manager.h
index 342f2c139d7dc97ddfca349211ca97841f4d74ea..633a9ad1c79c81a077c5a047507479f79fbcc291 100644
--- a/content/browser/speech/speech_input_manager.h
+++ b/content/browser/speech/speech_input_manager.h
@@ -64,6 +64,12 @@ class SpeechInputManager {
   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