Title: [158348] trunk/Source
Revision
158348
Author
beid...@apple.com
Date
2013-10-30 23:27:02 -0700 (Wed, 30 Oct 2013)

Log Message

IDB Database versions are uint64_t, not int64_t
https://bugs.webkit.org/show_bug.cgi?id=123556

Reviewed by Alexey Proskuryakov.

Source/WebCore:

* Modules/indexeddb/IDBBackingStoreInterface.h:

* Modules/indexeddb/IDBDatabaseBackendImpl.cpp:
(WebCore::IDBDatabaseBackendImpl::processPendingCalls):
(WebCore::IDBDatabaseBackendImpl::openConnection):
* Modules/indexeddb/IDBDatabaseBackendImpl.h:

* Modules/indexeddb/IDBFactoryBackendInterface.h:

* Modules/indexeddb/IDBPendingOpenCall.h:
(WebCore::IDBPendingOpenCall::create):
(WebCore::IDBPendingOpenCall::version):
(WebCore::IDBPendingOpenCall::IDBPendingOpenCall):

* Modules/indexeddb/IDBTransactionBackendOperations.cpp:
(WebCore::IDBDatabaseBackendImpl::VersionChangeOperation::perform):

* Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.cpp:
(WebCore::IDBBackingStoreLevelDB::updateIDBDatabaseVersion):
* Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.h:

* Modules/indexeddb/leveldb/IDBFactoryBackendLevelDB.cpp:
(WebCore::IDBFactoryBackendLevelDB::open):
* Modules/indexeddb/leveldb/IDBFactoryBackendLevelDB.h:

Source/WebKit2:

* WebProcess/Databases/IndexedDB/WebIDBFactoryBackend.cpp:
(WebKit::WebIDBFactoryBackend::open):
* WebProcess/Databases/IndexedDB/WebIDBFactoryBackend.h:

* WebProcess/Databases/IndexedDB/WebProcessIDBDatabaseBackend.cpp:
(WebKit::WebProcessIDBDatabaseBackend::openConnection):
* WebProcess/Databases/IndexedDB/WebProcessIDBDatabaseBackend.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (158347 => 158348)


--- trunk/Source/WebCore/ChangeLog	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/ChangeLog	2013-10-31 06:27:02 UTC (rev 158348)
@@ -1,3 +1,35 @@
+2013-10-30  Brady Eidson  <beid...@apple.com>
+
+        IDB Database versions are uint64_t, not int64_t
+        https://bugs.webkit.org/show_bug.cgi?id=123556
+
+        Reviewed by Alexey Proskuryakov.
+
+        * Modules/indexeddb/IDBBackingStoreInterface.h:
+
+        * Modules/indexeddb/IDBDatabaseBackendImpl.cpp:
+        (WebCore::IDBDatabaseBackendImpl::processPendingCalls):
+        (WebCore::IDBDatabaseBackendImpl::openConnection):
+        * Modules/indexeddb/IDBDatabaseBackendImpl.h:
+
+        * Modules/indexeddb/IDBFactoryBackendInterface.h:
+
+        * Modules/indexeddb/IDBPendingOpenCall.h:
+        (WebCore::IDBPendingOpenCall::create):
+        (WebCore::IDBPendingOpenCall::version):
+        (WebCore::IDBPendingOpenCall::IDBPendingOpenCall):
+
+        * Modules/indexeddb/IDBTransactionBackendOperations.cpp:
+        (WebCore::IDBDatabaseBackendImpl::VersionChangeOperation::perform):
+
+        * Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.cpp:
+        (WebCore::IDBBackingStoreLevelDB::updateIDBDatabaseVersion):
+        * Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.h:
+
+        * Modules/indexeddb/leveldb/IDBFactoryBackendLevelDB.cpp:
+        (WebCore::IDBFactoryBackendLevelDB::open):
+        * Modules/indexeddb/leveldb/IDBFactoryBackendLevelDB.h:
+
 2013-10-30  Brendan Long  <b.l...@cablelabs.com>
 
         [Gtk] Build is failing after r158317

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBBackingStoreInterface.h (158347 => 158348)


