Open Bug 1390528 Opened 7 years ago Updated 2 years ago

Remove AsyncStatementJSHelper and StatementJSHelper

Categories

(Toolkit :: Storage, enhancement, P3)

enhancement

Tracking

()

Tracking Status
firefox57 --- wontfix

People

(Reporter: jandem, Unassigned)

Details

See bug 1390147 and bug 1389510.

In bug 1389510 comment 3, bz said:

> That said, looking at these consumers...  AsyncStatementJSHelper only
> implements a getter so it can do it for the string "params".  That's dumb;
> we could just have IDL for this bit with an nsISupports return type, even
> without converting to webidl.  I other words, there's no real reason I see
> to have AsyncStatement be a proxy at all; there's a bunch of pointless
> complexity here.  There might be some issues about threadsafety and making
> sure things get released on the right threads, but that can be handled just
> as easily with a cached nsIWhatever pointer as with the JS object holder bit.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.