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

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

Issue 11493003: Remove the protector service. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge & cleanup Created 8 years 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/web_database.cc
diff --git a/chrome/browser/webdata/web_database.cc b/chrome/browser/webdata/web_database.cc
index cdf9d5828cb624ee7aa13d169169f546efaf6c5c..73b8467778091564ae229037101c7aa9f44594af 100644
--- a/chrome/browser/webdata/web_database.cc
+++ b/chrome/browser/webdata/web_database.cc
@@ -21,11 +21,11 @@
// corresponding changes must happen in the unit tests, and new migration test
// added. See |WebDatabaseMigrationTest::kCurrentTestedVersionNumber|.
// static
-const int WebDatabase::kCurrentVersionNumber = 47;
+const int WebDatabase::kCurrentVersionNumber = 48;
namespace {
-const int kCompatibleVersionNumber = 47;
+const int kCompatibleVersionNumber = 48;
// Change the version number and possibly the compatibility version of
// |meta_table_|.
@@ -158,7 +158,7 @@ sql::InitStatus WebDatabase::Init(const FilePath& db_name) {
}
sql::InitStatus WebDatabase::MigrateOldVersionsAsNeeded() {
- // Some malware tries to force protector to re-sign things by lowering the
+ // Some malware tried to force protector to re-sign things by lowering the
Peter Kasting 2012/12/11 20:09:06 Nit: In the future, people won't know what protect
Ivan Korotkov 2012/12/12 17:02:57 Done.
// version number, causing migration to fail. Ensure the version number is at
// least as high as the compatible version number.
int current_version = std::max(meta_table_.GetVersionNumber(),
@@ -350,6 +350,13 @@ sql::InitStatus WebDatabase::MigrateOldVersionsAsNeeded() {
ChangeVersion(&meta_table_, 47, true);
// FALL THROUGH
+ case 47:
+ if (!keyword_table_->MigrateToVersion48RemoveKeywordsBackup())
+ return FailedMigrationTo(48);
+
+ ChangeVersion(&meta_table_, 48, true);
+ // FALL THROUGH
+
// Add successive versions here. Each should set the version number and
// compatible version number as appropriate, then fall through to the next
// case.

Powered by Google App Engine
This is Rietveld 408576698