Raze old WebDatabases.

components/webdata/common/web_database.cc

 // Copyright (c) 2012 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 "components/webdata/common/web_database.h"
 
 #include <algorithm>
 
 #include "base/stl_util.h"
 #include "sql/statement.h"
 #include "sql/transaction.h"
 
 // Current version number.  Note: when changing the current version number,
 // corresponding changes must happen in the unit tests, and new migration test
 // added.  See |WebDatabaseMigrationTest::kCurrentTestedVersionNumber|.
 // static
 const int WebDatabase::kCurrentVersionNumber = 62;
 
+const int WebDatabase::kDeprecatedVersionNumber = 51;
+
 namespace {
 
 const int kCompatibleVersionNumber = 61;
 
 // Change the version number and possibly the compatibility version of
 // |meta_table_|.
 void ChangeVersion(sql::MetaTable* meta_table,
                    int version_num,
                    bool update_compatible_version_num) {
   meta_table->SetVersionNumber(version_num);
@@ skipping 51 matching lines @@
   // infrequent. So we go with a small cache size.
   db_.set_cache_size(32);
 
   // Run the database in exclusive mode. Nobody else should be accessing the
   // database while we're running, and this will give somewhat improved perf.
   db_.set_exclusive_locking();
 
   if (!db_.Open(db_name))
     return sql::INIT_FAILURE;
 
-  // Initialize various tables
+  // Clobber really old databases.
+  static_assert(kDeprecatedVersionNumber < kCurrentVersionNumber,
+                "Deprecation version must be less than current");
+  sql::MetaTable::RazeIfDeprecated(&db_, kDeprecatedVersionNumber);
+
+  // Scope initialization in a transaction so we can't be partially
+  // initialized.
   sql::Transaction transaction(&db_);
   if (!transaction.Begin())
     return sql::INIT_FAILURE;
 
   // Version check.
   if (!meta_table_.Init(&db_, kCurrentVersionNumber, kCompatibleVersionNumber))
     return sql::INIT_FAILURE;
   if (meta_table_.GetCompatibleVersionNumber() > kCurrentVersionNumber) {
     LOG(WARNING) << "Web database is too new.";
     return sql::INIT_TOO_NEW;
@@ skipping 27 matching lines @@
 
 sql::InitStatus WebDatabase::MigrateOldVersionsAsNeeded() {
   // Some malware used to lower the 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(),
                                  meta_table_.GetCompatibleVersionNumber());
   if (current_version > meta_table_.GetVersionNumber())
     ChangeVersion(&meta_table_, current_version, false);
 
-  if (current_version < 20) {
-    // Versions 1 - 19 are unhandled.  Version numbers greater than
-    // kCurrentVersionNumber should have already been weeded out by the caller.
-    //
-    // When the version is too old, we return failure error code.  The schema
-    // is too out of date to migrate.
-    //
-    // There should not be a released product that makes a database too old to
-    // migrate. If we do encounter such a legacy database, we will need a
-    // better solution to handle it (i.e., pop up a dialog to tell the user,
-    // erase all their prefs and start over, etc.).
-    LOG(WARNING) << "Web database version " << current_version
-                 << " is too old to handle.";
-    NOTREACHED();
-    return sql::INIT_FAILURE;
-  }
+  DCHECK_LT(current_version, kDeprecatedVersionNumber);

[Scott Hess - ex-Googler, 2015/02/28 00:19:32]

Probably meant DCHECK_GT(), or swap the parameters.

[Peter Kasting, 2015/02/28 00:26:20]

(Use the GT route rather than the param swap route)

[Evan Stade, 2015/02/28 00:29:08]

Done.

 
   for (int next_version = current_version + 1;
        next_version <= kCurrentVersionNumber;
        ++next_version) {
 
     // Do any database-wide migrations.
     bool update_compatible_version = false;
     if (!MigrateToVersion(next_version, &update_compatible_version))
       return FailedMigrationTo(next_version);
 
@@ skipping 28 matching lines @@
 
 bool WebDatabase::MigrateToVersion58DropWebAppsAndIntents() {
   sql::Transaction transaction(&db_);
   return transaction.Begin() &&
       db_.Execute("DROP TABLE IF EXISTS web_apps") &&
       db_.Execute("DROP TABLE IF EXISTS web_app_icons") &&
       db_.Execute("DROP TABLE IF EXISTS web_intents") &&
       db_.Execute("DROP TABLE IF EXISTS web_intents_defaults") &&
       transaction.Commit();
 }