Last cleanup of #define VIEW_SOURCE_HTML

VERIFIED FIXED in mozilla0.9.6

Status

()

Core
HTML: Parser
--
trivial
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: rbs, Assigned: bz)

Tracking

Trunk
mozilla0.9.6
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
There are left-over of #ifdef VIEW_SOURCE_HTML in
mozilla/layout/build/nsLayoutDLF.cpp
(Reporter)

Comment 1

17 years ago
And in: mozilla/content/build/nsContentDLF.cpp
accepting.  I'll deal with this once I get back into town in mid-June.  If
someone else wants to do this in the meantime, go for it.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.3
Target Milestone: mozilla0.9.3 → mozilla0.9.4
vacation interfered.  pushing out.
Target Milestone: mozilla0.9.4 → mozilla0.9.6

Updated

17 years ago
QA Contact: bsharma → moied
rbs, would you review?
Keywords: patch, review
(Reporter)

Comment 6

17 years ago
Comment on attachment 51460 [details] [diff] [review]
kill that define off

r=rbs
(LayoutDLF is gone, right?)
Attachment #51460 - Flags: review+
Yep.  Makes it hard to patch...  :)
Comment on attachment 51460 [details] [diff] [review]
kill that define off

sr=jst
Attachment #51460 - Flags: superreview+
aaaand checked in.  :)

Thanks for the prompt reviews, all!
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 10

17 years ago
rbs: could you please verify this.
Thanks,
(Reporter)

Comment 11

17 years ago
verified -- source level changes: '#ifdef VIEW_SOURCE_HTML' is gone for good.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.