Last Comment Bug 744611 - Use right compartment iterator when profiling scripts
: Use right compartment iterator when profiling scripts
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: unspecified
: x86 Mac OS X
-- normal (vote)
: mozilla14
Assigned To: Brian Hackett (:bhackett)
: Jason Orendorff [:jorendorff]
Depends on:
  Show dependency treegraph
Reported: 2012-04-11 15:13 PDT by Brian Hackett (:bhackett)
Modified: 2012-04-13 10:44 PDT (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (1.42 KB, patch)
2012-04-11 15:13 PDT, Brian Hackett (:bhackett)
wmccloskey: review+
Details | Diff | Splinter Review

Description User image Brian Hackett (:bhackett) 2012-04-11 15:13:59 PDT
Created attachment 614191 [details] [diff] [review]

Compartment iteration done during script profiling uses the wrong kind of iterator, which busts asserts in debug builds and causes profiles to be empty in release builds.
Comment 1 User image Brian Hackett (:bhackett) 2012-04-11 15:55:26 PDT
Comment 2 User image :Ms2ger (⌚ UTC+1/+2) 2012-04-12 10:25:58 PDT
Comment 3 User image Dan Mosedale (:dmose) 2012-04-13 10:44:20 PDT
Since jit-inspector is now working on today's nightly, I'm going to mark this as verified.  Presumably the AMO can now be told the add-on should work with Fx 14.  Thanks for the quick fix!

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