If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

XMLParser omits the last char of a DOCTYPE node

VERIFIED FIXED in flash10.1

Status

Tamarin
Virtual Machine
P2
normal
VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: Steven Johnson, Assigned: Steven Johnson)

Tracking

unspecified
flash10.1
Bug Flags:
in-testsuite +
flashplayer-qrb +
flashplayer-triage +

Details

Attachments

(2 attachments)

(Assignee)

Description

8 years ago
This is trivial to fix, but annoyingly, a testcase can't be written in AS3/E4X, as these nodes are parsed-and-discarded and aren't accessible to E4X parsing. (We test that it is well-formed, but we return the wrong text value.) Flash/AIR, on the other hand, leverage the same code but look at these nodes, so we should probably add some way to test all of them.
(Assignee)

Comment 1

8 years ago
Created attachment 441858 [details] [diff] [review]
Patch
Assignee: nobody → stejohns
Attachment #441858 - Flags: review?(wsharp)

Updated

8 years ago
Attachment #441858 - Flags: review?(wsharp) → review+
(Assignee)

Comment 2

8 years ago
http://asteam.macromedia.com/hg/tamarin-redux-argo/rev/45c29866e814

Leaving open for now until we figure out the best way to implement a testcase. (Selftests perhaps?)

Updated

8 years ago
Flags: flashplayer-qrb+
Priority: -- → P2
Target Milestone: --- → flash10.1

Updated

8 years ago
Whiteboard: needs-testcase

Updated

8 years ago
Flags: in-testsuite?
(Assignee)

Updated

8 years ago
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED

Comment 3

8 years ago
Created attachment 445711 [details] [diff] [review]
selftest
Attachment #445711 - Flags: review?(stejohns)

Updated

8 years ago
Flags: flashplayer-triage+

Updated

8 years ago
QA Contact: vm → brbaker
(Assignee)

Updated

8 years ago
Attachment #445711 - Flags: review?(stejohns) → review+

Comment 4

8 years ago
Pushed to tr-argo -> 4037:cc22bf69f7cc
Status: RESOLVED → VERIFIED
Flags: in-testsuite? → in-testsuite+
Whiteboard: needs-testcase
You need to log in before you can comment on or make changes to this bug.