14199 2011-09-18 18:24:14 +0000 IDBVersionChangeEvent still uses DOMString for version 2012-01-24 04:57:30 +0000 1 1 1 Unclassified WebAppsWG Indexed Database API unspecified All All RESOLVED FIXED P2 normal --- 1 me eliotgra eliotgra jonas mike Ms2ger public-webapps public-webapps-bugzilla oldest_to_newest 56948 0 me 2011-09-18 18:24:14 +0000 After the recent setVersion changes it should be using long long. 57421 1 me 2011-09-27 16:43:38 +0000 Actually, it should use 'any' for newVersion, since in a deleteDatabase call it is null. 59461 2 eliotgra 2011-11-02 00:41:05 +0000 Will make long long. 59487 3 eliotgra 2011-11-02 15:01:09 +0000 Fixed in the 1 November Editor's Draft: interface IDBVersionChangeEvent : Event { readonly attribute unsigned long long oldVersion; readonly attribute unsigned long long newVersion; void initIDBVersionChangeEvent (DOMString typeArg, boolean canBubbleArg, boolean cancelableArg, long long oldVersion, long long newVersion); }; I think newVersion should be long long rather than any. The only time it is null is when it is deleted. Thanks, Eliot 59494 4 me 2011-11-02 15:21:50 +0000 (In reply to comment #3) > Fixed in the 1 November Editor's Draft: > > interface IDBVersionChangeEvent : Event { > readonly attribute unsigned long long oldVersion; > readonly attribute unsigned long long newVersion; > void initIDBVersionChangeEvent (DOMString typeArg, boolean canBubbleArg, > boolean cancelableArg, long long oldVersion, long long newVersion); > }; > > I think newVersion should be long long rather than any. The only time it is > null is when it is deleted. It should be nullable ("unsigned long long?") then, right? /me admits to not knowing much about this whole WebIDL thing. 59497 5 Ms2ger 2011-11-02 15:48:39 +0000 (In reply to comment #4) > (In reply to comment #3) > > Fixed in the 1 November Editor's Draft: > > > > interface IDBVersionChangeEvent : Event { > > readonly attribute unsigned long long oldVersion; > > readonly attribute unsigned long long newVersion; > > void initIDBVersionChangeEvent (DOMString typeArg, boolean canBubbleArg, > > boolean cancelableArg, long long oldVersion, long long newVersion); > > }; > > > > I think newVersion should be long long rather than any. The only time it is > > null is when it is deleted. > > It should be nullable ("unsigned long long?") then, right? Yes. 62068 6 eliotgra 2011-12-27 22:03:28 +0000 made all appropriate elements nullable, in the following: IDBVersionChangeEvent.oldversion IDBVersionChangeEvent.newversion IDBVersionChangeEvent.initIDBVersionChangeEvent() IDBFactory.open() IDBDatabase.version IDBFactorySync.open() IDBDatabaseSync.version IDBVersionChangeCallback.transactionStarted() Changes are published in the Editor's Draft of Dec 27. Thanks, Eliot 63037 7 jonas 2012-01-24 04:57:30 +0000 Actually, only the newVersion property on the IDBVersionChangeEvent can be null. All other version properties/arguments are always non-null. Changed things to reflect this. I also removed the initIDBVersionChangeEvent function in favor of a constructor since that's how DOM-Events are initialized these days.