--- trunk/Source/WebCore/Modules/indexeddb/IDBBackingStoreInterface.h	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBBackingStoreInterface.h	2013-10-31 06:27:02 UTC (rev 158348)
@@ -90,7 +90,7 @@
     virtual bool keyExistsInIndex(IDBBackingStoreInterface::Transaction&, int64_t databaseId, int64_t objectStoreId, int64_t indexId, const IDBKey&, RefPtr<IDBKey>& foundPrimaryKey, bool& exists) = 0;
 
     virtual bool deleteDatabase(const String& name) = 0;
-    virtual bool updateIDBDatabaseIntVersion(IDBBackingStoreInterface::Transaction&, int64_t rowId, int64_t intVersion) = 0;
+    virtual bool updateIDBDatabaseVersion(IDBBackingStoreInterface::Transaction&, int64_t rowId, uint64_t version) = 0;
 
     virtual bool getRecord(IDBBackingStoreInterface::Transaction&, int64_t databaseId, int64_t objectStoreId, const IDBKey&, Vector<char>& record) = 0;
     virtual bool putRecord(IDBBackingStoreInterface::Transaction&, int64_t databaseId, int64_t objectStoreId, const IDBKey&, PassRefPtr<SharedBuffer> value, IDBRecordIdentifier*) = 0;

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBDatabaseBackendImpl.cpp (158347 => 158348)


--- trunk/Source/WebCore/Modules/indexeddb/IDBDatabaseBackendImpl.cpp	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBDatabaseBackendImpl.cpp	2013-10-31 06:27:02 UTC (rev 158348)
@@ -391,8 +391,7 @@
 void IDBDatabaseBackendImpl::processPendingCalls()
 {
     if (m_pendingSecondHalfOpen) {
-        // FIXME: Database versions are now of type uint64_t, but this code expected int64_t.
-        ASSERT(m_pendingSecondHalfOpen->version() == (int64_t)m_metadata.version);
+        ASSERT(m_pendingSecondHalfOpen->version() == m_metadata.version);
         ASSERT(m_metadata.id != InvalidId);
         m_pendingSecondHalfOpen->callbacks()->onSuccess(this, this->metadata());
         m_pendingSecondHalfOpen.release();
@@ -439,7 +438,7 @@
     m_transactions.add(transactionId, transaction.get());
 }
 
-void IDBDatabaseBackendImpl::openConnection(PassRefPtr<IDBCallbacks> prpCallbacks, PassRefPtr<IDBDatabaseCallbacks> prpDatabaseCallbacks, int64_t transactionId, int64_t version)
+void IDBDatabaseBackendImpl::openConnection(PassRefPtr<IDBCallbacks> prpCallbacks, PassRefPtr<IDBDatabaseCallbacks> prpDatabaseCallbacks, int64_t transactionId, uint64_t version)
 {
     ASSERT(m_backingStore.get());
     RefPtr<IDBCallbacks> callbacks = prpCallbacks;
@@ -460,7 +459,7 @@
             if (version == IDBDatabaseMetadata::NoIntVersion)
                 message = "Internal error opening database with no version specified.";
             else
-                message = String::format("Internal error opening database with version %lld", static_cast<long long>(version));
+                message = String::format("Internal error opening database with version %llu", static_cast<unsigned long long>(version));
             callbacks->onError(IDBDatabaseError::create(IDBDatabaseException::UnknownError, message));
             return;
         }
@@ -488,20 +487,17 @@
         version = 1;
     }
 
