IDB: storage/indexeddb/mozilla/versionchange-abort.html fails
authorbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 9 Feb 2014 00:01:16 +0000 (00:01 +0000)
committerbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 9 Feb 2014 00:01:16 +0000 (00:01 +0000)
<rdar://problem/16018887> and https://bugs.webkit.org/show_bug.cgi?id=128442

Reviewed by Dan Bernstein.

Source/WebCore:

Tested by storage/indexeddb/mozilla/versionchange-abort.html (and probably others)

* Modules/indexeddb/IDBDatabase.cpp:
(WebCore::IDBDatabase::version): If the version is NoIntVersion, return DefaultIntVersion to script.

LayoutTests:

* platform/mac-wk2/TestExpectations: Enable the test.
  Also add all the other storage/indexeddb/mozilla test with expectations as they
  are what I’m focusing on right now.

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

LayoutTests/ChangeLog
LayoutTests/platform/mac-wk2/TestExpectations
Source/WebCore/ChangeLog
Source/WebCore/Modules/indexeddb/IDBDatabase.cpp

index b24f8a1..e120080 100644 (file)
@@ -1,5 +1,16 @@
 2014-02-08  Brady Eidson  <beidson@apple.com>
 
+        IDB: storage/indexeddb/mozilla/versionchange-abort.html fails
+        <rdar://problem/16018887> and https://bugs.webkit.org/show_bug.cgi?id=128442
+
+        Reviewed by Dan Bernstein.
+
+        * platform/mac-wk2/TestExpectations: Enable the test.
+          Also add all the other storage/indexeddb/mozilla test with expectations as they
+          are what I’m focusing on right now.
+
+2014-02-08  Brady Eidson  <beidson@apple.com>
+
         IDB: storage/indexeddb/mozilla/cursors.html fails
         <rdar://problem/16017998> and https://bugs.webkit.org/show_bug.cgi?id=128423
 
index d0b0aee..26ce45b 100644 (file)
@@ -478,6 +478,7 @@ storage/indexeddb/mozilla/create-index-with-integer-keys.html [ Pass ]
 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/versionchange-abort.html [ Pass ]
 
 ### END OF (5) Features that are not supported in WebKit1, so skipped in mac/TestExpectations then re-enabled here
 ########################################
index 7424af1..c5d5557 100644 (file)
@@ -1,5 +1,17 @@
 2014-02-08  Brady Eidson  <beidson@apple.com>
 
+        IDB: storage/indexeddb/mozilla/versionchange-abort.html fails
+        <rdar://problem/16018887> and https://bugs.webkit.org/show_bug.cgi?id=128442
+
+        Reviewed by Dan Bernstein.
+
+        Tested by storage/indexeddb/mozilla/versionchange-abort.html (and probably others)
+
+        * Modules/indexeddb/IDBDatabase.cpp:
+        (WebCore::IDBDatabase::version): If the version is NoIntVersion, return DefaultIntVersion to script.
+
+2014-02-08  Brady Eidson  <beidson@apple.com>
+
         IDB: storage/indexeddb/mozilla/cursors.html fails
         <rdar://problem/16017998> and https://bugs.webkit.org/show_bug.cgi?id=128423
 
index 83e47df..d22bde6 100644 (file)
@@ -135,7 +135,9 @@ PassRefPtr<DOMStringList> IDBDatabase::objectStoreNames() const
 
 uint64_t IDBDatabase::version() const
 {
-    return m_metadata.version;
+    // NoIntVersion is a special value for internal use only and shouldn't be exposed to script.
+    // DefaultIntVersion should be exposed instead.
+    return m_metadata.version != IDBDatabaseMetadata::NoIntVersion ? m_metadata.version : IDBDatabaseMetadata::DefaultIntVersion;
 }
 
 PassRefPtr<IDBObjectStore> IDBDatabase::createObjectStore(const String& name, const Dictionary& options, ExceptionCode& ec)