Bug 160120
| Summary: | [GTK] Unskip layout test editing/deleting/delete-emoji.html | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Michael Catanzaro <mcatanzaro> |
| Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | aperez, bugs-noreply, bugzilla, mcatanzaro, mmaxfield |
| Priority: | P2 | ||
| Version: | Other | ||
| Hardware: | PC | ||
| OS: | Linux | ||
| Bug Depends on: | 178742 | ||
| Bug Blocks: | |||
Michael Catanzaro
I'm skipping the layout test editing/deleting/delete-emoji.html because it doesn't have expected results yet, and the actual results I see do not look reasonable. On Mac there are a bunch of emoji there, but on GTK there are only placeholder characters shown. Something in the GTK stack doesn't support these new emoji properly.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Bastien Nocera
Duplicate of 156579?
Michael Catanzaro
I'm not sure. I would assume bug #156579 would be just about adding color to the emoji. The emoji in this test are not drawn properly at all.
Adrian Perez
(In reply to Michael Catanzaro from comment #0)
> I'm skipping the layout test editing/deleting/delete-emoji.html because it
> doesn't have expected results yet, and the actual results I see do not look
> reasonable. On Mac there are a bunch of emoji there, but on GTK there are
> only placeholder characters shown. Something in the GTK stack doesn't
> support these new emoji properly.
Today I have been able to verify that the output for the GTK+ port matches
that of the Mac port for the following tests:
* editing/deleting/delete-emoji.html
* editing/deleting/delete-emoji-1.html
I'll add the expectations and unskip these two only, and check the rest of
the emoji deletion tests later on.
Adrian Perez
(In reply to Adrian Perez from comment #3)
> [...]
> * editing/deleting/delete-emoji.html
> * editing/deleting/delete-emoji-1.html
>
> I'll add the expectations and unskip these two only, and check the rest of
> the emoji deletion tests later on.
Done in bug #178742
Myles C. Maxfield
https://trac.webkit.org/changeset/223578/webkit This is the patch which caused this.
This patch removes custom character-iteration code inside RenderText in favor of platform-supplied character iterators. On GTK, this would take the form of ubrk_ ICU functions using the CHARACTER enum.
The best thing to do would be to verify that the deletion behavior matches some other native app on GTK, and if so, generate expected results.