-    // FIXME: Database versions are now of type uint64_t, but this code expected int64_t.
-    if (version > (int64_t)m_metadata.version) {
+    if (version > m_metadata.version) {
         runIntVersionChangeTransaction(callbacks, databaseCallbacks, transactionId, version);
         return;
     }
 
-    // FIXME: Database versions are now of type uint64_t, but this code expected int64_t.
-    if (version < (int64_t)m_metadata.version) {
-        callbacks->onError(IDBDatabaseError::create(IDBDatabaseException::VersionError, String::format("The requested version (%lld) is less than the existing version (%lld).", static_cast<long long>(version), static_cast<long long>(m_metadata.version))));
+    if (version < m_metadata.version) {
+        callbacks->onError(IDBDatabaseError::create(IDBDatabaseException::VersionError, String::format("The requested version (%llu) is less than the existing version (%llu).", static_cast<unsigned long long>(version), static_cast<unsigned long long>(m_metadata.version))));
         return;
     }
 
-    // FIXME: Database versions are now of type uint64_t, but this code expected int64_t.
-    ASSERT(version == (int64_t)m_metadata.version);
+    ASSERT(version == m_metadata.version);
     m_databaseCallbacksSet.add(databaseCallbacks);
     callbacks->onSuccess(this, this->metadata());
 }

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBDatabaseBackendImpl.h (158347 => 158348)


--- trunk/Source/WebCore/Modules/indexeddb/IDBDatabaseBackendImpl.h	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBDatabaseBackendImpl.h	2013-10-31 06:27:02 UTC (rev 158348)
@@ -57,7 +57,7 @@
     virtual void addIndex(int64_t objectStoreId, const IDBIndexMetadata&, int64_t newMaxIndexId) OVERRIDE;
     virtual void removeIndex(int64_t objectStoreId, int64_t indexId) OVERRIDE;
 
-    void openConnection(PassRefPtr<IDBCallbacks>, PassRefPtr<IDBDatabaseCallbacks>, int64_t transactionId, int64_t version);
+    void openConnection(PassRefPtr<IDBCallbacks>, PassRefPtr<IDBDatabaseCallbacks>, int64_t transactionId, uint64_t version);
     void deleteDatabase(PassRefPtr<IDBCallbacks>);
 
     // IDBDatabaseBackendInterface

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBFactoryBackendInterface.h (158347 => 158348)


--- trunk/Source/WebCore/Modules/indexeddb/IDBFactoryBackendInterface.h	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBFactoryBackendInterface.h	2013-10-31 06:27:02 UTC (rev 158348)
@@ -59,7 +59,7 @@
     virtual ~IDBFactoryBackendInterface() { }
 
     virtual void getDatabaseNames(PassRefPtr<IDBCallbacks>, PassRefPtr<SecurityOrigin>, ScriptExecutionContext*, const String& dataDir) = 0;
-    virtual void open(const String& name, int64_t version, int64_t transactionId, PassRefPtr<IDBCallbacks>, PassRefPtr<IDBDatabaseCallbacks>, const SecurityOrigin& openingOrigin, const SecurityOrigin& mainFrameOrigin) = 0;
+    virtual void open(const String& name, uint64_t version, int64_t transactionId, PassRefPtr<IDBCallbacks>, PassRefPtr<IDBDatabaseCallbacks>, const SecurityOrigin& openingOrigin, const SecurityOrigin& mainFrameOrigin) = 0;
     virtual void deleteDatabase(const String& name, PassRefPtr<IDBCallbacks>, PassRefPtr<SecurityOrigin>, ScriptExecutionContext*, const String& dataDir) = 0;
 
     virtual void removeIDBDatabaseBackend(const String& uniqueIdentifier) = 0;

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBPendingOpenCall.h (158347 => 158348)


--- trunk/Source/WebCore/Modules/indexeddb/IDBPendingOpenCall.h	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBPendingOpenCall.h	2013-10-31 06:27:02 UTC (rev 158348)
@@ -38,17 +38,17 @@
 
 class IDBPendingOpenCall {
 public:
-    static PassOwnPtr<IDBPendingOpenCall> create(IDBCallbacks& callbacks, IDBDatabaseCallbacks& databaseCallbacks, int64_t transactionId, int64_t version)
+    static PassOwnPtr<IDBPendingOpenCall> create(IDBCallbacks& callbacks, IDBDatabaseCallbacks& databaseCallbacks, int64_t transactionId, uint64_t version)
     {
         return adoptPtr(new IDBPendingOpenCall(callbacks, databaseCallbacks, transactionId, version));
     }
     IDBCallbacks* callbacks() { return m_callbacks.get(); }
     IDBDatabaseCallbacks* databaseCallbacks() { return m_databaseCallbacks.get(); }
-    int64_t version() { return m_version; }
+    uint64_t version() { return m_version; }
     int64_t transactionId() const { return m_transactionId; }
 
 private:
-    IDBPendingOpenCall(IDBCallbacks& callbacks, IDBDatabaseCallbacks& databaseCallbacks, int64_t transactionId, int64_t version)
+    IDBPendingOpenCall(IDBCallbacks& callbacks, IDBDatabaseCallbacks& databaseCallbacks, int64_t transactionId, uint64_t version)
         : m_callbacks(&callbacks)
         , m_databaseCallbacks(&databaseCallbacks)
         , m_version(version)
@@ -57,7 +57,7 @@
     }
     RefPtr<IDBCallbacks> m_callbacks;
     RefPtr<IDBDatabaseCallbacks> m_databaseCallbacks;
-    int64_t m_version;
+    uint64_t m_version;
     const int64_t m_transactionId;
 };
 

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBTransactionBackendOperations.cpp (158347 => 158348)


