IDB: storage/indexeddb/mozilla/object-cursors.html fails
authorbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 9 Feb 2014 06:08:37 +0000 (06:08 +0000)
committerbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 9 Feb 2014 06:08:37 +0000 (06:08 +0000)
<rdar://problem/16020585> and https://bugs.webkit.org/show_bug.cgi?id=128479

Reviewed by Dan Bernstein.

Source/WebKit2:

* DatabaseProcess/IndexedDB/sqlite/UniqueIDBDatabaseBackingStoreSQLite.cpp:
(WebKit::UniqueIDBDatabaseBackingStoreSQLite::createAndPopulateInitialMetadata): Update the IndexInfo
  table schema to allow for indexes with the same id/name as previous indexes.
  But with different object stores, of course.

LayoutTests:

* platform/mac-wk2/TestExpectations: Enable the test.

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

LayoutTests/ChangeLog
LayoutTests/platform/mac-wk2/TestExpectations
Source/WebKit2/ChangeLog
Source/WebKit2/DatabaseProcess/IndexedDB/sqlite/UniqueIDBDatabaseBackingStoreSQLite.cpp

index daa7d14..c00e0ed 100644 (file)
@@ -1,5 +1,14 @@
 2014-02-08  Brady Eidson  <beidson@apple.com>
 
+        IDB: storage/indexeddb/mozilla/object-cursors.html fails
+        <rdar://problem/16020585> and https://bugs.webkit.org/show_bug.cgi?id=128479
+
+        Reviewed by Dan Bernstein.
+
+        * platform/mac-wk2/TestExpectations: Enable the test.
+
+2014-02-08  Brady Eidson  <beidson@apple.com>
+
         IDB: storage/indexeddb/mozilla/index-prev-no-duplicate.html fails
         <rdar://problem/16018918> and https://bugs.webkit.org/show_bug.cgi?id=128444
 
index ad31717..e976afa 100644 (file)
@@ -479,6 +479,7 @@ storage/indexeddb/mozilla/cursor-mutation-objectstore-only.html [ Pass ]
 storage/indexeddb/mozilla/cursor-mutation.html [ Pass ]
 storage/indexeddb/mozilla/cursors.html [ Pass ]
 storage/indexeddb/mozilla/index-prev-no-duplicate.html [ Pass ]
+storage/indexeddb/mozilla/object-cursors.html [ Pass ]
 storage/indexeddb/mozilla/versionchange-abort.html [ Pass ]
 
 ### END OF (5) Features that are not supported in WebKit1, so skipped in mac/TestExpectations then re-enabled here
index 1e6ea9d..4937acd 100644 (file)
@@ -1,5 +1,17 @@
 2014-02-08  Brady Eidson  <beidson@apple.com>
 
+        IDB: storage/indexeddb/mozilla/object-cursors.html fails
+        <rdar://problem/16020585> and https://bugs.webkit.org/show_bug.cgi?id=128479
+
+        Reviewed by Dan Bernstein.
+
+        * DatabaseProcess/IndexedDB/sqlite/UniqueIDBDatabaseBackingStoreSQLite.cpp:
+        (WebKit::UniqueIDBDatabaseBackingStoreSQLite::createAndPopulateInitialMetadata): Update the IndexInfo
+          table schema to allow for indexes with the same id/name as previous indexes.
+          But with different object stores, of course.
+
+2014-02-08  Brady Eidson  <beidson@apple.com>
+
         IDB: storage/indexeddb/mozilla/index-prev-no-duplicate.html fails
         <rdar://problem/16018918> and https://bugs.webkit.org/show_bug.cgi?id=128444
 
index 0fdcc24..3edf46c 100644 (file)
@@ -94,7 +94,7 @@ std::unique_ptr<IDBDatabaseMetadata> UniqueIDBDatabaseBackingStoreSQLite::create
         return nullptr;
     }
 
-    if (!m_sqliteDB->executeCommand("CREATE TABLE IndexInfo (id INTEGER PRIMARY KEY NOT NULL ON CONFLICT FAIL, name TEXT NOT NULL ON CONFLICT FAIL UNIQUE ON CONFLICT FAIL, objectStoreID INTEGER NOT NULL ON CONFLICT FAIL, keyPath BLOB NOT NULL ON CONFLICT FAIL, isUnique INTEGER NOT NULL ON CONFLICT FAIL, multiEntry INTEGER NOT NULL ON CONFLICT FAIL);")) {
+    if (!m_sqliteDB->executeCommand("CREATE TABLE IndexInfo (id INTEGER NOT NULL ON CONFLICT FAIL, name TEXT NOT NULL ON CONFLICT FAIL, objectStoreID INTEGER NOT NULL ON CONFLICT FAIL, keyPath BLOB NOT NULL ON CONFLICT FAIL, isUnique INTEGER NOT NULL ON CONFLICT FAIL, multiEntry INTEGER NOT NULL ON CONFLICT FAIL);")) {
         LOG_ERROR("Could not create IndexInfo table in database (%i) - %s", m_sqliteDB->lastError(), m_sqliteDB->lastErrorMsg());
         m_sqliteDB = nullptr;
         return nullptr;