Remove SubstValueForCode()

VERIFIED FIXED

Status

()

Core
Printing: Output
VERIFIED FIXED
13 years ago
13 years ago

People

(Reporter: smontagu, Assigned: smontagu)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

13 years ago
With the checkin for bug 272386, nsPageFrame::RemoveSubstValueForCode() has no
callers. It should die.
(Assignee)

Comment 1

13 years ago
Created attachment 182145 [details] [diff] [review]
Patch
Attachment #182145 - Flags: superreview?(roc)
Attachment #182145 - Flags: review?(roc)
Attachment #182145 - Flags: superreview?(roc)
Attachment #182145 - Flags: superreview+
Attachment #182145 - Flags: review?(roc)
Attachment #182145 - Flags: review+
(Assignee)

Comment 2

13 years ago
Comment on attachment 182145 [details] [diff] [review]
Patch

Please approve this patch (removal of unused code).
Attachment #182145 - Flags: approval1.8b2?

Comment 3

13 years ago
Comment on attachment 182145 [details] [diff] [review]
Patch

moving request out to b3. We're very nearly wrapped up on 1.8b2.
Attachment #182145 - Flags: approval1.8b3?
Attachment #182145 - Flags: approval1.8b2?
Attachment #182145 - Flags: approval1.8b2-
Comment on attachment 182145 [details] [diff] [review]
Patch

a=shaver
Attachment #182145 - Flags: approval1.8b3? → approval1.8b3+
Simon: this landed  Did you mean to mark this fixed?  Or are there other trees
to check-in to?
(Assignee)

Comment 6

13 years ago
Thanks for the reminder :)
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.