Protector histograms for default search provider change added.

chrome/browser/protector/histograms.h

 // 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.
 
 #ifndef CHROME_BROWSER_PROTECTOR_HISTOGRAMS_H_
 #define CHROME_BROWSER_PROTECTOR_HISTOGRAMS_H_
 #pragma once
 
+class TemplateURL;
+
 namespace protector {
 
 // Histogram name to report protection errors for the default search
 // provider.
 extern const char kProtectorHistogramDefaultSearchProvider[];
 
 extern const char kProtectorBackupInvalidCounter[];

[whywhat, 2011/11/21 17:14:57]

Could you add comments to these counters?

[Ivan Korotkov, 2011/11/21 17:39:22]

Done.

 extern const char kProtectorValueChangedCounter[];
 extern const char kProtectorValueValidCounter[];
 
 // Protector histogram values.
 enum ProtectorError {
   kProtectorErrorBackupInvalid,
   kProtectorErrorValueChanged,
   kProtectorErrorValueValid,
 
   // This is for convenience only, must always be the last.
   kProtectorErrorCount
 };
 
+// Histogram name to report the new default search provider.
+extern const char kProtectorHistogramNewSearchProvider[];
+// Histogram name to report when user accepts new default search provider.
+extern const char kProtectorHistogramSearchProviderApplied[];
+// Histogram name to report when user keeps previous default search provider.
+extern const char kProtectorHistogramSearchProviderDiscarded[];
+// Histogram name to report when user ignores search provider change.
+extern const char kProtectorHistogramSearchProviderTimeout[];
+
+// Returns index to be used in histograms for given search provider (which may
+// be NULL, in which case a special index will be returned).
+int GetSearchProviderHistogramID(const TemplateURL* t_url);
+
+// Maximum value of search provider index in histogram enums.
+extern const int kProtectorMaxSearchProviderID;
+
 }  // namespace protector
 
 #endif  // CHROME_BROWSER_PROTECTOR_HISTOGRAMS_H_
-