Added histogram on successful check. Safe verification

chrome/browser/webdata/keyword_table_unittest.cc

 // Copyright (c) 2011 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 "base/file_util.h"
 #include "base/path_service.h"
 #include "base/string_number_conversions.h"
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/common/chrome_paths.h"
@@ skipping 150 matching lines @@
 
   ASSERT_EQ(sql::INIT_OK, db.Init(file_));
 
   EXPECT_EQ(0, db.GetKeywordTable()->GetDefaultSearchProviderID());
 
   ASSERT_TRUE(db.GetKeywordTable()->SetDefaultSearchProviderID(10));
   EXPECT_EQ(10, db.GetKeywordTable()->GetDefaultSearchProviderID());
   EXPECT_EQ(10, db.GetKeywordTable()->GetDefaultSearchProviderIDBackup());
   EXPECT_FALSE(db.GetKeywordTable()->DidDefaultSearchProviderChange());
 
-  // Verify that backup doesn't affect the actual value for now.
   ASSERT_TRUE(db.GetKeywordTable()->SetDefaultSearchProviderBackupID(11));
   EXPECT_EQ(10, db.GetKeywordTable()->GetDefaultSearchProviderID());
   EXPECT_EQ(11, db.GetKeywordTable()->GetDefaultSearchProviderIDBackup());
   EXPECT_TRUE(db.GetKeywordTable()->DidDefaultSearchProviderChange());
 }
 
 TEST_F(KeywordTableTest, UpdateKeyword) {
   WebDatabase db;
 
   ASSERT_EQ(sql::INIT_OK, db.Init(file_));
@@ skipping 112 matching lines @@
   EXPECT_TRUE(db.GetKeywordTable()->GetKeywords(&template_urls));
   EXPECT_EQ(1U, template_urls.size());
   const TemplateURL* restored_url = template_urls.front();
 
   EXPECT_EQ(template_url.short_name(), restored_url->short_name());
   EXPECT_EQ(template_url.keyword(), restored_url->keyword());
   EXPECT_EQ(GetID(&template_url), GetID(restored_url));
   EXPECT_FALSE(restored_url->sync_guid().empty());
   delete restored_url;
 }