Closed Bug 81132 Opened 23 years ago Closed 23 years ago

Last cleanup of #define VIEW_SOURCE_HTML

Categories

(Core :: DOM: HTML Parser, defect)

defect
Not set
trivial

Tracking

()

VERIFIED FIXED
mozilla0.9.6

People

(Reporter: rbs, Assigned: bzbarsky)

Details

Attachments

(1 file)

There are left-over of #ifdef VIEW_SOURCE_HTML in mozilla/layout/build/nsLayoutDLF.cpp
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
QA Contact: bsharma → moied
Attached patch kill that define off (deleted) — Splinter Review
rbs, would you review?
Keywords: patch, review
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
Closed: 23 years ago
Resolution: --- → FIXED
rbs: could you please verify this. Thanks,
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.

Attachment

General

Created:
Updated:
Size: