Closed
Bug 431233
Opened 18 years ago
Closed 18 years ago
Log spam: "Header value = <foo>"
Categories
(Core :: DOM: Serializers, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: stuart.morgan+bugzilla, Assigned: stuart.morgan+bugzilla)
References
Details
(Keywords: regression)
Attachments
(1 file)
|
1.07 KB,
patch
|
jst
:
review+
jst
:
superreview+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
There's an unconditional printf at:
http://mxr.mozilla.org/mozilla/source/content/base/src/nsHTMLContentSerializer.cpp#760
that appears to have accidentally slipped in with the checkin of bug 380668.
|
Assignee | |
Comment 1•18 years ago
|
||
It would be nice to get this into 1.9, since it's a 0-risk regression fix (and log spam is annoying)
Assignee: nobody → stuart.morgan
Status: NEW → ASSIGNED
Attachment #318490 -
Flags: superreview?(jst)
Attachment #318490 -
Flags: review?(jst)
Attachment #318490 -
Flags: approval1.9?
|
|
Assignee | |
Updated•18 years ago
|
Keywords: regression
|
||
Comment 2•18 years ago
|
||
Comment on attachment 318490 [details] [diff] [review]
fix
re-request approval once reviews have been completed.
Attachment #318490 -
Flags: approval1.9? → approval1.9-
|
||
Updated•18 years ago
|
Attachment #318490 -
Flags: superreview?(jst)
Attachment #318490 -
Flags: superreview+
Attachment #318490 -
Flags: review?(jst)
Attachment #318490 -
Flags: review+
Attachment #318490 -
Flags: approval1.9?
Attachment #318490 -
Flags: approval1.9-
|
|
Assignee | |
Comment 3•18 years ago
|
||
(Sorry about that; I haven't been doing enough work in core to be familiar with the 1.9 approval process)
|
||
Comment 4•18 years ago
|
||
Comment on attachment 318490 [details] [diff] [review]
fix
a1.9=beltzner
Attachment #318490 -
Flags: approval1.9? → approval1.9+
|
|
Assignee | |
Comment 5•18 years ago
|
||
Checking in content/base/src/nsHTMLContentSerializer.cpp;
/cvsroot/mozilla/content/base/src/nsHTMLContentSerializer.cpp,v <-- nsHTMLContentSerializer.cpp
new revision: 1.118; previous revision: 1.117
done
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•