Closed Bug 1390147 Opened 2 years ago Closed 2 years ago
Property Class hook from Statement JSHelper and Async Statement JSHelper
We want to remove these ClassOps, see bug 1389510. There's more to do for storage/ but these *StatementJSHelper ones are easy to fix. In bug 1389510 comment 3 bz mentioned we could remove these JSHelper classes completely and use IDL for the "params" and "row" properties. That's a lot more complicated though - the easiest fix is to replace the getProperty hook with a resolve hook that defines the property.
Attachment #8896982 - Flags: review?(mrbkap)
Comment on attachment 8896982 [details] [diff] [review] Patch Please do file follow bugs on getting rid of these scriptable helpers.
Attachment #8896982 - Flags: review?(mrbkap) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/35622c347169 Remove getProperty Class hook from StatementJSHelper and AsyncStatementJSHelper. r=mrbkap
(In reply to Blake Kaplan (:mrbkap) from comment #1) > Please do file follow bugs on getting rid of these scriptable helpers. Filed bug 1390528.
You need to log in before you can comment on or make changes to this bug.