Move setGCCallback from shell/js.cpp to TestingFunctions.cpp

RESOLVED FIXED in Firefox 42

Status

()

--
enhancement
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jruderman, Assigned: sfink)

Tracking

Trunk
mozilla42
Points:
---

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Bug 1117768 added setGCCallback() to js/shell.cpp. It should be moved to TestingFunctions.cpp, so it can also be called from browser-based fuzz tests.
Created attachment 8634878 [details] [diff] [review]
move.setgccallback.testing.patch

Here's the patch (it's trivial). If you want to keep ownership on this code, I am fine with inverting the author and reviewer fields, after you've r+ it (and if that's not too nasty).
Attachment #8634878 - Flags: review?(sphink)
(Assignee)

Comment 2

3 years ago
Comment on attachment 8634878 [details] [diff] [review]
move.setgccallback.testing.patch

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

So you're saying that as long as I don't tell you to do anything different, you're taking ownership of this code?

\o/  \o/  \o/

(sucker!)
Attachment #8634878 - Flags: review?(sphink) → review+
Logic error: syllogism detected, line 1. ;)
https://hg.mozilla.org/mozilla-central/rev/36c0d857aa86
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox42: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.