Re: [Bug 10430] New: [IndexedDB] We need to make it more clear IDBRequests can be reused and spec readyState's behavior

On Wed, Aug 25, 2010 at 4:41 PM, Jeremy Orlow <jorlow@chromium.org> wrote:

> On Wed, Aug 25, 2010 at 4:12 PM, Jeremy Orlow <jorlow@chromium.org> wrote:
>
>> Also, the constants for the ready state should start with 0, not 1.
>>
>> Also, what happens if someone does the following:
>> request = objectStore.openCursor();
>> request.onsuccess(function() {
>>     event.result.continue();
>>     request.abort();
>>     event.result.continue();
>> }
>>
>
> After thinking about this more, I think we have 2 options: either .abort()
> kills all pending and future requests on that IDBRequest object or it
> cancels all requests that have been made up until that point.  I.e. if we
> did
>
> event.result.continue();
> event.result.continue();
> event.abort();
> event.result.continue();
> event.result.continue();
>
> Either the 3rd and 4th results should be returned or no results should be
> returned, depending on what we decide here.
>
> Unless someone can present a good use case for why abort shouldn't abort
> pending and future requests, I suggest we go with that behavior.
>

Sorry for the spam, but is there any reason that ABORTED is not one of the
readyStates?


> Also, is it possible to get the IDBRequest object from within the
>> openCursor/continue events without saving it off somewhere?  If not, it
>> probably should be.
>>
>> J
>>
>> On Wed, Aug 25, 2010 at 3:50 PM, <bugzilla@jessica.w3.org> wrote:
>>
>>> http://www.w3.org/Bugs/Public/show_bug.cgi?id=10430
>>>
>>>           Summary: [IndexedDB] We need to make it more clear IDBRequests
>>>                    can be reused and spec readyState's behavior
>>>           Product: WebAppsWG
>>>           Version: unspecified
>>>          Platform: PC
>>>        OS/Version: All
>>>            Status: NEW
>>>          Severity: normal
>>>          Priority: P2
>>>         Component: Indexed Database API
>>>        AssignedTo: andreip@google.com
>>>        ReportedBy: jorlow@chromium.org
>>>         QAContact: member-webapi-cvs@w3.org
>>>                CC: mike@w3.org, public-webapps@w3.org
>>>
>>>
>>> http://www.w3.org/TR/IndexedDB/#idl-def-IDBRequest right now clearly
>>> assumes
>>> IDBRequests are only going to be used once.  But IDBCursor.continue()
>>> reuses
>>> them.  At the very least, we'll need to clean up the .readyState section
>>> to
>>> make it clear it's possible to go from DONE to LOADING (or we can remove
>>> it
>>> altogether?).  I think some notes explicitly pointing out that some parts
>>> of
>>> the API reuse them would be helpful as well.
>>>
>>> --
>>> Configure bugmail: http://www.w3.org/Bugs/Public/userprefs.cgi?tab=email
>>> ------- You are receiving this mail because: -------
>>> You are on the CC list for the bug.
>>>
>>>
>>
>