Bug 141539

Summary: Additional emoji group support
Product: WebKit Reporter: Enrica Casucci <enrica>
Component: Layout and RenderingAssignee: Enrica Casucci <enrica>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, commit-queue, esprehn+autocc, glenn, kondapallykalyan, mitz, mmaxfield, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
buildbot: commit-queue-
Archive of layout-test-results from ews105 for mac-mavericks-wk2
none
Archive of layout-test-results from ews101 for mac-mavericks
none
Patch2 sam: review+

Description Enrica Casucci 2015-02-12 15:03:06 PST
Adding some new emoji groups.

rdar://problem/19727527
Comment 1 Enrica Casucci 2015-02-12 15:06:55 PST
Created attachment 246479 [details]
Patch
Comment 2 WebKit Commit Bot 2015-02-12 15:08:38 PST
Attachment 246479 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/text/TextBreakIterator.cpp:211:  One space before end of line comments  [whitespace/comments] [5]
ERROR: Source/WebCore/platform/text/TextBreakIterator.cpp:233:  One space before end of line comments  [whitespace/comments] [5]
ERROR: Source/WebCore/platform/text/TextBreakIterator.cpp:252:  One space before end of line comments  [whitespace/comments] [5]
Total errors found: 3 in 6 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Build Bot 2015-02-12 15:42:09 PST
Comment on attachment 246479 [details]
Patch

Attachment 246479 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5085886555357184

New failing tests:
editing/deleting/regional-indicators.html
editing/selection/regional-indicators.html
Comment 4 Build Bot 2015-02-12 15:42:14 PST
Created attachment 246482 [details]
Archive of layout-test-results from ews105 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 5 Enrica Casucci 2015-02-12 15:46:59 PST
Looking into the test failures.
Comment 6 Build Bot 2015-02-12 15:53:38 PST
Comment on attachment 246479 [details]
Patch

Attachment 246479 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/5381294405976064

New failing tests:
editing/deleting/regional-indicators.html
editing/selection/regional-indicators.html
Comment 7 Build Bot 2015-02-12 15:53:42 PST
Created attachment 246484 [details]
Archive of layout-test-results from ews101 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 8 Enrica Casucci 2015-02-12 16:06:11 PST
Created attachment 246485 [details]
Patch2

Found a bug in the ICU rules. This fixes the failing tests.
Comment 9 WebKit Commit Bot 2015-02-12 16:08:14 PST
Attachment 246485 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/text/TextBreakIterator.cpp:211:  One space before end of line comments  [whitespace/comments] [5]
ERROR: Source/WebCore/platform/text/TextBreakIterator.cpp:233:  One space before end of line comments  [whitespace/comments] [5]
ERROR: Source/WebCore/platform/text/TextBreakIterator.cpp:252:  One space before end of line comments  [whitespace/comments] [5]
Total errors found: 3 in 6 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 10 Enrica Casucci 2015-02-13 15:44:26 PST
Committed revision 180087.