Open
Bug 1390528
Opened 7 years ago
Updated 2 years ago
Remove AsyncStatementJSHelper and StatementJSHelper
Categories
(Toolkit :: Storage, enhancement, P3)
Toolkit
Storage
Tracking
()
NEW
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.
Updated•7 years ago
|
status-firefox57:
--- → wontfix
Priority: -- → P3
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•