Reviewed by Brady.
authorandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Nov 2007 19:49:25 +0000 (19:49 +0000)
committerandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Nov 2007 19:49:25 +0000 (19:49 +0000)
        Get rid of SQLiteStatement::bindText and rename bindText16 to bindText.
        Remove the copy parameter from bindText and bindBlob and always copy passed in data instead.

        * loader/icon/IconDatabase.cpp:
        (WebCore::IconDatabase::setIconIDForPageURLInSQLDatabase):
        (WebCore::IconDatabase::removePageURLFromSQLDatabase):
        (WebCore::IconDatabase::getIconIDForIconURLFromSQLDatabase):
        (WebCore::IconDatabase::addIconURLToSQLDatabase):
        (WebCore::IconDatabase::getImageDataForIconURLFromSQLDatabase):
        (WebCore::IconDatabase::writeIconSnapshotToSQLDatabase):
        * platform/sql/SQLiteStatement.cpp:
        (WebCore::SQLiteStatement::prepare):
        * platform/sql/SQLiteStatement.h:
        * storage/Database.cpp:
        (WebCore::setTextValueInDatabase):
        * storage/DatabaseTracker.cpp:
        (WebCore::DatabaseTracker::fullPathForDatabase):
        (WebCore::DatabaseTracker::databaseNamesForOrigin):
        (WebCore::DatabaseTracker::addDatabase):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@27813 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/loader/icon/IconDatabase.cpp
WebCore/platform/sql/SQLiteStatement.cpp
WebCore/platform/sql/SQLiteStatement.h
WebCore/storage/Database.cpp
WebCore/storage/DatabaseTracker.cpp

index ed9e792d9fec1e7135f267eb58b259cfb9bd6758..f1166abe4f7cea814a347fbdba7941949c75eca2 100644 (file)
@@ -1,3 +1,27 @@
+2007-11-15  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by Brady.
+
+        Get rid of SQLiteStatement::bindText and rename bindText16 to bindText.
+        Remove the copy parameter from bindText and bindBlob and always copy passed in data instead.
+
+        * loader/icon/IconDatabase.cpp:
+        (WebCore::IconDatabase::setIconIDForPageURLInSQLDatabase):
+        (WebCore::IconDatabase::removePageURLFromSQLDatabase):
+        (WebCore::IconDatabase::getIconIDForIconURLFromSQLDatabase):
+        (WebCore::IconDatabase::addIconURLToSQLDatabase):
+        (WebCore::IconDatabase::getImageDataForIconURLFromSQLDatabase):
+        (WebCore::IconDatabase::writeIconSnapshotToSQLDatabase):
+        * platform/sql/SQLiteStatement.cpp:
+        (WebCore::SQLiteStatement::prepare):
+        * platform/sql/SQLiteStatement.h:
+        * storage/Database.cpp:
+        (WebCore::setTextValueInDatabase):
+        * storage/DatabaseTracker.cpp:
+        (WebCore::DatabaseTracker::fullPathForDatabase):
+        (WebCore::DatabaseTracker::databaseNamesForOrigin):
+        (WebCore::DatabaseTracker::addDatabase):
+
 2007-11-14  Brady Eidson <beidson@apple.com>
 
         Rubberstamped by Sam
