Closed
Bug 81132
Opened 23 years ago
Closed 23 years ago
Last cleanup of #define VIEW_SOURCE_HTML
Categories
(Core :: DOM: HTML Parser, defect)
Core
DOM: HTML Parser
Tracking
()
VERIFIED
FIXED
mozilla0.9.6
People
(Reporter: rbs, Assigned: bzbarsky)
Details
Attachments
(1 file)
(deleted),
patch
|
rbs
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
There are left-over of #ifdef VIEW_SOURCE_HTML in
mozilla/layout/build/nsLayoutDLF.cpp
Assignee | ||
Comment 2•23 years ago
|
||
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
Assignee | ||
Updated•23 years ago
|
Target Milestone: mozilla0.9.3 → mozilla0.9.4
Assignee | ||
Comment 3•23 years ago
|
||
vacation interfered. pushing out.
Target Milestone: mozilla0.9.4 → mozilla0.9.6
Assignee | ||
Comment 4•23 years ago
|
||
Assignee | ||
Comment 5•23 years ago
|
||
rbs, would you review?
Comment on attachment 51460 [details] [diff] [review]
kill that define off
r=rbs
(LayoutDLF is gone, right?)
Attachment #51460 -
Flags: review+
Assignee | ||
Comment 7•23 years ago
|
||
Yep. Makes it hard to patch... :)
Comment 8•23 years ago
|
||
Comment on attachment 51460 [details] [diff] [review]
kill that define off
sr=jst
Attachment #51460 -
Flags: superreview+
Assignee | ||
Comment 9•23 years ago
|
||
aaaand checked in. :)
Thanks for the prompt reviews, all!
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 10•23 years ago
|
||
rbs: could you please verify this.
Thanks,
Reporter | ||
Comment 11•23 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.
Description
•