diff options
author | jeremy@chromium.org <jeremy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-06 16:40:40 +0000 |
---|---|---|
committer | jeremy@chromium.org <jeremy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-06 16:40:40 +0000 |
commit | b15a406c41e2ef0c053256edc76696c84d9bfe99 (patch) | |
tree | a011df71c33c160170864532a70cf3adac520423 /chrome/browser | |
parent | d0488cac770fe5b2ab08bf8271391d21e3174a32 (diff) | |
download | chromium_src-b15a406c41e2ef0c053256edc76696c84d9bfe99.zip chromium_src-b15a406c41e2ef0c053256edc76696c84d9bfe99.tar.gz chromium_src-b15a406c41e2ef0c053256edc76696c84d9bfe99.tar.bz2 |
Remove code to collect spellchecker stats
These stats were originally introduced to gauge the use of Hunspell vs the native spellchecker on OS X. Now that we know that Hunspell is hardly used, the code can be removed.
BUG=69944
TEST=None
Review URL: http://codereview.chromium.org/8805025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113218 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
-rw-r--r-- | chrome/browser/spellchecker/spellcheck_host_impl.cc | 36 |
1 files changed, 0 insertions, 36 deletions
diff --git a/chrome/browser/spellchecker/spellcheck_host_impl.cc b/chrome/browser/spellchecker/spellcheck_host_impl.cc index 7d4ac5a..a3846d9 100644 --- a/chrome/browser/spellchecker/spellcheck_host_impl.cc +++ b/chrome/browser/spellchecker/spellcheck_host_impl.cc @@ -32,9 +32,6 @@ #include "net/url_request/url_request_context_getter.h" #include "third_party/hunspell/google/bdict.h" #include "ui/base/l10n/l10n_util.h" -#if defined(OS_MACOSX) -#include "base/metrics/histogram.h" -#endif using content::BrowserThread; @@ -48,32 +45,6 @@ FilePath GetFirstChoiceFilePath(const std::string& language) { return SpellCheckCommon::GetVersionedFileName(language, dict_dir); } -#if defined(OS_MACOSX) -// Collect metrics on how often Hunspell is used on OS X vs the native -// spellchecker. -void RecordSpellCheckStats(bool native_spellchecker_used, - const std::string& language) { - CR_DEFINE_STATIC_LOCAL(std::set<std::string>, languages_seen, ()); - - // Only count a language code once for each session.. - if (languages_seen.find(language) != languages_seen.end()) { - return; - } - languages_seen.insert(language); - - enum { - SPELLCHECK_OSX_NATIVE_SPELLCHECKER_USED = 0, - SPELLCHECK_HUNSPELL_USED = 1 - }; - - bool engine_used = native_spellchecker_used ? - SPELLCHECK_OSX_NATIVE_SPELLCHECKER_USED : - SPELLCHECK_HUNSPELL_USED; - - UMA_HISTOGRAM_COUNTS("SpellCheck.OSXEngineUsed", engine_used); -} -#endif - #if defined(OS_WIN) FilePath GetFallbackFilePath(const FilePath& first_choice) { FilePath dict_dir; @@ -132,9 +103,6 @@ void SpellCheckHostImpl::Initialize() { if (SpellCheckerPlatform::SpellCheckerAvailable() && SpellCheckerPlatform::PlatformSupportsLanguage(language_)) { -#if defined(OS_MACOSX) - RecordSpellCheckStats(true, language_); -#endif use_platform_spellchecker_ = true; SpellCheckerPlatform::SetLanguage(language_); MessageLoop::current()->PostTask(FROM_HERE, @@ -143,10 +111,6 @@ void SpellCheckHostImpl::Initialize() { return; } -#if defined(OS_MACOSX) - RecordSpellCheckStats(false, language_); -#endif - BrowserThread::PostTaskAndReply(BrowserThread::FILE, FROM_HERE, base::Bind(&SpellCheckHostImpl::InitializeDictionaryLocation, base::Unretained(this)), |