index cdbc5be99dc5ee27360b9c0f6c6fd67230287e94..562e5b8f4576d511e748e0991f7a354b085b4028 100644 (file)
@@ -1809,7 +1809,7 @@ void IconDatabase::setIconIDForPageURLInSQLDatabase(int64_t iconID, const String
     ASSERT_ICON_SYNC_THREAD();
     
     readySQLiteStatement(m_setIconIDForPageURLStatement, m_syncDB, "INSERT INTO PageURL (url, iconID) VALUES ((?), ?);");
-    m_setIconIDForPageURLStatement->bindText16(1, pageURL, false);
+    m_setIconIDForPageURLStatement->bindText(1, pageURL);
     m_setIconIDForPageURLStatement->bindInt64(2, iconID);
 
     int result = m_setIconIDForPageURLStatement->step();
@@ -1826,7 +1826,7 @@ void IconDatabase::removePageURLFromSQLDatabase(const String& pageURL)
     ASSERT_ICON_SYNC_THREAD();
     
     readySQLiteStatement(m_removePageURLStatement, m_syncDB, "DELETE FROM PageURL WHERE url = (?);");
-    m_removePageURLStatement->bindText16(1, pageURL, false);
+    m_removePageURLStatement->bindText(1, pageURL);
 
     if (m_removePageURLStatement->step() != SQLResultDone)
         LOG_ERROR("removePageURLFromSQLDatabase failed for url %s", urlForLogging(pageURL).ascii().data());
@@ -1840,7 +1840,7 @@ int64_t IconDatabase::getIconIDForIconURLFromSQLDatabase(const String& iconURL)
     ASSERT_ICON_SYNC_THREAD();
     
     readySQLiteStatement(m_getIconIDForIconURLStatement, m_syncDB, "SELECT IconInfo.iconID FROM IconInfo WHERE IconInfo.url = (?);");
-    m_getIconIDForIconURLStatement->bindText16(1, iconURL, false);
+    m_getIconIDForIconURLStatement->bindText(1, iconURL);
     
     int64_t result = m_getIconIDForIconURLStatement->step();
     if (result == SQLResultRow)
@@ -1864,7 +1864,7 @@ int64_t IconDatabase::addIconURLToSQLDatabase(const String& iconURL)
     // here is unnecessary
     
     readySQLiteStatement(m_addIconToIconInfoStatement, m_syncDB, "INSERT INTO IconInfo (url, stamp) VALUES (?, 0);");
-    m_addIconToIconInfoStatement->bindText16(1, iconURL);
+    m_addIconToIconInfoStatement->bindText(1, iconURL);
     
     int result = m_addIconToIconInfoStatement->step();
     m_addIconToIconInfoStatement->reset();
@@ -1894,7 +1894,7 @@ PassRefPtr<SharedBuffer> IconDatabase::getImageDataForIconURLFromSQLDatabase(con
     RefPtr<SharedBuffer> imageData;
     
     readySQLiteStatement(m_getImageDataForIconURLStatement, m_syncDB, "SELECT IconData.data FROM IconData WHERE IconData.iconID IN (SELECT iconID FROM IconInfo WHERE IconInfo.url = (?));");
-    m_getImageDataForIconURLStatement->bindText16(1, iconURL, false);
+    m_getImageDataForIconURLStatement->bindText(1, iconURL);
     
     int result = m_getImageDataForIconURLStatement->step();
     if (result == SQLResultRow) {
@@ -1974,7 +1974,7 @@ void IconDatabase::writeIconSnapshotToSQLDatabase(const IconSnapshot& snapshot)
     if (iconID) {    
         readySQLiteStatement(m_updateIconInfoStatement, m_syncDB, "UPDATE IconInfo SET stamp = ?, url = ? WHERE iconID = ?;");
         m_updateIconInfoStatement->bindInt64(1, snapshot.timestamp);
-        m_updateIconInfoStatement->bindText16(2, snapshot.iconURL);
+        m_updateIconInfoStatement->bindText(2, snapshot.iconURL);
         m_updateIconInfoStatement->bindInt64(3, iconID);
 
         if (m_updateIconInfoStatement->step() != SQLResultDone)
@@ -1998,7 +1998,7 @@ void IconDatabase::writeIconSnapshotToSQLDatabase(const IconSnapshot& snapshot)
         m_updateIconDataStatement->reset();
     } else {    
         readySQLiteStatement(m_setIconInfoStatement, m_syncDB, "INSERT INTO IconInfo (url,stamp) VALUES (?, ?);");
-        m_setIconInfoStatement->bindText16(1, snapshot.iconURL);
+        m_setIconInfoStatement->bindText(1, snapshot.iconURL);
         m_setIconInfoStatement->bindInt64(2, snapshot.timestamp);
 
         if (m_setIconInfoStatement->step() != SQLResultDone)
index ce37e7192910ddb17e23eda122b3f0cedd878fe3..3926b89d58faf94ce21a46e172882cdc34b653b8 100644 (file)
@@ -121,33 +121,19 @@ bool SQLiteStatement::returnsAtLeastOneResult()
 
 }
 
-int SQLiteStatement::bindBlob(int index, const void* blob, int size, bool copy)
+int SQLiteStatement::bindBlob(int index, const void* blob, int size)
 {
     ASSERT(blob);
     ASSERT(size > -1);
-    if (copy)
-        sqlite3_bind_blob(m_statement, index, blob, size, SQLITE_TRANSIENT);
-    else
-        sqlite3_bind_blob(m_statement, index, blob, size, SQLITE_STATIC);
-    return lastError();
-}
 
-int SQLiteStatement::bindText(int index, const char* text, bool copy)
-{
-    ASSERT(text);
-    if (copy)
-        sqlite3_bind_text(m_statement, index, text, strlen(text), SQLITE_TRANSIENT);
-    else
-        sqlite3_bind_text(m_statement, index, text, strlen(text), SQLITE_STATIC);
+    sqlite3_bind_blob(m_statement, index, blob, size, SQLITE_TRANSIENT);
+
     return lastError();
 }
 
-int SQLiteStatement::bindText16(int index, const String& text, bool copy)
+int SQLiteStatement::bindText(int index, const String& text)
 {
-    if (copy)
-        sqlite3_bind_text16(m_statement, index, text.characters(), sizeof(UChar) * text.length(), SQLITE_TRANSIENT);
-    else
-        sqlite3_bind_text16(m_statement, index, text.characters(), sizeof(UChar) * text.length(), SQLITE_STATIC);
+    sqlite3_bind_text16(m_statement, index, text.characters(), sizeof(UChar) * text.length(), SQLITE_TRANSIENT);
     return lastError();
 }
 
@@ -171,7 +157,7 @@ int SQLiteStatement::bindValue(int index, const SQLValue& value)
 {
     switch (value.type()) {
         case SQLValue::StringValue:
-            return bindText16(index, value.string());
+            return bindText(index, value.string());
         case SQLValue::NumberValue:
             return bindDouble(index, value.number());
         case SQLValue::NullValue:
index 62859028c4600d6ff89607b72804208d0d826576..2dad82fc9013d698c3745bef79486d2c623f12db 100644 (file)
@@ -45,9 +45,8 @@ public:
     
     int prepare();
     bool isPrepared() const { return m_statement; }
-    int bindBlob(int index, const void* blob, int size, bool copy = true);
-    int bindText(int index, const char* text, bool copy = true);
-    int bindText16(int index, const String& text, bool copy = true);
+    int bindBlob(int index, const void* blob, int size);
+    int bindText(int index, const String& text);
     int bindInt64(int index, int64_t integer);
     int bindDouble(int index, double number);
     int bindNull(int index);
index 6ceba00d3382f6699ae8a8d69389c3da04c8e4bc..5415300d86292abc401f7a37d4be56301f741172 100644 (file)
@@ -221,7 +221,7 @@ static bool setTextValueInDatabase(SQLiteDatabase& db, const String& query, cons
         return false;
     }
 
-    statement.bindText16(1, value);
+    statement.bindText(1, value);
 
     result = statement.step();
     if (result != SQLResultDone) {
index ca59f17642b0d90f0edccfb8835789f7589828e1..79e755a0ec40f9e6c1ae6c230d171095dd4aaed7 100644 (file)
@@ -88,8 +88,8 @@ String DatabaseTracker::fullPathForDatabase(const String& origin, const String&
     if (statement.prepare() != SQLResultOk)
         return "";
 
-    statement.bindText16(1, origin, false);
-    statement.bindText16(2, name, false);
+    statement.bindText(1, origin);
+    statement.bindText(2, name);
 
     int result = statement.step();
 
@@ -172,7 +172,7 @@ bool DatabaseTracker::databaseNamesForOrigin(const String& origin, Vector<String
     if (statement.prepare() != SQLResultOk)
         return false;
 
-    statement.bindText16(1, origin, false);
+    statement.bindText(1, origin);
 
     int result;
     while ((result = statement.step()) == SQLResultRow)
@@ -196,9 +196,9 @@ bool DatabaseTracker::addDatabase(const String& origin, const String& name, cons
     if (statement.prepare() != SQLResultOk)
         return false;
 
-    statement.bindText16(1, origin, true);
-    statement.bindText16(2, name, true);
-    statement.bindText16(3, path, true);
+    statement.bindText(1, origin);
+    statement.bindText(2, name);
+    statement.bindText(3, path);
 
     if (!statement.executeCommand()) {
         LOG_ERROR("Failed to add database %s to origin %s: %s\n", name.ascii().data(), origin.ascii().data(), statement.lastErrorMsg());