Open Bug 1390528 Opened 8 years ago Updated 1 year ago

Remove AsyncStatementJSHelper and StatementJSHelper

Categories

(Core :: SQLite and Embedded Database Bindings, 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
Product: Toolkit → Core
You need to log in before you can comment on or make changes to this bug.