<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>160120</bug_id>
          
          <creation_ts>2016-07-23 09:41:21 -0700</creation_ts>
          <short_desc>[GTK] Unskip layout test editing/deleting/delete-emoji.html</short_desc>
          <delta_ts>2025-12-25 09:14:45 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKitGTK</component>
          <version>Other</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>178742</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Catanzaro">mcatanzaro</reporter>
          <assigned_to name="Claudio Saavedra">csaavedra</assigned_to>
          <cc>aperez</cc>
    
    <cc>bugs-noreply</cc>
    
    <cc>bugzilla</cc>
    
    <cc>csaavedra</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>mmaxfield</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1213920</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2016-07-23 09:41:21 -0700</bug_when>
    <thetext>I&apos;m skipping the layout test editing/deleting/delete-emoji.html because it doesn&apos;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&apos;t support these new emoji properly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1239163</commentid>
    <comment_count>1</comment_count>
    <who name="Bastien Nocera">bugzilla</who>
    <bug_when>2016-10-11 21:11:13 -0700</bug_when>
    <thetext>Duplicate of 156579?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1239321</commentid>
    <comment_count>2</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2016-10-12 09:33:08 -0700</bug_when>
    <thetext>I&apos;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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1363804</commentid>
    <comment_count>3</comment_count>
    <who name="Adrian Perez">aperez</who>
    <bug_when>2017-10-24 12:09:33 -0700</bug_when>
    <thetext>(In reply to Michael Catanzaro from comment #0)
&gt; I&apos;m skipping the layout test editing/deleting/delete-emoji.html because it
&gt; doesn&apos;t have expected results yet, and the actual results I see do not look
&gt; reasonable. On Mac there are a bunch of emoji there, but on GTK there are
&gt; only placeholder characters shown. Something in the GTK stack doesn&apos;t
&gt; 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&apos;ll add the expectations and unskip these two only, and check the rest of
the emoji deletion tests later on.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1364143</commentid>
    <comment_count>4</comment_count>
    <who name="Adrian Perez">aperez</who>
    <bug_when>2017-10-25 01:05:50 -0700</bug_when>
    <thetext>(In reply to Adrian Perez from comment #3)

&gt; [...]
&gt;  * editing/deleting/delete-emoji.html
&gt;  * editing/deleting/delete-emoji-1.html
&gt; 
&gt; I&apos;ll add the expectations and unskip these two only, and check the rest of
&gt; the emoji deletion tests later on.

Done in bug #178742</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1364944</commentid>
    <comment_count>5</comment_count>
    <who name="Myles C. Maxfield">mmaxfield</who>
    <bug_when>2017-10-26 11:43:09 -0700</bug_when>
    <thetext>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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2168146</commentid>
    <comment_count>6</comment_count>
    <who name="Claudio Saavedra">csaavedra</who>
    <bug_when>2025-12-25 04:27:04 -0800</bug_when>
    <thetext>All tests referencing this bug now pass, I am unskipping them and closing this one soon.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2168147</commentid>
    <comment_count>7</comment_count>
    <who name="Claudio Saavedra">csaavedra</who>
    <bug_when>2025-12-25 04:33:40 -0800</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/55863</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2168165</commentid>
    <comment_count>8</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2025-12-25 08:15:14 -0800</bug_when>
    <thetext>Test gardening commit 304935@main (89935439d6da): &lt;https://commits.webkit.org/304935@main&gt;

Reviewed commits have been landed. Closing PR #55863 and removing active labels.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>