XMLParser omits the last char of a DOCTYPE node

VERIFIED FIXED in flash10.1

Status

P2
normal
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: stejohns, Assigned: stejohns)

Tracking

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

Details

Attachments

(2 attachments)

(Assignee)

Description

9 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

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

Updated

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

Comment 2

9 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

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

Updated

9 years ago
Whiteboard: needs-testcase

Updated

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

Updated

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

Comment 3

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

Updated

9 years ago
Flags: flashplayer-triage+

Updated

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

Updated

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

Comment 4

9 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.