Bug 51199

Summary: Add tests for UTF-16 decoding and its variants
Product: WebKit Reporter: Yong Li <yong.li.webkit>
Component: TextAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, commit-queue
Priority: P3    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
the patch
ap: review+, yong.li.webkit: commit-queue-
added comments none

Description Yong Li 2010-12-16 10:59:28 PST
51035 is closed as invalid. But the tests are good to have.
Comment 1 Yong Li 2010-12-16 11:42:45 PST
Created attachment 76794 [details]
the patch
Comment 2 Alexey Proskuryakov 2010-12-16 11:54:55 PST
Comment on attachment 76794 [details]
the patch

Thank you!

Please consider adding an explanation of expected results as a comment in the test (something like "According to HTML5 and for IE compatibility, UTF-16 is treated as little endian. These subtest fail as of Firefox 3.6.13.")

To mention that the test fail in Firefox, you'll need to separate those that actually do (UTF-16LE  and unicodeFEFF should obviously pass everywhere).
Comment 3 Yong Li 2010-12-16 12:33:54 PST
Created attachment 76801 [details]
added comments
Comment 4 WebKit Commit Bot 2010-12-16 18:29:01 PST
Comment on attachment 76801 [details]
added comments

Clearing flags on attachment: 76801

Committed r74225: <http://trac.webkit.org/changeset/74225>
Comment 5 WebKit Commit Bot 2010-12-16 18:29:06 PST
All reviewed patches have been landed.  Closing bug.