nsCSSFrameConstructor's debug output shout be controllable via environment variable

VERIFIED FIXED in mozilla0.9.3

Status

()

Core
Layout
P3
minor
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: Chris Waterson, Assigned: Chris Waterson)

Tracking

Trunk
mozilla0.9.3
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

17 years ago
Right now, the debug output in nsCSSFrameConstructor is only accessable by
recompiling. Attaching a patch to fix that.
(Assignee)

Updated

17 years ago
Severity: normal → minor
Status: NEW → ASSIGNED
Keywords: donttest
Priority: -- → P3
Target Milestone: --- → mozilla0.9.3
(Assignee)

Comment 1

17 years ago
Created attachment 37285 [details] [diff] [review]
work-in-progress patch; still need to do some noisy frame tree output; e.g. in ContentInserted()
(Assignee)

Updated

17 years ago
Keywords: patch
r=dbaron, although you might want to move the |#include "prenv.h"| outside of
the |#ifdef DEBUG| to prevent future opt-only bustage (although the only other
thing prenv.h includes is prtypes.h, so the risk is very low...)
(Assignee)

Comment 3

17 years ago
Created attachment 40151 [details] [diff] [review]
address dbaron's comments
(Assignee)

Comment 4

17 years ago
marc, here's more of the same bug for the frame ctor. Could you take a look, and
if you like, sr=? thanks...

Comment 5

17 years ago
I Like - sr=attinasi
(Assignee)

Comment 6

17 years ago
Checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 7

17 years ago
Marking verified per last comments
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.