diff options
author | leandrogracia@chromium.org <leandrogracia@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-08 11:04:59 +0000 |
---|---|---|
committer | leandrogracia@chromium.org <leandrogracia@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-08 11:04:59 +0000 |
commit | 9af487d4c521414c9f006ae342d3b9974f5a6bed (patch) | |
tree | e6be4ee402e4638f38330f7ef64cda5799082728 /chrome/browser | |
parent | 36c4a17155a08f8ca36bbc7760ecfc97c6a348db (diff) | |
download | chromium_src-9af487d4c521414c9f006ae342d3b9974f5a6bed.zip chromium_src-9af487d4c521414c9f006ae342d3b9974f5a6bed.tar.gz chromium_src-9af487d4c521414c9f006ae342d3b9974f5a6bed.tar.bz2 |
Fixing misleading name in a speech input setting and related variables.
This patch replaces all occurrences of 'censor results' with 'filter profanities', which is a much accurate description of what it does and avoids unfortunate confusions.
BUG=79991
TEST=none
Review URL: http://codereview.chromium.org/8199010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104653 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
-rw-r--r-- | chrome/browser/profiles/profile.cc | 2 | ||||
-rw-r--r-- | chrome/browser/profiles/profile_impl.cc | 8 | ||||
-rw-r--r-- | chrome/browser/speech/chrome_speech_input_preferences.cc | 17 | ||||
-rw-r--r-- | chrome/browser/speech/chrome_speech_input_preferences.h | 6 | ||||
-rw-r--r-- | chrome/browser/tab_contents/render_view_context_menu.cc | 20 |
5 files changed, 28 insertions, 25 deletions
diff --git a/chrome/browser/profiles/profile.cc b/chrome/browser/profiles/profile.cc index df0cdb4..f1a1ebe 100644 --- a/chrome/browser/profiles/profile.cc +++ b/chrome/browser/profiles/profile.cc @@ -140,7 +140,7 @@ void Profile::RegisterUserPrefs(PrefService* prefs) { prefs->RegisterBooleanPref(prefs::kEnableAutoSpellCorrect, true, PrefService::UNSYNCABLE_PREF); - prefs->RegisterBooleanPref(prefs::kSpeechInputCensorResults, + prefs->RegisterBooleanPref(prefs::kSpeechInputFilterProfanities, true, PrefService::UNSYNCABLE_PREF); #if defined(TOOLKIT_USES_GTK) diff --git a/chrome/browser/profiles/profile_impl.cc b/chrome/browser/profiles/profile_impl.cc index b76b3df..f3eedcf9 100644 --- a/chrome/browser/profiles/profile_impl.cc +++ b/chrome/browser/profiles/profile_impl.cc @@ -374,7 +374,7 @@ void ProfileImpl::DoFinalInit() { pref_change_registrar_.Add(prefs::kSpellCheckDictionary, this); pref_change_registrar_.Add(prefs::kEnableSpellCheck, this); pref_change_registrar_.Add(prefs::kEnableAutoSpellCorrect, this); - pref_change_registrar_.Add(prefs::kSpeechInputCensorResults, this); + pref_change_registrar_.Add(prefs::kSpeechInputFilterProfanities, this); pref_change_registrar_.Add(prefs::kClearSiteDataOnExit, this); pref_change_registrar_.Add(prefs::kGoogleServicesUsername, this); pref_change_registrar_.Add(prefs::kDefaultZoomLevel, this); @@ -1500,9 +1500,9 @@ void ProfileImpl::Observe(int type, RenderProcessHost* process = i.GetCurrentValue(); process->Send(new SpellCheckMsg_EnableAutoSpellCorrect(enabled)); } - } else if (*pref_name_in == prefs::kSpeechInputCensorResults) { - GetSpeechInputPreferences()->set_censor_results(prefs->GetBoolean( - prefs::kSpeechInputCensorResults)); + } else if (*pref_name_in == prefs::kSpeechInputFilterProfanities) { + GetSpeechInputPreferences()->set_filter_profanities(prefs->GetBoolean( + prefs::kSpeechInputFilterProfanities)); } else if (*pref_name_in == prefs::kClearSiteDataOnExit) { clear_local_state_on_exit_ = prefs->GetBoolean(prefs::kClearSiteDataOnExit); diff --git a/chrome/browser/speech/chrome_speech_input_preferences.cc b/chrome/browser/speech/chrome_speech_input_preferences.cc index e6a2982..ff88d31 100644 --- a/chrome/browser/speech/chrome_speech_input_preferences.cc +++ b/chrome/browser/speech/chrome_speech_input_preferences.cc @@ -11,25 +11,26 @@ ChromeSpeechInputPreferences::ChromeSpeechInputPreferences( PrefService* pref_service) - : censor_results_( - pref_service->GetBoolean(prefs::kSpeechInputCensorResults)) { + : filter_profanities_( + pref_service->GetBoolean(prefs::kSpeechInputFilterProfanities)) { } ChromeSpeechInputPreferences::~ChromeSpeechInputPreferences() { } -bool ChromeSpeechInputPreferences::censor_results() const { +bool ChromeSpeechInputPreferences::filter_profanities() const { DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); - return censor_results_; + return filter_profanities_; } -void ChromeSpeechInputPreferences::set_censor_results(bool censor_results) { +void ChromeSpeechInputPreferences::set_filter_profanities( + bool filter_profanities) { if (!BrowserThread::CurrentlyOn(BrowserThread::IO)) { BrowserThread::PostTask( BrowserThread::IO, FROM_HERE, - base::Bind(&ChromeSpeechInputPreferences::set_censor_results, - this, censor_results)); + base::Bind(&ChromeSpeechInputPreferences::set_filter_profanities, + this, filter_profanities)); return; } - censor_results_ = censor_results; + filter_profanities_ = filter_profanities; } diff --git a/chrome/browser/speech/chrome_speech_input_preferences.h b/chrome/browser/speech/chrome_speech_input_preferences.h index feca5a6..ebfef3b 100644 --- a/chrome/browser/speech/chrome_speech_input_preferences.h +++ b/chrome/browser/speech/chrome_speech_input_preferences.h @@ -17,14 +17,14 @@ class ChromeSpeechInputPreferences : public SpeechInputPreferences { explicit ChromeSpeechInputPreferences(PrefService* pref_service); // SpeechInputPreferences methods. - virtual bool censor_results() const OVERRIDE; - virtual void set_censor_results(bool censor_results) OVERRIDE; + virtual bool filter_profanities() const OVERRIDE; + virtual void set_filter_profanities(bool filter_profanities) OVERRIDE; private: virtual ~ChromeSpeechInputPreferences(); // Only to be accessed and modified on the IO thread. - bool censor_results_; + bool filter_profanities_; DISALLOW_COPY_AND_ASSIGN(ChromeSpeechInputPreferences); }; diff --git a/chrome/browser/tab_contents/render_view_context_menu.cc b/chrome/browser/tab_contents/render_view_context_menu.cc index 04f5240..8022aee 100644 --- a/chrome/browser/tab_contents/render_view_context_menu.cc +++ b/chrome/browser/tab_contents/render_view_context_menu.cc @@ -945,9 +945,9 @@ void RenderViewContextMenu::AppendSpellcheckOptionsSubMenu() { void RenderViewContextMenu::AppendSpeechInputOptionsSubMenu() { if (params_.speech_input_enabled) { speech_input_submenu_model_.AddCheckItem( - IDC_CONTENT_CONTEXT_SPEECH_INPUT_CENSOR_RESULTS, + IDC_CONTENT_CONTEXT_SPEECH_INPUT_FILTER_PROFANITIES, l10n_util::GetStringUTF16( - IDS_CONTENT_CONTEXT_SPEECH_INPUT_CENSOR_RESULTS)); + IDS_CONTENT_CONTEXT_SPEECH_INPUT_FILTER_PROFANITIES)); speech_input_submenu_model_.AddItemWithStringId( IDC_CONTENT_CONTEXT_SPEECH_INPUT_ABOUT, @@ -1311,7 +1311,7 @@ bool RenderViewContextMenu::IsCommandIdEnabled(int id) const { case IDC_SPELLCHECK_MENU: return true; - case IDC_CONTENT_CONTEXT_SPEECH_INPUT_CENSOR_RESULTS: + case IDC_CONTENT_CONTEXT_SPEECH_INPUT_FILTER_PROFANITIES: case IDC_CONTENT_CONTEXT_SPEECH_INPUT_ABOUT: case IDC_SPEECH_INPUT_MENU: return true; @@ -1377,8 +1377,9 @@ bool RenderViewContextMenu::IsCommandIdChecked(int id) const { } // Check box for menu item 'Block offensive words'. - if (id == IDC_CONTENT_CONTEXT_SPEECH_INPUT_CENSOR_RESULTS) { - return profile_->GetPrefs()->GetBoolean(prefs::kSpeechInputCensorResults); + if (id == IDC_CONTENT_CONTEXT_SPEECH_INPUT_FILTER_PROFANITIES) { + return profile_->GetPrefs()->GetBoolean( + prefs::kSpeechInputFilterProfanities); } // Don't bother getting the display language vector if this isn't a spellcheck @@ -1832,11 +1833,12 @@ void RenderViewContextMenu::ExecuteCommand(int id, int event_flags) { break; } - case IDC_CONTENT_CONTEXT_SPEECH_INPUT_CENSOR_RESULTS: { + case IDC_CONTENT_CONTEXT_SPEECH_INPUT_FILTER_PROFANITIES: { PrefService* prefs = profile_->GetPrefs(); - const bool censor = !prefs->GetBoolean(prefs::kSpeechInputCensorResults); - prefs->SetBoolean(prefs::kSpeechInputCensorResults, censor); - profile_->GetSpeechInputPreferences()->set_censor_results(censor); + const bool filter = !prefs->GetBoolean( + prefs::kSpeechInputFilterProfanities); + prefs->SetBoolean(prefs::kSpeechInputFilterProfanities, filter); + profile_->GetSpeechInputPreferences()->set_filter_profanities(filter); break; } |