--- trunk/Source/WebCore/Modules/indexeddb/IDBTransactionBackendOperations.cpp	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBTransactionBackendOperations.cpp	2013-10-31 06:27:02 UTC (rev 158348)
@@ -352,7 +352,7 @@
     // FIXME: Database versions are now of type uint64_t, but this code expected int64_t.
     ASSERT(m_version > (int64_t)oldVersion);
     database.setCurrentVersion(m_version);
-    if (!database.backingStore()->updateIDBDatabaseIntVersion(m_transaction->backingStoreTransaction(), databaseId, database.metadata().version)) {
+    if (!database.backingStore()->updateIDBDatabaseVersion(m_transaction->backingStoreTransaction(), databaseId, database.metadata().version)) {
         RefPtr<IDBDatabaseError> error = IDBDatabaseError::create(IDBDatabaseException::UnknownError, "Error writing data to stable storage when updating version.");
         m_callbacks->onError(error);
         m_transaction->abort(error);

Modified: trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.cpp (158347 => 158348)


--- trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.cpp	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.cpp	2013-10-31 06:27:02 UTC (rev 158348)
@@ -592,12 +592,12 @@
     return true;
 }
 
-bool IDBBackingStoreLevelDB::updateIDBDatabaseIntVersion(IDBBackingStoreInterface::Transaction& transaction, int64_t rowId, int64_t intVersion)
+bool IDBBackingStoreLevelDB::updateIDBDatabaseVersion(IDBBackingStoreInterface::Transaction& transaction, int64_t rowId, uint64_t version)
 {
-    if (intVersion == IDBDatabaseMetadata::NoIntVersion)
-        intVersion = IDBDatabaseMetadata::DefaultIntVersion;
-    ASSERT_WITH_MESSAGE(intVersion >= 0, "intVersion was %lld", static_cast<long long>(intVersion));
-    putVarInt(Transaction::levelDBTransactionFrom(transaction), DatabaseMetaDataKey::encode(rowId, DatabaseMetaDataKey::UserIntVersion), intVersion);
+    if (version == IDBDatabaseMetadata::NoIntVersion)
+        version = IDBDatabaseMetadata::DefaultIntVersion;
+    ASSERT_WITH_MESSAGE(version >= 0, "version was %llu", static_cast<unsigned long long>(version));
+    putVarInt(Transaction::levelDBTransactionFrom(transaction), DatabaseMetaDataKey::encode(rowId, DatabaseMetaDataKey::UserIntVersion), version);
     return true;
 }
 

Modified: trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.h (158347 => 158348)


--- trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.h	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.h	2013-10-31 06:27:02 UTC (rev 158348)
@@ -73,7 +73,7 @@
     virtual bool getIDBDatabaseMetaData(const String& name, IDBDatabaseMetadata*, bool& success) OVERRIDE WARN_UNUSED_RETURN;
     virtual bool createIDBDatabaseMetaData(const String& name, const String& version, int64_t intVersion, int64_t& rowId) OVERRIDE;
     virtual bool updateIDBDatabaseMetaData(IDBBackingStoreInterface::Transaction&, int64_t rowId, const String& version);
-    virtual bool updateIDBDatabaseIntVersion(IDBBackingStoreInterface::Transaction&, int64_t rowId, int64_t intVersion) OVERRIDE;
+    virtual bool updateIDBDatabaseVersion(IDBBackingStoreInterface::Transaction&, int64_t rowId, uint64_t version) OVERRIDE;
     virtual bool deleteDatabase(const String& name) OVERRIDE;
 
     virtual bool getObjectStores(int64_t databaseId, IDBDatabaseMetadata::ObjectStoreMap*) OVERRIDE WARN_UNUSED_RETURN;

Modified: trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBFactoryBackendLevelDB.cpp (158347 => 158348)


--- trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBFactoryBackendLevelDB.cpp	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBFactoryBackendLevelDB.cpp	2013-10-31 06:27:02 UTC (rev 158348)
@@ -162,7 +162,7 @@
     return 0;
 }
 
