Bug 141419 - [GTK] Layout Test editing/deleting/delete-emoji.html is failing
Summary: [GTK] Layout Test editing/deleting/delete-emoji.html is failing
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-02-10 02:07 PST by Marcos Chavarría Teijeiro (irc: chavaone)
Modified: 2015-03-30 05:41 PDT (History)
9 users (show)

See Also:


Attachments
Patch (3.14 KB, patch)
2015-03-10 02:54 PDT, Marcos Chavarría Teijeiro (irc: chavaone)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Marcos Chavarría Teijeiro (irc: chavaone) 2015-02-10 02:07:30 PST
The following layout test is failing on GTK+ since it was added on r179567 (http://trac.webkit.org/r179567)

The diff can be found here https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20%28Tests%29/r179867%20%286096%29/editing/deleting/delete-emoji-pretty-diff.html

The delete emoji operation doesn't seem to be working.
Comment 1 Marcos Chavarría Teijeiro (irc: chavaone) 2015-03-10 02:54:49 PDT
Created attachment 248322 [details]
Patch
Comment 2 Marcos Chavarría Teijeiro (irc: chavaone) 2015-03-10 03:04:25 PDT
The code for deleting special characters like emoji was specific for EFL and COCOA plarforms, there was a #if preprocessor directive like the following:

#if PLATFORM(COCOA) || PLATFORM(EFL)

I added PLATFORM(GTK) to the previous directive and now the emoji deletion is working on GTK+

I don't know if it has some side effects but I have run the Layout tests and they seem to work properly.
Comment 3 WebKit Commit Bot 2015-03-30 05:41:48 PDT
Comment on attachment 248322 [details]
Patch

Clearing flags on attachment: 248322

Committed r182137: <http://trac.webkit.org/changeset/182137>
Comment 4 WebKit Commit Bot 2015-03-30 05:41:53 PDT
All reviewed patches have been landed.  Closing bug.