nsMathMLForeignFrameWrapper class and related files are dead

RESOLVED FIXED in mozilla14

Status

()

Core
MathML
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Ehren Metcalfe, Assigned: Ehren Metcalfe)

Tracking

unspecified
mozilla14
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

11.14 KB, patch
Ehren Metcalfe
: review+
Details | Diff | Splinter Review
(Assignee)

Description

5 years ago
Here's an MXR search: http://mxr.mozilla.org/mozilla-central/search?string=nsMathMLForeignFrameWrapper
(Assignee)

Comment 1

5 years ago
Created attachment 609020 [details] [diff] [review]
patch

(tested on linux)
Attachment #609020 - Flags: review?(karlt)
Comment on attachment 609020 [details] [diff] [review]
patch

Thanks.
Can you remove the declaration for the removed function NewMathMLForeignFrameWrapper too, please?
http://hg.mozilla.org/mozilla-central/annotate/3e4735893504/layout/mathml/nsMathMLParts.h#l46
Attachment #609020 - Flags: review?(karlt) → review+
(Assignee)

Comment 3

5 years ago
Created attachment 609163 [details] [diff] [review]
patch for checkin

no prob.

(carrying over r+)
Attachment #609020 - Attachment is obsolete: true
Attachment #609163 - Flags: review+
(Assignee)

Updated

5 years ago
Attachment #609163 - Flags: checkin?
(Assignee)

Updated

5 years ago
Attachment #609163 - Flags: checkin?
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
(Assignee)

Comment 4

5 years ago
Comment on attachment 609163 [details] [diff] [review]
patch for checkin

(sry for the spam. checkin flag is new to me)
Attachment #609163 - Flags: checkin?
http://hg.mozilla.org/integration/mozilla-inbound/rev/3018a98a65b2
Keywords: checkin-needed
Target Milestone: --- → mozilla14
This patch has landed in mozilla-central and will be included in tomorrow's nightly build.  Thanks!

https://hg.mozilla.org/mozilla-central/rev/3018a98a65b2
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Attachment #609163 - Flags: checkin?
You need to log in before you can comment on or make changes to this bug.