Note: There are a few cases of duplicates in user autocompletion which are being worked on.

two minor improvements to cycle collector dumps




6 years ago
6 years ago


(Reporter: dbaron, Assigned: dbaron)



Firefox Tracking Flags

(Not tracked)



(3 attachments, 1 obsolete attachment)



6 years ago
During the debugging session described in I wrote two patches to fix things that we ran into during that session.  (I still need to test them...)

Comment 1

6 years ago
Created attachment 527730 [details] [diff] [review]
patch 1: get better JSTracer data

Now that JSTracer is binary-compatible between opt and debug, we should no longer need these ifdefs, and we can get better data in opt builds when WantDebugInfo is set.

Comment 2

6 years ago
Created attachment 527731 [details] [diff] [review]
patch 2: allow much longer urls

We needed the full URLs but couldn't get to them.

Comment 3

6 years ago
Created attachment 527732 [details] [diff] [review]
patch 3: make XPCOM_CC_DRAW_GRAPHS work at shutdown too

Here's a third one that's older and that I have tested.  I don't feel like filing a separate bug for it.  This makes XPCOM_CC_DRAW_GRAPHS apply to the shutdown collections too.
Attachment #527732 - Flags: review?(peterv)
Created attachment 527839 [details] [diff] [review]
peterv's CC shutdown logging patch

Peterv sent me this patch for CC shutdown logging.  Basically the same, but  factored a bit differently.  Just some food for thought.
Attachment #527732 - Flags: review?(peterv) → review+

Comment 5

6 years ago
Comment on attachment 527731 [details] [diff] [review]
patch 2: allow much longer urls

Well, I finally got around to testing these other 2 in opt builds.  This patch definitely works; I can't find any evidence of the other one doing so, though.
Attachment #527731 - Flags: review?(peterv)

Comment 6

6 years ago
Landed patch 3:
Attachment #527731 - Flags: review?(peterv) → review+

Comment 7

6 years ago
Landed patch 2:
Attachment #527731 - Flags: checkin+
Attachment #527732 - Flags: checkin+
Attachment #527839 - Attachment is obsolete: true
dbaron, you could probably close this.  2 of 3 patches landed, and the JSTracer stuff in part 1 is going to require additional futzing with the JS engine, so I'll deal with that in its own bug, bug 701415.

Comment 9

6 years ago
Sounds good.  (And yes, I didn't land patch 1 because it didn't actually do anything useful.
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.