Closed Bug 1031878 Opened 7 years ago Closed 7 years ago

JS_SetWatchPoint / JS_ClearWatchPoint / JS_ClearWatchPointsForObject can be removed

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla34

People

(Reporter: jimb, Assigned: evilpie)

References

Details

Attachments

(1 file)

JS_SetWatchPoint, JS_ClearWatchPoint, and JS_ClearWatchPointsForObject no longer have any callers, and can be removed.

The machinery that supports them needs to stick around, because it's still 
available via Object.prototype.watch.
Attached patch v1Splinter Review
We could probably simplify jswatchpoint.cpp a bit now, but let's first remove the API.
Assignee: nobody → evilpies
Status: NEW → ASSIGNED
Attachment #8461772 - Flags: review?(jimb)
Attachment #8461772 - Flags: review?(jimb) → review+
https://hg.mozilla.org/mozilla-central/rev/94e20e7d2782
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.