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
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: All All
-- normal (vote)
: mozilla13
Assigned To: Andrew McCreight [:mccr8]
: Jason Orendorff [:jorendorff]
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:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

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

Description User image 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 User image Andrew McCreight [:mccr8] 2012-01-31 10:44:09 PST
Created attachment 593158 [details] [diff] [review]

Should be pretty simple.
Comment 2 User image 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 User image Bill McCloskey (:billm) 2012-02-01 13:23:33 PST
Comment on attachment 593158 [details] [diff] [review]

No, this is good.
Comment 4 User image Andrew McCreight [:mccr8] 2012-02-01 17:58:34 PST
Comment 5 User image Andrew McCreight [:mccr8] 2012-02-02 06:49:52 PST

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