[Spellcheck] Record the number of words in the custom dictionary when the custom dictionary is load…

chrome/browser/spellchecker/spellcheck_custom_dictionary_unittest.cc

 // 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.
 
 #include <vector>
 
 #include "base/file_util.h"
 #include "base/message_loop.h"
+#include "base/metrics/histogram.h"
+#include "base/metrics/histogram_samples.h"
+#include "base/metrics/statistics_recorder.h"
 #include "base/strings/string_number_conversions.h"
 #include "chrome/browser/spellchecker/spellcheck_custom_dictionary.h"
 #include "chrome/browser/spellchecker/spellcheck_factory.h"
+#include "chrome/browser/spellchecker/spellcheck_host_metrics.h"
 #include "chrome/browser/spellchecker/spellcheck_service.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/spellcheck_common.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/test/test_browser_thread.h"
 #include "sync/api/sync_change.h"
 #include "sync/api/sync_data.h"
 #include "sync/api/sync_error_factory.h"
 #include "sync/api/sync_error_factory_mock.h"
 #include "sync/protocol/sync.pb.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
+using base::HistogramBase;
+using base::HistogramSamples;
+using base::StatisticsRecorder;
 using content::BrowserThread;
 using chrome::spellcheck_common::WordList;
 
 namespace {
 
 // Get all sync data for the custom dictionary without limiting to maximum
 // number of syncable words.
 syncer::SyncDataList GetAllSyncDataNoLimit(
     const SpellcheckCustomDictionary* dictionary) {
   syncer::SyncDataList data;
@@ skipping 20 matching lines @@
   SpellcheckCustomDictionaryTest()
       : ui_thread_(BrowserThread::UI, &message_loop_),
         file_thread_(BrowserThread::FILE, &message_loop_),
         profile_(new TestingProfile) {
   }
 
   virtual void SetUp() OVERRIDE {
     // Use SetTestingFactoryAndUse to force creation and initialization.
     SpellcheckServiceFactory::GetInstance()->SetTestingFactoryAndUse(
         profile_.get(), &BuildSpellcheckService);
+
+    base::StatisticsRecorder::Initialize();

[please use gerrit instead, 2013/04/04 00:31:55]

Nit: No need for "base::" here since you're "using base::StatisticsRecorder"
above.

[rpetterson, 2013/04/04 20:24:37]

Done.

+    metrics_.reset(new SpellCheckHostMetrics);

[please use gerrit instead, 2013/04/04 00:31:55]

No need for an instance of SpellcheckHostMetrics object, since you've made
RecordCustomWordCountStats static. (I think that
spellcheck_host_metrics_unittest.cc also calls RecordCustomWordCountStats
dynamically. Please update that test to call the method statically.)

[rpetterson, 2013/04/04 20:24:37]

Done.

   }
 
   virtual void TearDown() OVERRIDE {
     MessageLoop::current()->RunUntilIdle();
   }
 
   // A wrapper around SpellcheckCustomDictionary::LoadDictionaryFile private
   // function to avoid a large number of FRIEND_TEST declarations in
   // SpellcheckCustomDictionary.
   chrome::spellcheck_common::WordList LoadDictionaryFile(
@@ skipping 25 matching lines @@
       SpellcheckCustomDictionary& dictionary,
       const SpellcheckCustomDictionary::Change& change) {
     return dictionary.Apply(change);
   }
 
   MessageLoop message_loop_;
   content::TestBrowserThread ui_thread_;
   content::TestBrowserThread file_thread_;
 
   scoped_ptr<TestingProfile> profile_;
+  scoped_ptr<SpellCheckHostMetrics> metrics_;

[please use gerrit instead, 2013/04/04 00:31:55]

Ditto.

[rpetterson, 2013/04/04 20:24:37]

Done.

 };
 
 // A wrapper around SpellcheckCustomDictionary that does not own the wrapped
 // object. An instance of this class can be inside of a scoped pointer safely
 // while the dictionary is managed by another scoped pointer.
 class SyncChangeProcessorDelegate : public syncer::SyncChangeProcessor {
  public:
   explicit SyncChangeProcessorDelegate(SpellcheckCustomDictionary* dictionary)
       : dictionary_(dictionary) {}
   virtual ~SyncChangeProcessorDelegate() {}
@@ skipping 947 matching lines @@
     EXPECT_FALSE(client_custom_dictionary->IsSyncing());
     EXPECT_EQ(chrome::spellcheck_common::MAX_SYNCABLE_DICTIONARY_WORDS * 2,
               client_custom_dictionary->GetWords().size());
   }
 
   // The sync server should not receive more words, because it has the maximum
   // number of words already.
   EXPECT_EQ(chrome::spellcheck_common::MAX_SYNCABLE_DICTIONARY_WORDS,
             server_custom_dictionary->GetWords().size());
 }
+
+TEST_F(SpellcheckCustomDictionaryTest, RecordSizeStatsCorrectly) {
+  // Record a baseline.
+  metrics_->RecordCustomWordCountStats(123);

[please use gerrit instead, 2013/04/04 00:31:55]

Ditto.

[rpetterson, 2013/04/04 20:24:37]

Done.

+
+  HistogramBase* histogram =
+      StatisticsRecorder::FindHistogram("SpellCheck.CustomWords");
+  ASSERT_TRUE(histogram != NULL);
+  scoped_ptr<HistogramSamples> baseline = histogram->SnapshotSamples();
+
+  // Load the dictionary which should be empty.
+  base::FilePath path =
+      profile_->GetPath().Append(chrome::kCustomDictionaryFileName);
+  WordList loaded_custom_words = LoadDictionaryFile(path);
+  EXPECT_EQ(0u, loaded_custom_words.size());
+
+  // We expect there to be an entry with 0.
+  histogram =
+      StatisticsRecorder::FindHistogram("SpellCheck.CustomWords");
+  ASSERT_TRUE(histogram != NULL);
+  scoped_ptr<HistogramSamples> samples = histogram->SnapshotSamples();
+
+  samples->Subtract(*baseline);
+  EXPECT_EQ(0,samples->sum());
+
+  SpellcheckCustomDictionary::Change change;
+  change.AddWord("bar");
+  change.AddWord("foo");
+  UpdateDictionaryFile(change, path);
+
+  // Load the dictionary again and it should have 2 entries.
+  loaded_custom_words = LoadDictionaryFile(path);
+  EXPECT_EQ(2u, loaded_custom_words.size());
+
+  histogram =
+      StatisticsRecorder::FindHistogram("SpellCheck.CustomWords");
+  ASSERT_TRUE(histogram != NULL);
+  scoped_ptr<HistogramSamples> samples2 = histogram->SnapshotSamples();
+
+  samples2->Subtract(*baseline);
+  EXPECT_EQ(2,samples2->sum());
+}