Move migration code out of WebDatabase and into table-specific classes.

chrome/browser/webdata/web_database.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_WEBDATA_WEB_DATABASE_H_
 #define CHROME_BROWSER_WEBDATA_WEB_DATABASE_H_
 #pragma once
 
 #include "app/sql/connection.h"
 #include "app/sql/init_status.h"
@@ skipping 25 matching lines @@
   virtual AutofillTable* GetAutofillTable();
   virtual KeywordTable* GetKeywordTable();
   virtual LoginsTable* GetLoginsTable();
   virtual TokenServiceTable* GetTokenServiceTable();
   virtual WebAppsTable* GetWebAppsTable();
 
   // Exposed for testing only.
   sql::Connection* GetSQLConnection();
 
  private:
+  // Change the version number and possibly the compatibility version of

[dhollowa, 2011/04/04 16:13:20]

These would be better pulled into the .cc as utility functions within the
anonymous namespace.

[Bons, 2011/04/05 11:05:13]

Done.

+  // |meta_table_|.
+  void ChangeVersion(int version_num, bool update_compatible_version_num);
+
+  // Outputs the failed version number as a warning and always returns
+  // |sql::INIT_FAILURE|.
+  sql::InitStatus FailedMigrationTo(int version_num);
+
   // Used by |Init()| to migration database schema from older versions to
   // current version.
   sql::InitStatus MigrateOldVersionsAsNeeded();
 
   sql::Connection db_;
   sql::MetaTable meta_table_;
 
   scoped_ptr<AutofillTable> autofill_table_;
   scoped_ptr<KeywordTable> keyword_table_;
   scoped_ptr<LoginsTable> logins_table_;
   scoped_ptr<TokenServiceTable> token_service_table_;
   scoped_ptr<WebAppsTable> web_apps_table_;
 
   scoped_ptr<NotificationService> notification_service_;
 
   DISALLOW_COPY_AND_ASSIGN(WebDatabase);
 };
 
 #endif  // CHROME_BROWSER_WEBDATA_WEB_DATABASE_H_