Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: terrence, Assigned: terrence)

Tracking

Trunk
mozilla46
Points:
---

Firefox Tracking Flags

(firefox46 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Created attachment 8707974 [details] [diff] [review]
remove_HashKeyRef-v0.diff

All of the simple cases that used this have been converted over to stable hash codes.
Attachment #8707974 - Flags: review?(jcoppeard)
(Assignee)

Comment 1

3 years ago
Comment on attachment 8707974 [details] [diff] [review]
remove_HashKeyRef-v0.diff

Review of attachment 8707974 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/gc/Marking.h
@@ -431,5 @@
> -            return;
> -        TraceManuallyBarrieredEdge(trc, &key, "HashKeyRef");
> -        map->rekeyIfMoved(prior, key);
> -    }
> -};

On a personal note: \o/
Comment on attachment 8707974 [details] [diff] [review]
remove_HashKeyRef-v0.diff

Review of attachment 8707974 [details] [diff] [review]:
-----------------------------------------------------------------

Nice!
Attachment #8707974 - Flags: review?(jcoppeard) → review+
(Assignee)

Comment 4

3 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/3d8d87127bc2383d2c9380f6af19b53752b9acf5
Bug 1239754 - Remove HashKeyRef now that all of its users are using stable hashing; r=jonco

Comment 6

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3d8d87127bc2
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox46: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.