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

tests/html/indexeddb_4_test.dart

 #library('IndexedDB4Test');
 #import('../../pkg/unittest/unittest.dart');
 #import('../../pkg/unittest/html_config.dart');
 #import('dart:html');
 
 // Test for IDBKeyRange and IDBCursor.
 
 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.
+      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.
+      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) {
+      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)); }
 
   testRange(range, expectedFirst, expectedLast) {
     IDBTransaction txn = db.transaction(STORE_NAME, 'readonly');
     IDBObjectStore objectStore = txn.objectStore(STORE_NAME);
     IDBRequest cursorRequest = objectStore.openCursor(range);
     int itemCount = 0;
     int firstKey = null;
     int lastKey = null;
     cursorRequest.on.success.add(expectAsync1((e) {
       var cursor = e.target.result;
@@ skipping 46 matching lines @@
   bound5() =>
       testRange(new IDBKeyRange.bound(20, 30, lowerOpen: true, upperOpen: true),
                          21, 29);
 
 }
 
 main() {
   useHtmlConfiguration();
 
   var test_ = new Test();
-  test('prepare', test_.start);
+  test('prepare', test_.setupDb);
 
   test('only1', test_.only1);
   test('only2', test_.only2);
   test('only3', test_.only3);
 
   test('lower1', test_.lower1);
   test('lower2', test_.lower2);
   test('lower3', test_.lower3);
 
   test('upper1', test_.upper1);
   test('upper2', test_.upper2);
   test('upper3', test_.upper3);
 
   test('bound1', test_.bound1);
   test('bound2', test_.bound2);
   test('bound3', test_.bound3);
   test('bound4', test_.bound4);
   test('bound5', test_.bound5);
 }