Re: [Bug 11348] New: [IndexedDB] Overhaul of the event model

On Fri, Feb 11, 2011 at 11:30 AM, ben turner <bent.mozilla@gmail.com> wrote:
> It looks like I was wrong. Our current impl throws NOT_ALLOWED_ERR for
> getting errorCode *and* result before readyState is set to DONE.
>
> And now that I think about it I think I like that best. If we returned
> NO_ERR from errorCode before DONE then it seems to imply that the
> request succeeded when the reality is we don't yet know. Checking
> errorCode before DONE is most likely a bug in the page script just as
> calling result before DONE, so I'm happy with throwing here.
>
> Sound ok?

Ah, I thought that's what you were saying in your previous email :-)

I.e. throw when it's almost surely a bug in the script (reading too
early), and return 0/undefined once there is a result of some sort.

Sounds ok to me.

/ Jonas

Received on Friday, 11 February 2011 19:39:55 UTC