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)
|
1.20 KB,
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
•