fix display list printing to play nicely with fennec and b2g

RESOLVED FIXED in mozilla29

Status

()

Core
Layout: Misc Code
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: tnikkel, Assigned: tnikkel)

Tracking

29 Branch
mozilla29
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

4 years ago
Created attachment 8363566 [details] [diff] [review]
patch

The output for a single fprintf_stderr call ends up on its own line in the logcat output even if there is no newline in the string. So we have to create the the whole and then output it at once. Printing out indentation spaces makes the output somewhat hard to interpret.
Attachment #8363566 - Flags: review?(matspal)
Comment on attachment 8363566 [details] [diff] [review]
patch

While you're here, please add MOZ_OVERRIDE on the first WriteDebugInfo
in layout/base/nsDisplayList.h too.

Same comment as before regarding nsAutoCString.
Attachment #8363566 - Flags: review?(matspal) → review+
(Assignee)

Comment 2

4 years ago
Created attachment 8364868 [details] [diff] [review]
output the scroll and scrolled frames for scroll layer items

Putting this in here because it's small and not worth another bug.
Attachment #8364868 - Flags: review?(matspal)

Updated

4 years ago
Attachment #8364868 - Flags: review?(matspal) → review+
(Assignee)

Comment 3

4 years ago
(In reply to Mats Palmgren (:mats) from comment #1)
> Comment on attachment 8363566 [details] [diff] [review]
> patch
> 
> While you're here, please add MOZ_OVERRIDE on the first WriteDebugInfo
> in layout/base/nsDisplayList.h too.

I'm not sure what you mean here, all the WriteDebugInfo have MOZ_OVERRIDE that I can tell, except for the original declaration, which isn't an override.

> Same comment as before regarding nsAutoCString.

Done.
Sorry, I didn't realize the first one was actually the root.
https://hg.mozilla.org/mozilla-central/rev/b7f02d545988
https://hg.mozilla.org/mozilla-central/rev/bd61bfb42d02
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.