Address open requests as well

[inexorabletash]

Implicitly, the proposal allows for:

let connection = await indexedDB.open('db').ready;

But that doesn't handle initial opens/upgrades, where you need to watch the upgradeneeded event on the request.

As a follow-on, we could do this:

interface IDBFactory {
  IDBOpenDBRequest open(DOMString name, [EnforceRange] optional unsigned long long version);
  IDBOpenDBRequest open(DOMString name, IDBOpenOptions options); // new overload!
  ...
}

dictionary IDBOpenOptions {
  optional unsigned long long version;
  optional UpgradeCallback upgrade;
};

callback UpgradeCallback = void (IDBDatabase connection,
                                 IDBTransaction transaction,
                                 unsigned long long oldVersion,
                                 unsigned long long newVersion);

So you'd write:

let connection = await indexedDB.open('db', {
  version: 2,
  upgrade: (db, tx, oldVersion, newVersion) => {
    if (oldVersion < 1) { ... }
    if (newVersion < 2) { ... }
  }}).ready;

[jakearchibald]

Strongly agree with having upgrade as part of the options object - much easier to reason-about as they say.

[inexorabletash]

Note that if we don't do this, developers would need to write:

let request = indexedDB.open('db', 2);
request.onupgradeneeded = (e) => {
  if (e.oldVersion < 1) { ... }
  if (e.oldVersion < 2) { ... }
};
let connection = await request.ready;

... which is not the end of the world, but it makes sense to pursue the event-less approach as well. I guess if you supply the upgrade callback we call it in addition to events, with the same semantics? (exception -> abort, etc)