Bug 142548 - Add support for more emoji with variation
Summary: Add support for more emoji with variation
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-03-10 15:37 PDT by Enrica Casucci
Modified: 2015-03-10 17:20 PDT (History)
0 users

See Also:


Attachments
Patch (6.73 KB, patch)
2015-03-10 15:41 PDT, Enrica Casucci
thorton: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Enrica Casucci 2015-03-10 15:37:35 PDT
ICU rules need to be adjusted.
Comment 1 Enrica Casucci 2015-03-10 15:41:13 PDT
Created attachment 248366 [details]
Patch
Comment 2 Tim Horton 2015-03-10 16:06:10 PDT
Comment on attachment 248366 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=248366&action=review

> LayoutTests/editing/selection/extend-by-character-007.html:11
> +<script src=../editing.js language="JavaScript" type="text/JavaScript" ></script>

quotes around the src value, no need for language or type attrs.

> LayoutTests/editing/selection/extend-by-character-007.html:16
> +    for (i = 0; i < 4; i++) {

var i
Comment 3 Enrica Casucci 2015-03-10 16:13:34 PDT
Committed revision 181351.
Comment 4 Myles C. Maxfield 2015-03-10 17:20:42 PDT
Comment on attachment 248366 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=248366&action=review

> LayoutTests/editing/selection/extend-by-character-007-expected.txt:16
> +            text run at (14,16) width 275: "\x{D83D}\x{DEA3}\x{D83C}\x{DFFB}\x{D83D}\x{DEA3}\x{D83C}\x{DFFC}\x{D83D}\x{DEA3}\x{D83C}\x{DFFD}\x{D83D}\x{DEA3}\x{D83C}\x{DFFE}\x{D83D}\x{DEA3}\x{D83C}\x{DFFF}\x{D83D}\x{DEA3}"

Does this actually test anything? It looks like this is the same string as in the .html