Closed Bug 1481555 Opened 2 years ago Closed 2 years ago

Update lldbinit to summarise nsAtom

Categories

(Core :: XPCOM, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: mossop, Assigned: mossop)

References

Details

Attachments

(1 file)

The current summary string doesn't work since nsAtom changed.

https://searchfox.org/mozilla-central/source/.lldbinit#25
Tom, do you know if we can just change that line to call a method on the atom instead, or does calling methods imply that we have to move to a more complicated printer for nsAtom?
Blocks: 1447951, 1411469
Flags: needinfo?(ttromey)
Priority: -- → P3
Not quite sure this is right, but it seems to give the right results. Who could review this?
FWIW that patch looks reasonable to me.

I don't know the answer to Nathan's question.  However the patch seems to make it moot.
Flags: needinfo?(ttromey)
Comment on attachment 8998369 [details]
Bug 1481555: Use lldb's python API to summarise nsAtoms.

Nathan Froyd [:froydnj] has approved the revision.
Attachment #8998369 - Flags: review+
Pushed by dtownsend@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ff3a0784588f
Use lldb's python API to summarise nsAtoms. r=froydnj
https://hg.mozilla.org/mozilla-central/rev/ff3a0784588f
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Assignee: nobody → dtownsend
You need to log in before you can comment on or make changes to this bug.