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

Unified Diff: sql/diagnostic_error_delegate.h

Issue 10872032: Revert 152946 - Replace HistoryQuickProvider protobuf-based caching with an SQLite-based database. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 4 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: sql/diagnostic_error_delegate.h
===================================================================
--- sql/diagnostic_error_delegate.h (revision 152962)
+++ sql/diagnostic_error_delegate.h (working copy)
@@ -12,10 +12,6 @@
namespace sql {
-// The histogram values from sqlite result codes currently range from 1 to 26
-// but 50 gives them room to grow.
-static const int kMaxSqliteError = 50;
-
// This class handles the exceptional sqlite errors that we might encounter
// if for example the db is corrupted. Right now we just generate a UMA
// histogram for release and an assert for debug builds.
@@ -43,7 +39,9 @@
// Trim off the extended error codes.
error &= 0xff;
- UMA_HISTOGRAM_ENUMERATION(UniqueT::name(), error, kMaxSqliteError);
+ // The histogram values from sqlite result codes go currently from 1 to
+ // 26 currently but 50 gives them room to grow.
+ UMA_HISTOGRAM_ENUMERATION(UniqueT::name(), error, 50);
}
};
Property changes on: sql/diagnostic_error_delegate.h
___________________________________________________________________
Deleted: svn:mergeinfo
« no previous file with comments | « chrome/test/data/History/url_history_provider_test_limited.db.txt ('k') | tools/valgrind/memcheck/suppressions.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698