Fix JSDOMParser textContent setter

RESOLVED FIXED in Firefox 16

Status

()

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: bnicholson, Assigned: bnicholson)

Tracking

unspecified
Firefox 17
ARM
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox16 fixed)

Details

Attachments

(1 attachment)

The textContent setter is incorrectly called "textContext". Also, we should clear the parentNode for its children when we replace the childNodes.
Assignee

Comment 1

7 years ago
Attachment #653483 - Flags: review?(lucasr.at.mozilla)
Assignee

Updated

7 years ago
Blocks: 779796
Assignee

Comment 2

7 years ago
Comment on attachment 653483 [details] [diff] [review]
Fix JSDOMParser textContent setter

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 779796
User impact if declined: some parsed articles may show up incorrectly
Testing completed (on m-c, etc.): none yet
Risk to taking this patch (and alternatives if risky): low risk
String or UUID changes made by this patch: none
Attachment #653483 - Flags: approval-mozilla-aurora?
Attachment #653483 - Flags: review?(lucasr.at.mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/fcb1cb11935f
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 17
Attachment #653483 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.