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

tests/html/indexeddb_2_test.dart

 #library('IndexedDB1Test');
 #import('../../pkg/unittest/unittest.dart');
 #import('../../pkg/unittest/html_config.dart');
 #import('dart:html');
 #import('dart:coreimpl');
 #import('utils.dart');
 
 // Write and re-read Maps: simple Maps; Maps with DAGs; Maps with cycles.
 
 const String DB_NAME = 'Test';
 const String STORE_NAME = 'TEST';
-const String VERSION = '1';
+const int VERSION = 1;
 
 testReadWrite(key, value, check,
               [dbName = DB_NAME,
                storeName = STORE_NAME,
                version = VERSION]) => () {
   var db;
 
   fail(e) {
     guardAsync(() {
-      Expect.fail('IndexedDB failure');
+      throw const Exception('IndexedDB failure');
     });
   }
 
-  createObjectStore() {
+  createObjectStore(db) {
     var store = db.createObjectStore(storeName);
-    Expect.isNotNull(store);
+    expect(store, isNotNull);
   }
 
   step2(e) {
     var transaction = db.transaction(storeName, 'readonly');
     var request = transaction.objectStore(storeName).getObject(key);
     request.on.success.add(expectAsync1((e) {
-        var object = e.target.result;
-        check(value, object);
-      })
-    );
+      var object = e.target.result;
+      db.close();
+      check(value, object);
+    }));
     request.on.error.add(fail);
   }
 
   step1() {
     var transaction = db.transaction([storeName], 'readwrite');
     var request = transaction.objectStore(storeName).put(value, key);
     request.on.success.add(expectAsync1(step2));
     request.on.error.add(fail);
   }
 
   initDb(e) {
     db = e.target.result;
     if (version != db.version) {
-      // 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);
+      // Legacy 'setVersion' upgrade protocol.
+      var request = db.setVersion('$version');
       request.on.success.add(
         expectAsync1((e) {
-          createObjectStore();
-
+          createObjectStore(db);
           var transaction = e.target.result;
           transaction.on.complete.add(expectAsync1((e) => step1()));
           transaction.on.error.add(fail);
         })
       );
       request.on.error.add(fail);
     } else {
       step1();
     }
   }
 
-  var request = window.indexedDB.open(dbName);
-  Expect.isNotNull(request);
-  request.on.success.add(expectAsync1(initDb));
-  request.on.error.add(fail);
+  openDb(e) {
+    var request = window.indexedDB.open(dbName, version);
+    expect(request, isNotNull);
+    request.on.success.add(expectAsync1(initDb));
+    request.on.error.add(fail);
+    if (request is IDBOpenDBRequest) {
+      // New upgrade protocol.  Old API has no 'upgradeNeeded' and uses
+      // setVersion instead.
+      request.on.upgradeNeeded.add((e) {
+          guardAsync(() {
+              createObjectStore(e.target.result);
+            });
+        });
+    }
+  }
+
+  // Delete any existing DB.
+  var deleteRequest = window.indexedDB.deleteDatabase(dbName);
+  deleteRequest.on.success.add(expectAsync1(openDb));
+  deleteRequest.on.error.add(fail);
 };
 
 
 main() {
   useHtmlConfiguration();
 
   var obj1 = {'a': 100, 'b': 's'};
   var obj2 = {'x': obj1, 'y': obj1};  // DAG.
 
   var obj3 = {};
   obj3['a'] = 100;
   obj3['b'] = obj3;  // Cycle.
 
   var obj4 = new SplayTreeMap<String, Dynamic>();  // Different implementation.
   obj4['a'] = 100;
   obj4['b'] = 's';
 
   var cyclic_list = [1, 2, 3];
   cyclic_list[1] = cyclic_list;
 
   go(name, data) => test(name, testReadWrite(123, data, verifyGraph));
 
   test('test_verifyGraph', () {
       // Nice to know verifyGraph is working before we rely on it.
-      Expect.mapEquals(obj4, obj4);
       verifyGraph(obj4, obj4);
       verifyGraph(obj1, new Map.from(obj1));
       verifyGraph(obj4, new Map.from(obj4));
 
       var l1 = [1,2,3];
       var l2 = [const [1, 2, 3], const [1, 2, 3]];
       verifyGraph([l1, l1], l2);
-      Expect.throws(() => verifyGraph([[1, 2, 3], [1, 2, 3]], l2));
+      expect(() => verifyGraph([[1, 2, 3], [1, 2, 3]], l2), throws);
 
       verifyGraph(cyclic_list, cyclic_list);
     });
 
   go('test_simple', obj1);
   go('test_DAG', obj2);
   go('test_cycle', obj3);
   go('test_simple_splay', obj4);
   go('const_array_1', const [const [1], const [2]]);
   go('const_array_dag', const [const [1], const [1]]);
   go('array_deferred_copy', [1,2,3, obj3, obj3, 6]);
   go('array_deferred_copy_2', [1,2,3, [4, 5, obj3], [obj3, 6]]);
   go('cyclic_list', cyclic_list);
 }