Last Comment Bug 721933 - add way to invoke a compartmental GC via the API with a reason
: add way to invoke a compartmental GC via the API with a reason
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla13
Assigned To: Andrew McCreight [:mccr8]
:
Mentors:
Depends on:
Blocks: 716014
  Show dependency treegraph
 
Reported: 2012-01-27 17:52 PST by Andrew McCreight [:mccr8]
Modified: 2012-02-02 06:50 PST (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
WIP (1.63 KB, patch)
2012-01-31 10:44 PST, Andrew McCreight [:mccr8]
wmccloskey: review+
Details | Diff | Splinter Review

Description Andrew McCreight [:mccr8] 2012-01-27 17:52:45 PST
Compartmental GCs are awesome!  GCs with API reasons are awesome!  Let's combine them!
Comment 1 Andrew McCreight [:mccr8] 2012-01-31 10:44:09 PST
Created attachment 593158 [details] [diff] [review]
WIP

Should be pretty simple.
Comment 2 Andrew McCreight [:mccr8] 2012-02-01 12:41:58 PST
I cargo culted everything over from JS_CompartmentGC that compiled.  Let me know if you think I should add more, or define GCForReason in terms of this or whatever else.
Comment 3 Bill McCloskey (:billm) 2012-02-01 13:23:33 PST
Comment on attachment 593158 [details] [diff] [review]
WIP

No, this is good.
Comment 4 Andrew McCreight [:mccr8] 2012-02-01 17:58:34 PST
https://hg.mozilla.org/integration/mozilla-inbound/rev/01839bc4fb68
Comment 5 Andrew McCreight [:mccr8] 2012-02-02 06:49:52 PST
https://hg.mozilla.org/mozilla-central/rev/01839bc4fb68

Note You need to log in before you can comment on or make changes to this bug.