Closed Bug 1445983 Opened 8 years ago Closed 8 years ago

bdi should be an HTMLElement, not HTMLUnknownElement

Categories

(Core :: DOM: Core & HTML, enhancement)

Product:
Core
Component:
DOM: Core & HTML
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla61
Tracking Flags:
Tracking Status
firefox61 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

(Depends on 1 open bug)

Details

Attachments

(3 files, 2 obsolete files)

Update upstream html parser to include <bdi>
9.42 KB, patch
hsivonen
: review+
Details | Diff | Splinter Review
Additional changes needed to pass tests
6.17 KB, patch
hsivonen
: review+
Details | Diff | Splinter Review
Patch with those additional changes rolled in
42.62 KB, patch
Details | Diff | Splinter Review
This was missed in bug 613149.
Depends on: 1446082
Depends on: 1446097
Depends on: 1446104
MozReview-Commit-ID: 9eJ45uXRDfR
Attachment #8959286 - Flags: review?(hsivonen)
Attachment #8959286 - Flags: review?(hsivonen) → review+
Attachment #8959285 - Flags: review?(hsivonen) → review+
Attached patch Now with less header reordering (obsolete) — — Splinter Review
Attachment #8959286 - Attachment is obsolete: true
Attachment #8959446 - Flags: review?(hsivonen)
Attachment #8959305 - Attachment is obsolete: true
Attachment #8959446 - Flags: review?(hsivonen) → review+
Pushed by bzbarsky@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/db37fe69c545 <bdi> should be an HTMLElement, not HTMLUnknownElement. r=hsivonen
https://hg.mozilla.org/mozilla-central/rev/db37fe69c545
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: