Closed Bug 977832 Opened 6 years ago Closed 6 years ago

Layout debugger needs to change how it invokes List() in response to bug 962443

Categories

(Core :: Layout, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: dholbert, Assigned: dholbert)

References

()

Details

Attachments

(1 file)

bug 962443 changed the function signature of List(), so that the second argument is a char* prefix instead of a numeric indentation-amount.

The layout debugger still invokes List() like so, though:
             root->List(out, 0);

That 0 gets treated as a null pointer, so frame dumps end up ending with
"(null)>"

We should probably just remove the second arg in that invocation (to get the default second arg, which is "").
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Summary: Layout debugger needs to change how it invokes List() for bug 962443 → Layout debugger needs to change how it invokes List() in response to bug 962443
Attachment #8383292 - Flags: review?(tnikkel) → review+
https://hg.mozilla.org/mozilla-central/rev/629bfbcb8b44
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.