Upgrade the history database to version 23

chrome/browser/history/android/android_urls_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 "chrome/browser/history/android/android_urls_database.h"
 
 #include "base/logging.h"
 
 namespace history {
 
 AndroidURLsDatabase::AndroidURLsDatabase() {
 }
 
 AndroidURLsDatabase::~AndroidURLsDatabase() {
 }
 
 bool AndroidURLsDatabase::CreateAndroidURLsTable() {
   const char* name = "android_urls";
   if (!GetDB().DoesTableExist(name)) {
     std::string sql;
-    sql.append("CREATE TABLE ");
+    sql.append("CREATE TABLE IF NOT EXISTS ");

[sky, 2012/04/13 21:32:05]

How come you need this if not exists? Doesn't line 19 handle this?

[michaelbai, 2012/04/13 23:02:35]

Yes, they are not needed.

On 2012/04/13 21:32:05, sky wrote:

     sql.append(name);
     sql.append("("
                "id INTEGER PRIMARY KEY,"
                "raw_url LONGVARCHAR,"              // Passed in raw url.
                "url_id INTEGER NOT NULL"           // url id in urls table.
                ")");
     if (!GetDB().Execute(sql.c_str())) {
       LOG(ERROR) << GetDB().GetErrorMessage();
       return false;
     }
 
-    if (!GetDB().Execute("CREATE INDEX android_urls_raw_url_idx"
+    if (!GetDB().Execute("CREATE INDEX IF NOT EXISTS android_urls_raw_url_idx"
                          " ON android_urls(raw_url)")) {
       LOG(ERROR) << GetDB().GetErrorMessage();
       return false;
     }
 
-    if (!GetDB().Execute("CREATE INDEX android_urls_url_id_idx"
+    if (!GetDB().Execute("CREATE INDEX IF NOT EXISTS android_urls_url_id_idx"
                          " ON android_urls(url_id)")) {
       LOG(ERROR) << GetDB().GetErrorMessage();
       return false;
     }
   }
   return true;
 }
 
 AndroidURLID AndroidURLsDatabase::AddAndroidURLRow(const std::string& raw_url,
                                                    URLID url_id) {
@@ skipping 87 matching lines @@
     return false;
   }
 
   return true;
 }
 
 bool AndroidURLsDatabase::ClearAndroidURLRows() {
   return GetDB().Execute("DELETE FROM android_urls");
 }
 
+bool AndroidURLsDatabase::MigrateToVersion22() {
+  if (!GetDB().DoesTableExist("android_urls"))
+    return true;
+
+  if (!GetDB().Execute("ALTER TABLE android_urls RENAME TO android_urls_tmp"))
+    return false;
+
+  if (!CreateAndroidURLsTable())
+    return false;
+
+  if (!GetDB().Execute(
+      "INSERT INTO android_urls (id, raw_url, url_id) "
+      "SELECT id, raw_url, url_id FROM android_urls_tmp"))
+    return false;
+
+  if (!GetDB().Execute("DROP TABLE android_urls_tmp"))
+    return false;
+
+  return true;
+}
+
 }  // namespace history