<?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>191700</bug_id>
          
          <creation_ts>2018-11-15 12:15:03 -0800</creation_ts>
          <short_desc>Regression: Layer doesn&apos;t move when transform updated after a transition</short_desc>
          <delta_ts>2019-12-17 13:32:22 -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>Animations</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>CONFIGURATION CHANGED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ali Juma">ajuma</reporter>
          <assigned_to name="Antoine Quint">graouts</assigned_to>
          <cc>dino</cc>
    
    <cc>fred.wang</cc>
    
    <cc>graouts</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1479020</commentid>
    <comment_count>0</comment_count>
      <attachid>354968</attachid>
    <who name="Ali Juma">ajuma</who>
    <bug_when>2018-11-15 12:15:03 -0800</bug_when>
    <thetext>Created attachment 354968
Test case

See attached test case: the blue box is animated using a CSS transition, and then its transform is updated again after the transition finishes. But the layer doesn&apos;t move for the latter update, until an update is forced by resizing the window and/or minimizing and restoring it.

This works in STP 69 but is broken on trunk.

Trying to debug this a bit, we do get as far as setting the new transform on CALayer, but the CALayer still has an animation on it (even though the transition is finished) and ignores the new transform. If that animation is removed before setting the transform, the layer does move to the right place.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1479037</commentid>
    <comment_count>1</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2018-11-15 12:36:06 -0800</bug_when>
    <thetext>Almost certainly a regression from my compositing changes.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1479252</commentid>
    <comment_count>2</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2018-11-15 21:08:46 -0800</bug_when>
    <thetext>I can&apos;t reproduce. Are you sure this wasn&apos;t fixed by r238170?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1479284</commentid>
    <comment_count>3</comment_count>
    <who name="Frédéric Wang Nélar">fred.wang</who>
    <bug_when>2018-11-15 23:15:02 -0800</bug_when>
    <thetext>I can only reproduce this issue on the iOS simulator, not macOS safari/minibrowser. I&apos;m at r238191</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1479407</commentid>
    <comment_count>4</comment_count>
    <who name="Ali Juma">ajuma</who>
    <bug_when>2018-11-16 07:34:00 -0800</bug_when>
    <thetext>I tried again at r238277, and this still reproduces for me on macOS MiniBrowser and Safari.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1479447</commentid>
    <comment_count>5</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2018-11-16 09:44:48 -0800</bug_when>
    <thetext>I can reproduce in the simulator.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1479469</commentid>
    <comment_count>6</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2018-11-16 10:06:14 -0800</bug_when>
    <thetext>It seems like we&apos;re failing to remove the transforms from the GraphicsLayer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1479471</commentid>
    <comment_count>7</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2018-11-16 10:10:55 -0800</bug_when>
    <thetext>I think this is an animations bug. We don&apos;t hit KeyframeEffect::applyPendingAcceleratedActions() when the transition ends.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1479478</commentid>
    <comment_count>8</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2018-11-16 10:20:38 -0800</bug_when>
    <thetext>https://trac.webkit.org/changeset/238128/webkit may be related.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1479480</commentid>
    <comment_count>9</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2018-11-16 10:21:38 -0800</bug_when>
    <thetext>&lt;rdar://problem/46130742&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1599543</commentid>
    <comment_count>10</comment_count>
    <who name="Antoine Quint">graouts</who>
    <bug_when>2019-12-17 13:32:22 -0800</bug_when>
    <thetext>Does not reproduce with ToT for me.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>354968</attachid>
            <date>2018-11-15 12:15:03 -0800</date>
            <delta_ts>2018-11-15 12:15:03 -0800</delta_ts>
            <desc>Test case</desc>
            <filename>animation-stops.html</filename>
            <type>text/html</type>
            <size>723</size>
            <attacher name="Ali Juma">ajuma</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIGh0bWw+CjxoZWFkPgo8c3R5bGU+CiNib3ggewogIGRpc3BsYXk6IGJsb2NrOwog
IHdpZHRoOiAxMDBweDsKICBoZWlnaHQ6IDEwMHB4OwogIGJhY2tncm91bmQtY29sb3I6IGJsdWU7
CiAgdHJhbnNmb3JtOiB0cmFuc2xhdGUzZCgxNTBweCwgMCwgMCk7Cn0KPC9zdHlsZT4KPC9oZWFk
PgoKPGJvZHk+CiAgPGRpdiBpZD0iYm94Ij48L2Rpdj4KICBUaGUgYm94IHNob3VsZCBmaXJzdCBt
b3ZlIGxlZnQsIGFuZCB0aGVuIG1vdmUgcmlnaHQuCjwvYm9keT4KICAKPHNjcmlwdCA+CmNvbnN0
IGJveCA9IGRvY3VtZW50LnF1ZXJ5U2VsZWN0b3IoJyNib3gnKTsKCmZ1bmN0aW9uIHNsaWRlKCkg
ewogIGJveC5zdHlsZS50cmFuc2Zvcm0gPSBgdHJhbnNsYXRlM2QoMzAwcHgsIDAsIDApYDsKICBi
b3guc3R5bGUuYmFja2dyb3VuZENvbG9yID0gJ2JsdWUnOwogIGJveC5zdHlsZS50cmFuc2l0aW9u
ID0gJ2luaXRpYWwnOwp9CgpmdW5jdGlvbiBhbmltYXRlKCkgewogICAgYm94LnN0eWxlLnRyYW5z
Zm9ybSA9IGB0cmFuc2xhdGUzZCgwcHgsIDAsIDApYDsKICAgIGJveC5zdHlsZS5iYWNrZ3JvdW5k
Q29sb3IgPSAncmVkJzsKICAgIGJveC5zdHlsZS50cmFuc2l0aW9uID0gJ3RyYW5zZm9ybSAwLjJz
JzsKICAgIAogICAgd2luZG93LnNldFRpbWVvdXQoKCkgPT4gc2xpZGUoKSwgMzAwKTsKfQoKcmVx
dWVzdEFuaW1hdGlvbkZyYW1lKGFuaW1hdGUpOwo8L3NjcmlwdD4K
</data>

          </attachment>
      

    </bug>

</bugzilla>