Generalize migration code in WebDatabase.

chrome/browser/webdata/web_database_table.h

Index: chrome/browser/webdata/web_database_table.h
diff --git a/chrome/browser/webdata/web_database_table.h b/chrome/browser/webdata/web_database_table.h
index a1b875d491c54eb90824e6697bfc271aa7f0b9b3..6f940b2cf59f20abcb50359a039e21ad909f7534 100644
--- a/chrome/browser/webdata/web_database_table.h
+++ b/chrome/browser/webdata/web_database_table.h
@@ -15,7 +15,7 @@ class MetaTable;
 // An abstract base class representing a table within a WebDatabase.
 // Each table should subclass this, adding type-specific methods as needed.
 class WebDatabaseTable {
- protected:
+ public:
   WebDatabaseTable(sql::Connection* db, sql::MetaTable* meta_table);
   virtual ~WebDatabaseTable();
 
@@ -29,6 +29,21 @@ class WebDatabaseTable {
   // TODO(andybons): Implement something more robust.
   virtual bool IsSyncable() = 0;
 
+  // Migrates this table to |version|. Returns false if there was
+  // migration work to do and it failed, true otherwise.
+  //
+  // |app_locale| is the locale of the app. Passed as a parameter as
+  // |it can only be safely queried on the UI thread.
+  //
+  // Implementations may set |*update_compatible_version| to true if
+  // the compatible version should be changed to

[dhollowa, 2013/03/12 01:38:16]

nit: This line wraps too early.

[Jói, 2013/03/12 02:00:44]

Done.

+  // |version|. Implementations should otherwise not modify this
+  // parameter.
+  virtual bool MigrateToVersion(int version,
+                                const std::string& app_locale,
+                                bool* update_compatible_version) = 0;
+
+ protected:
   sql::Connection* db_;
   sql::MetaTable* meta_table_;