Fix rooting hazards in the profiler

RESOLVED FIXED in mozilla23

Status

()

Core
Gecko Profiler
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

Trunk
mozilla23
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Created attachment 746750 [details] [diff] [review]
Patch (v1)
Attachment #746750 - Flags: review?(tschneidereit)
Attachment #746750 - Flags: review?(bgirard)
(Assignee)

Comment 1

5 years ago
Created attachment 746751 [details] [diff] [review]
Patch (v1)

Oops, that wouldn't build!
Assignee: nobody → ehsan
Attachment #746750 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #746750 - Flags: review?(tschneidereit)
Attachment #746750 - Flags: review?(bgirard)
Attachment #746751 - Flags: review?(tschneidereit)
Attachment #746751 - Flags: review?(bgirard)
Comment on attachment 746751 [details] [diff] [review]
Patch (v1)

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

::: tools/profiler/JSObjectBuilder.cpp
@@ +6,5 @@
>  #include "jsapi.h"
>  #include "nsStringGlue.h"
>  #include "JSObjectBuilder.h"
>  
> +JSObjectBuilder::JSObjectBuilder(JSContext *aCx) : mCx(aCx), mObj(aCx), mOk(JS_TRUE)

Why not just remove mCx? The rest looks fine.
Attachment #746751 - Flags: review?(bgirard) → review+
Comment on attachment 746751 [details] [diff] [review]
Patch (v1)

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

What BenWa said.
Attachment #746751 - Flags: review?(tschneidereit) → review+
(Assignee)

Comment 4

5 years ago
(In reply to Benoit Girard (:BenWa) from comment #2)
> Comment on attachment 746751 [details] [diff] [review]
> Patch (v1)
> 
> Review of attachment 746751 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: tools/profiler/JSObjectBuilder.cpp
> @@ +6,5 @@
> >  #include "jsapi.h"
> >  #include "nsStringGlue.h"
> >  #include "JSObjectBuilder.h"
> >  
> > +JSObjectBuilder::JSObjectBuilder(JSContext *aCx) : mCx(aCx), mObj(aCx), mOk(JS_TRUE)
> 
> Why not just remove mCx? The rest looks fine.

Remove mCx?  Not sure what you mean, mCx is used all over the place.
(Assignee)

Updated

5 years ago
Flags: needinfo?(bgirard)
Opps mObj*. I couldn't see it used anywhere last night.
Flags: needinfo?(bgirard)
https://hg.mozilla.org/mozilla-central/rev/202d4048a8b8
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.