-void IDBFactoryBackendLevelDB::open(const String& name, int64_t version, int64_t transactionId, PassRefPtr<IDBCallbacks> callbacks, PassRefPtr<IDBDatabaseCallbacks> databaseCallbacks, const SecurityOrigin& openingOrigin, const SecurityOrigin&)
+void IDBFactoryBackendLevelDB::open(const String& name, uint64_t version, int64_t transactionId, PassRefPtr<IDBCallbacks> callbacks, PassRefPtr<IDBDatabaseCallbacks> databaseCallbacks, const SecurityOrigin& openingOrigin, const SecurityOrigin&)
 {
     LOG(StorageAPI, "IDBFactoryBackendLevelDB::open");
     const String uniqueIdentifier = computeUniqueIdentifier(name, openingOrigin);

Modified: trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBFactoryBackendLevelDB.h (158347 => 158348)


--- trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBFactoryBackendLevelDB.h	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebCore/Modules/indexeddb/leveldb/IDBFactoryBackendLevelDB.h	2013-10-31 06:27:02 UTC (rev 158348)
@@ -59,7 +59,7 @@
     virtual void removeIDBDatabaseBackend(const String& uniqueIdentifier) OVERRIDE FINAL;
 
     virtual void getDatabaseNames(PassRefPtr<IDBCallbacks>, PassRefPtr<SecurityOrigin>, ScriptExecutionContext*, const String& dataDir) OVERRIDE FINAL;
-    virtual void open(const String& name, int64_t version, int64_t transactionId, PassRefPtr<IDBCallbacks>, PassRefPtr<IDBDatabaseCallbacks>, const SecurityOrigin& openingOrigin, const SecurityOrigin& mainFrameOrigin) OVERRIDE FINAL;
+    virtual void open(const String& name, uint64_t version, int64_t transactionId, PassRefPtr<IDBCallbacks>, PassRefPtr<IDBDatabaseCallbacks>, const SecurityOrigin& openingOrigin, const SecurityOrigin& mainFrameOrigin) OVERRIDE FINAL;
 
     virtual void deleteDatabase(const String& name, PassRefPtr<IDBCallbacks>, PassRefPtr<SecurityOrigin>, ScriptExecutionContext*, const String& dataDir) OVERRIDE FINAL;
 

Modified: trunk/Source/WebKit2/ChangeLog (158347 => 158348)


--- trunk/Source/WebKit2/ChangeLog	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebKit2/ChangeLog	2013-10-31 06:27:02 UTC (rev 158348)
@@ -1,3 +1,18 @@
+2013-10-30  Brady Eidson  <beid...@apple.com>
+
+        IDB Database versions are uint64_t, not int64_t
+        https://bugs.webkit.org/show_bug.cgi?id=123556
+
+        Reviewed by Alexey Proskuryakov.
+
+        * WebProcess/Databases/IndexedDB/WebIDBFactoryBackend.cpp:
+        (WebKit::WebIDBFactoryBackend::open):
+        * WebProcess/Databases/IndexedDB/WebIDBFactoryBackend.h:
+
+        * WebProcess/Databases/IndexedDB/WebProcessIDBDatabaseBackend.cpp:
+        (WebKit::WebProcessIDBDatabaseBackend::openConnection):
+        * WebProcess/Databases/IndexedDB/WebProcessIDBDatabaseBackend.h:
+
 2013-10-30  Dan Bernstein  <m...@apple.com>
 
         Fixed a potential race condition around the initializtion of WKObject’s target.

Modified: trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebIDBFactoryBackend.cpp (158347 => 158348)


--- trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebIDBFactoryBackend.cpp	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebIDBFactoryBackend.cpp	2013-10-31 06:27:02 UTC (rev 158348)
@@ -70,7 +70,7 @@
     notImplemented();
 }
 
