Update indexeddb tests to work with both methods of upgrading a database.

tests/html/indexeddb_3_test.dart

 #library('IndexedDB3Test');
 #import('../../pkg/unittest/unittest.dart');
 #import('../../pkg/unittest/html_config.dart');
 #import('dart:html');
 
 // Read with cursor.
 
 const String DB_NAME = 'Test';
 const String STORE_NAME = 'TEST';
-const String VERSION = '1';
+const int VERSION = 1;
 
 class Test {
+  fail(message) => (e) {
+    guardAsync(() {
+      Expect.fail('IndexedDB failure: $message'); 
+    });
+  };
+
+  _createObjectStore(db) {
+    try {
+      // Nuke object store if it already exists.
+      db.deleteObjectStore(STORE_NAME);
+    }
+    on IDBDatabaseException catch(e) { }  // Chrome
+    on DOMException catch(e) { }          // Firefox
+    db.createObjectStore(STORE_NAME);
+  }
+
   var db;
 
-  start() {
-    var request = window.indexedDB.open(DB_NAME);
-    Expect.isNotNull(request);
-    request.on.success.add(expectAsync1(initDb));
-    request.on.error.add(fail('open'));
+  _openDb(afterOpen()) {
+    var request = window.indexedDB.open(DB_NAME, VERSION);
+    if (request is IDBOpenDBRequest) {
+      // New upgrade protocol. FireFox 15, Chrome 24, hopefully IE10.
+      request.on.success.add(expectAsync1((e) {
+            db = e.target.result;
+            afterOpen();
+          }));
+      request.on.upgradeNeeded.add((e) {
+          guardAsync(() {
+              _createObjectStore(e.target.result);
+            });
+        });
+      request.on.error.add(fail('open'));
+    } else {
+      // Legacy setVersion upgrade protocol. Chrome < 23.
+      request.on.success.add(expectAsync1((e) {
+            db = e.target.result;
+            if (db.version != '$VERSION') {
+              var setRequest = db.setVersion('$VERSION');
+              setRequest.on.success.add(
+                  expectAsync1((e) {
+                      _createObjectStore(db);
+                      var transaction = e.target.result;
+                      transaction.on.complete.add(
+                          expectAsync1((e) => afterOpen()));
+                      transaction.on.error.add(fail('Upgrade'));
+                    }));
+              setRequest.on.error.add(fail('setVersion error'));
+            } else {
+              afterOpen();
+            }
+          }));
+      request.on.error.add(fail('open'));
+    }
   }
 
-  initDb(e) {
-    db = e.target.result;
-    // TODO.  Some browsers do this the w3 way - passing the VERSION to the
-    // open call and listening to onversionchange.  Can we feature-detect the
-    // difference and make it work?
-    var request = db.setVersion(VERSION);
-    request.on.success.add(
-      expectAsync1((e) {
-        try {
-          // Nuke object store if it already exists.
-          db.deleteObjectStore(STORE_NAME);
-        } on IDBDatabaseException catch(e) { }
-        db.createObjectStore(STORE_NAME);
-
-        var transaction = e.target.result;
-        transaction.on.complete.add(expectAsync1((e) => writeItems(0)));
-        transaction.on.error.add(fail);
-      })
-    );
-    request.on.error.add(fail('setVersion error'));
+  _createAndOpenDb(afterOpen()) {
+    var request = window.indexedDB.deleteDatabase(DB_NAME);
+    request.on.success.add(expectAsync1((e) { _openDb(afterOpen); }));
+    request.on.error.add(fail('delete old Db'));
   }
 
   writeItems(int index) {
     if (index < 100) {
       var transaction = db.transaction([STORE_NAME], 'readwrite');
       var request = transaction.objectStore(STORE_NAME)
           .put('Item $index', index);
       request.on.success.add(expectAsync1((e) {
           writeItems(index + 1);
         }
       ));
       request.on.error.add(fail('put'));
     }
   }
 
-  fail(message) => (e) {
-    guardAsync(() {
-      Expect.fail('IndexedDB failure: $message'); 
-    });
-  };
+  setupDb() { _createAndOpenDb(() => writeItems(0)); }
 
   readAllViaCursor() {
     IDBTransaction txn = db.transaction(STORE_NAME, 'readonly');
     IDBObjectStore objectStore = txn.objectStore(STORE_NAME);
     IDBRequest cursorRequest = objectStore.openCursor();
     int itemCount = 0;
     int sumKeys = 0;
     int lastKey = null;
     cursorRequest.on.success.add(expectAsync1((e) {
       var cursor = e.target.result;
       if (cursor != null) {
         lastKey = cursor.key;
         itemCount += 1;
         sumKeys += cursor.key;
+        window.console.log('${cursor.key}  ${cursor.value}');
         Expect.equals('Item ${cursor.key}', cursor.value);
         cursor.continueFunction();
       } else {
         // Done
         Expect.equals(99, lastKey);
         Expect.equals(100, itemCount);
         Expect.equals((100 * 99) ~/ 2, sumKeys);
       }
     }, count:101));
     cursorRequest.on.error.add(fail('openCursor'));
@@ skipping 23 matching lines @@
       }
     }, count:101));
     cursorRequest.on.error.add(fail('openCursor'));
   }
 }
 
 main() {
   useHtmlConfiguration();
 
   var test_ = new Test();
-  test('prepare', test_.start);
+  test('prepare', test_.setupDb);
   test('readAll1', test_.readAllViaCursor);
   test('readAll2', test_.readAllReversedViaCursor);
 }