Closed
Bug 518110
Opened 15 years ago
Closed 15 years ago
nuke HTMLBaseFontElement interface
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: annevk, Assigned: sicking)
Details
Attachments
(1 file)
67.57 KB,
patch
|
jst
:
review+
|
Details | Diff | Splinter Review |
Opera recently removed rendering support for the <basefont> element to be in line with Gecko/WebKit, but the interface is still there because it is still in Gecko/WebKit. Would it be possible to clean this up or are we forever stuck with it? (I realize we're forever stuck with the quirk in the parser.)
Reporter | ||
Comment 1•15 years ago
|
||
Assignee | ||
Comment 2•15 years ago
|
||
This patch does a few things
* Remove the nsIDOMHTMLBaseFontElement interface and tests for it.
* Remove (or rather make non-scriptable in order to preserve binary compat)
nsIDOMHTMLHeadElement.profile. This is largely useless and I think it can
be removed from the HTML5 spec.
* Kill nsHTMLHeadElement and nsHTMLHtmlElement. Instead use nsHTMLSharedElement
for these element types.
* Remove apparently bogus comment about keeping classinfo ids stable
Assignee: nobody → jonas
Attachment #437807 -
Flags: review?(jst)
Comment 3•15 years ago
|
||
Comment on attachment 437807 [details] [diff] [review]
Patch to fix
r=jst!
Attachment #437807 -
Flags: review?(jst) → review+
Assignee | ||
Comment 4•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•