-void WebIDBFactoryBackend::open(const String& databaseName, int64_t version, int64_t transactionId, PassRefPtr<IDBCallbacks> callbacks, PassRefPtr<IDBDatabaseCallbacks> databaseCallbacks, const SecurityOrigin& securityOrigin, const SecurityOrigin&)
+void WebIDBFactoryBackend::open(const String& databaseName, uint64_t version, int64_t transactionId, PassRefPtr<IDBCallbacks> callbacks, PassRefPtr<IDBDatabaseCallbacks> databaseCallbacks, const SecurityOrigin& securityOrigin, const SecurityOrigin&)
 {
     ASSERT(isMainThread());
     LOG(StorageAPI, "WebIDBFactoryBackend::open");

Modified: trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebIDBFactoryBackend.h (158347 => 158348)


--- trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebIDBFactoryBackend.h	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebIDBFactoryBackend.h	2013-10-31 06:27:02 UTC (rev 158348)
@@ -40,7 +40,7 @@
     virtual ~WebIDBFactoryBackend();
 
     virtual void getDatabaseNames(PassRefPtr<WebCore::IDBCallbacks>, PassRefPtr<WebCore::SecurityOrigin>, WebCore::ScriptExecutionContext*, const String& dataDir) OVERRIDE;
-    virtual void open(const String& name, int64_t version, int64_t transactionId, PassRefPtr<WebCore::IDBCallbacks>, PassRefPtr<WebCore::IDBDatabaseCallbacks>, const WebCore::SecurityOrigin& openingOrigin, const WebCore::SecurityOrigin& mainFrameOrigin) OVERRIDE;
+    virtual void open(const String& name, uint64_t version, int64_t transactionId, PassRefPtr<WebCore::IDBCallbacks>, PassRefPtr<WebCore::IDBDatabaseCallbacks>, const WebCore::SecurityOrigin& openingOrigin, const WebCore::SecurityOrigin& mainFrameOrigin) OVERRIDE;
     virtual void deleteDatabase(const String& name, PassRefPtr<WebCore::IDBCallbacks>, PassRefPtr<WebCore::SecurityOrigin>, WebCore::ScriptExecutionContext*, const String& dataDir) OVERRIDE;
 
     virtual void removeIDBDatabaseBackend(const String& uniqueIdentifier) OVERRIDE;

Modified: trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebProcessIDBDatabaseBackend.cpp (158347 => 158348)


--- trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebProcessIDBDatabaseBackend.cpp	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebProcessIDBDatabaseBackend.cpp	2013-10-31 06:27:02 UTC (rev 158348)
@@ -59,7 +59,7 @@
 {
 }
 
-void WebProcessIDBDatabaseBackend::openConnection(PassRefPtr<IDBCallbacks> prpCallbacks, PassRefPtr<IDBDatabaseCallbacks> prpDatabaseCallbacks, int64_t transactionId, int64_t version)
+void WebProcessIDBDatabaseBackend::openConnection(PassRefPtr<IDBCallbacks> prpCallbacks, PassRefPtr<IDBDatabaseCallbacks> prpDatabaseCallbacks, int64_t transactionId, uint64_t version)
 {
     // FIXME: Make this message include necessary arguments, and save off these callbacks and arguments for later use.
 

Modified: trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebProcessIDBDatabaseBackend.h (158347 => 158348)


--- trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebProcessIDBDatabaseBackend.h	2013-10-31 06:25:12 UTC (rev 158347)
+++ trunk/Source/WebKit2/WebProcess/Databases/IndexedDB/WebProcessIDBDatabaseBackend.h	2013-10-31 06:27:02 UTC (rev 158348)
@@ -86,7 +86,7 @@
 
     virtual WebCore::IDBFactoryBackendInterface& factoryBackend() OVERRIDE;
 
-    void openConnection(PassRefPtr<WebCore::IDBCallbacks>, PassRefPtr<WebCore::IDBDatabaseCallbacks>, int64_t transactionId, int64_t version);
+    void openConnection(PassRefPtr<WebCore::IDBCallbacks>, PassRefPtr<WebCore::IDBDatabaseCallbacks>, int64_t transactionId, uint64_t version);
 
     void establishDatabaseProcessBackend();
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to