Rename needsBarrier to needsIncrementalBarrier

RESOLVED FIXED in mozilla34

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: terrence, Assigned: terrence)

Tracking

(Blocks 1 bug)

Trunk
mozilla34
Points:
---
Dependency tree / graph
Bug Flags:
qe-verify -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

This is a bit confusing: the incremental isBarrierNeeded flag is public in JS::shadow::Runtime so that we can inline  it for ExposeToActiveJS and MarkGCThingAsLive, yet the only actual barriers we expose to the browser are ggc's post barriers. I've renamed needsBarrier to needsIncrementalBarrier to hopefully make this clearer.
Attachment #8464367 - Flags: review?(jcoppeard)
Attachment #8464367 - Flags: review?(jcoppeard) → review+
https://hg.mozilla.org/mozilla-central/rev/b58d03207432
Status: ASSIGNED → RESOLVED
Closed: 5 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.