Closed Bug 1501387 Opened 10 months ago Closed 10 months ago

Handlify Stream.h

Categories

(Core :: JavaScript Engine, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: jorendorff, Assigned: jorendorff)

References

Details

Attachments

(1 file)

The API presented in js/public/Stream.h is inconsistent. Sometimes parameters are handles; other times raw pointers. Sometimes the pointers are `const JSObject *`.

We should use handles whenever a `cx` is present, and not `const JSObject *` as that doesn't really express any helpful or dependable API guarantee.

As it turns out, all existing code passes rooted pointers anyway, so no downstream code needs to be changed when handlifying this stuff.
Assignee: nobody → jorendorff
Priority: -- → P1
Pushed by jorendorff@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9ca97be22a38
Handlify Streams.h and avoid using `const JSObject*`. r=tcampbell
https://hg.mozilla.org/mozilla-central/rev/9ca97be22a38
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Blocks: 1499813
You need to log in before you can comment on or make changes to this bug.