Title: [119603] trunk
Revision
119603
Author
[email protected]
Date
2012-06-06 12:00:04 -0700 (Wed, 06 Jun 2012)

Log Message

IndexedDB: Rename IDBFactory.getDatabaseNames() to webkitGetDatabaseNames()
https://bugs.webkit.org/show_bug.cgi?id=86874

Reviewed by Ojan Vafai.

Source/WebCore:

The function is non-standard, and should be prefixed.

Test: storage/indexeddb/factory-basics.html

* Modules/indexeddb/IDBFactory.idl:

LayoutTests:

* storage/indexeddb/factory-basics-expected.txt:
* storage/indexeddb/resources/factory-basics.js:
(test):
(openSuccess):
(deleteDatabaseSuccess):

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (119602 => 119603)


--- trunk/LayoutTests/ChangeLog	2012-06-06 18:59:35 UTC (rev 119602)
+++ trunk/LayoutTests/ChangeLog	2012-06-06 19:00:04 UTC (rev 119603)
@@ -1,3 +1,16 @@
+2012-06-06  Joshua Bell  <[email protected]>
+
+        IndexedDB: Rename IDBFactory.getDatabaseNames() to webkitGetDatabaseNames()
+        https://bugs.webkit.org/show_bug.cgi?id=86874
+
+        Reviewed by Ojan Vafai.
+
+        * storage/indexeddb/factory-basics-expected.txt:
+        * storage/indexeddb/resources/factory-basics.js:
+        (test):
+        (openSuccess):
+        (deleteDatabaseSuccess):
+
 2012-06-06  Ryosuke Niwa  <[email protected]>
 
         Chromium rebaseline after r119591 because it was failing with missing results.

Modified: trunk/LayoutTests/storage/indexeddb/factory-basics-expected.txt (119602 => 119603)


--- trunk/LayoutTests/storage/indexeddb/factory-basics-expected.txt	2012-06-06 18:59:35 UTC (rev 119602)
+++ trunk/LayoutTests/storage/indexeddb/factory-basics-expected.txt	2012-06-06 19:00:04 UTC (rev 119603)
@@ -7,20 +7,21 @@
 
 PASS typeof indexedDB.open is "function"
 PASS typeof indexedDB.cmp is "function"
-PASS typeof indexedDB.getDatabaseNames is "function"
 PASS typeof indexedDB.deleteDatabase is "function"
-indexedDB.getDatabaseNames()
+PASS typeof indexedDB.webkitGetDatabaseNames is "function"
+PASS typeof indexedDB.getDatabaseNames is "undefined"
+indexedDB.webkitGetDatabaseNames()
 databaseNames = event.target.result
 PASS databaseNames.contains('storage/indexeddb/factory-basics') is false
 PASS databaseNames.contains('DATABASE THAT DOES NOT EXIST') is false
 indexedDB.open(name)
 event.target.result.close()
-indexedDB.getDatabaseNames()
+indexedDB.webkitGetDatabaseNames()
 databaseNames = event.target.result
 PASS databaseNames.contains('storage/indexeddb/factory-basics') is true
 PASS databaseNames.contains('DATABASE THAT DOES NOT EXIST') is false
 indexedDB.deleteDatabase('storage/indexeddb/factory-basics')
-indexedDB.getDatabaseNames()
+indexedDB.webkitGetDatabaseNames()
 databaseNames = event.target.result
 PASS databaseNames.contains('storage/indexeddb/factory-basics') is false
 PASS databaseNames.contains('DATABASE THAT DOES NOT EXIST') is false

Modified: trunk/LayoutTests/storage/indexeddb/resources/factory-basics.js (119602 => 119603)


--- trunk/LayoutTests/storage/indexeddb/resources/factory-basics.js	2012-06-06 18:59:35 UTC (rev 119602)
+++ trunk/LayoutTests/storage/indexeddb/resources/factory-basics.js	2012-06-06 19:00:04 UTC (rev 119603)
@@ -11,13 +11,16 @@
 
     shouldBeEqualToString("typeof indexedDB.open", "function");
     shouldBeEqualToString("typeof indexedDB.cmp", "function");
-    shouldBeEqualToString("typeof indexedDB.getDatabaseNames", "function");
-
     shouldBeEqualToString("typeof indexedDB.deleteDatabase", "function");
 
+    // Non-standard, must be prefixed
+    shouldBeEqualToString("typeof indexedDB.webkitGetDatabaseNames", "function");
+    shouldBeEqualToString("typeof indexedDB.getDatabaseNames", "undefined");
+
+
     name = 'storage/indexeddb/factory-basics';
 
-    request = evalAndLog("indexedDB.getDatabaseNames()");
+    request = evalAndLog("indexedDB.webkitGetDatabaseNames()");
     request._onsuccess_ = getDatabaseNamesSuccess1;
     request._onerror_ = unexpectedErrorCallback;
 }
@@ -37,7 +40,7 @@
 function openSuccess()
 {
     evalAndLog("event.target.result.close()");
-    request = evalAndLog("indexedDB.getDatabaseNames()");
+    request = evalAndLog("indexedDB.webkitGetDatabaseNames()");
     request._onsuccess_ = getDatabaseNamesSuccess2;
     request._onerror_ = unexpectedErrorCallback;
 }
@@ -56,7 +59,7 @@
 
 function deleteDatabaseSuccess()
 {
-    request = evalAndLog("indexedDB.getDatabaseNames()");
+    request = evalAndLog("indexedDB.webkitGetDatabaseNames()");
     request._onsuccess_ = getDatabaseNamesSuccess3;
     request._onerror_ = unexpectedErrorCallback;
 }
@@ -78,4 +81,4 @@
     finishJSTest();
 }
 
-test();
\ No newline at end of file
+test();

Modified: trunk/Source/WebCore/ChangeLog (119602 => 119603)


--- trunk/Source/WebCore/ChangeLog	2012-06-06 18:59:35 UTC (rev 119602)
+++ trunk/Source/WebCore/ChangeLog	2012-06-06 19:00:04 UTC (rev 119603)
@@ -1,3 +1,16 @@
+2012-06-06  Joshua Bell  <[email protected]>
+
+        IndexedDB: Rename IDBFactory.getDatabaseNames() to webkitGetDatabaseNames()
+        https://bugs.webkit.org/show_bug.cgi?id=86874
+
+        Reviewed by Ojan Vafai.
+
+        The function is non-standard, and should be prefixed.
+
+        Test: storage/indexeddb/factory-basics.html
+
+        * Modules/indexeddb/IDBFactory.idl:
+
 2012-06-06  Julien Chaffraix  <[email protected]>
 
         Add support for direction on table row group with collapsing borders

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBFactory.idl (119602 => 119603)


--- trunk/Source/WebCore/Modules/indexeddb/IDBFactory.idl	2012-06-06 18:59:35 UTC (rev 119602)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBFactory.idl	2012-06-06 19:00:04 UTC (rev 119603)
@@ -28,7 +28,7 @@
     interface [
         Conditional=INDEXED_DATABASE
     ] IDBFactory {
-        [CallWith=ScriptExecutionContext] IDBRequest getDatabaseNames();
+        [CallWith=ScriptExecutionContext, ImplementedAs=getDatabaseNames] IDBRequest webkitGetDatabaseNames();
 
         [CallWith=ScriptExecutionContext] IDBRequest open(in DOMString name)
             raises (IDBDatabaseException);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to