Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(730)

Unified Diff: chrome/browser/webdata/keyword_table.cc

Issue 8430027: Added histogram on successful check. Safe verification (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/webdata/keyword_table.cc
diff --git a/chrome/browser/webdata/keyword_table.cc b/chrome/browser/webdata/keyword_table.cc
index b5391948028ed73a6780c7b34ec9bd2b59688082..ee8ef790b79e326816bafaab5483fe7b7d463b0b 100644
--- a/chrome/browser/webdata/keyword_table.cc
+++ b/chrome/browser/webdata/keyword_table.cc
@@ -79,7 +79,7 @@ std::string GetSearchProviderIDSignature(int64 id) {
// Checks if signature for search provider id is correct and returns the
// result.
bool IsSearchProviderIDValid(int64 id, const std::string& signature) {
- return signature == GetSearchProviderIDSignature(id);
+ return protector::IsSettingValid(base::Int64ToString(id), signature);
}
} // anonymous namespace
@@ -289,6 +289,10 @@ bool KeywordTable::DidDefaultSearchProviderChange() {
protector::kProtectorErrorCount);
return true;
}
+ UMA_HISTOGRAM_ENUMERATION(
+ protector::kProtectorHistogramDefaultSearchProvider,
+ protector::kProtectorErrorNoError,
+ protector::kProtectorErrorCount);
return false;
}

Powered by Google App Engine
This is Rietveld 408576698