[Bug 17649] Address WebIDL comments from Kyle Huey

https://www.w3.org/Bugs/Public/show_bug.cgi?id=17649

Eliot Graff <eliotgra@microsoft.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |eliotgra@microsoft.com
         AssignedTo|dave.null@w3.org            |jonas@sicking.cc

--- Comment #1 from Eliot Graff <eliotgra@microsoft.com> 2012-07-23 23:45:00 UTC ---
Jonas,

I'm assigning this to you. I've taken care of some of Kyle's feedback, but I'll
defer to you on things like the default values for the enums.


I've FIXED the following:

3.1.9

The return type of the static functions on IDBKeyRange is not 'static
IDBKeyRange', it is just 'IDBKeyRange'.

3.2.1

The correct type name is "object", not "Object" (note the capitalization).

IDBRequest.readyState should be an enum type, not a DOMString.

3.2.2

IDBVersionChangeEvent should probably reference whatever spec defines how
constructors work for DOM events.

3.2.7

"Object" should be "object".

3.2.8

IDBTransaction's mode attribute should be an enum type.



I DID NOT FIX the following items in the mail:

3.2.4

IDBDatabase.transaction's mode argument should be an enum type, with a
default value specified in IDL instead of in prose.

3.2.5

Is it intentional that IDBObjectStore.indexNames does not return the same
DOMStringList every time, unlike IDBDatabase.objectStoreNames (yes, I
realize that the circumstances under which the former can change are much
broader).

IDBObjectStore.openCursor's direction argument should be an enum type, with
a default value specified in IDL (right now it is unspecified).

3.2.6

IDBIndex.openCursor and IDBIndex.openKeyCursor's direction argument should
be an enum type, with a default value specified in IDL.



Also, it would be nice if we could tighten up keys from 'any' to a union.

-- 
Configure bugmail: https://www.w3.org/Bugs/Public/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.

Received on Monday, 23 July 2012 23:45:02 UTC