<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[208263] branches/safari-602-branch</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/208263">208263</a></dd>
<dt>Author</dt> <dd>matthew_hanson@apple.com</dd>
<dt>Date</dt> <dd>2016-11-01 17:25:57 -0700 (Tue, 01 Nov 2016)</dd>
</dl>
<h3>Log Message</h3>
<pre>Roll out <a href="http://trac.webkit.org/projects/webkit/changeset/208025">r208025</a> via <a href="http://trac.webkit.org/projects/webkit/changeset/208167">r208167</a>. rdar://problem/28216240</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#branchessafari602branchLayoutTestsChangeLog">branches/safari-602-branch/LayoutTests/ChangeLog</a></li>
<li><a href="#branchessafari602branchSourceWebCoreChangeLog">branches/safari-602-branch/Source/WebCore/ChangeLog</a></li>
<li><a href="#branchessafari602branchSourceWebCorepageanimationAnimationControllercpp">branches/safari-602-branch/Source/WebCore/page/animation/AnimationController.cpp</a></li>
<li><a href="#branchessafari602branchSourceWebCorepageanimationCSSPropertyAnimationcpp">branches/safari-602-branch/Source/WebCore/page/animation/CSSPropertyAnimation.cpp</a></li>
<li><a href="#branchessafari602branchSourceWebCorepageanimationCompositeAnimationcpp">branches/safari-602-branch/Source/WebCore/page/animation/CompositeAnimation.cpp</a></li>
<li><a href="#branchessafari602branchSourceWebCorepageanimationKeyframeAnimationcpp">branches/safari-602-branch/Source/WebCore/page/animation/KeyframeAnimation.cpp</a></li>
<li><a href="#branchessafari602branchSourceWebCorepageanimationKeyframeAnimationh">branches/safari-602-branch/Source/WebCore/page/animation/KeyframeAnimation.h</a></li>
<li><a href="#branchessafari602branchSourceWebCorerenderingRenderLayercpp">branches/safari-602-branch/Source/WebCore/rendering/RenderLayer.cpp</a></li>
<li><a href="#branchessafari602branchSourceWebCorerenderingstyleWillChangeDatacpp">branches/safari-602-branch/Source/WebCore/rendering/style/WillChangeData.cpp</a></li>
<li><a href="#branchessafari602branchSourceWebCorerenderingstyleWillChangeDatah">branches/safari-602-branch/Source/WebCore/rendering/style/WillChangeData.h</a></li>
</ul>
<h3>Removed Paths</h3>
<ul>
<li><a href="#branchessafari602branchLayoutTestsanimationsstackingcontextfillforwardsexpectedhtml">branches/safari-602-branch/LayoutTests/animations/stacking-context-fill-forwards-expected.html</a></li>
<li><a href="#branchessafari602branchLayoutTestsanimationsstackingcontextfillforwardshtml">branches/safari-602-branch/LayoutTests/animations/stacking-context-fill-forwards.html</a></li>
<li><a href="#branchessafari602branchLayoutTestsanimationsstackingcontextnotfillforwardsexpectedhtml">branches/safari-602-branch/LayoutTests/animations/stacking-context-not-fill-forwards-expected.html</a></li>
<li><a href="#branchessafari602branchLayoutTestsanimationsstackingcontextnotfillforwardshtml">branches/safari-602-branch/LayoutTests/animations/stacking-context-not-fill-forwards.html</a></li>
<li><a href="#branchessafari602branchLayoutTestsanimationsstackingcontextunchangedwhilerunningexpectedhtml">branches/safari-602-branch/LayoutTests/animations/stacking-context-unchanged-while-running-expected.html</a></li>
<li><a href="#branchessafari602branchLayoutTestsanimationsstackingcontextunchangedwhilerunninghtml">branches/safari-602-branch/LayoutTests/animations/stacking-context-unchanged-while-running.html</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="branchessafari602branchLayoutTestsChangeLog"></a>
<div class="modfile"><h4>Modified: branches/safari-602-branch/LayoutTests/ChangeLog (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/LayoutTests/ChangeLog        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/LayoutTests/ChangeLog        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -1,3 +1,7 @@
</span><ins>+2016-11-01 Matthew Hanson <matthew_hanson@apple.com>
+
+ Rollout r208167. rdar://problem/28216240
+
</ins><span class="cx"> 2016-10-31 Matthew Hanson <matthew_hanson@apple.com>
</span><span class="cx">
</span><span class="cx"> Merge r208025. rdar://problem/28216240
</span></span></pre></div>
<a id="branchessafari602branchLayoutTestsanimationsstackingcontextfillforwardsexpectedhtml"></a>
<div class="delfile"><h4>Deleted: branches/safari-602-branch/LayoutTests/animations/stacking-context-fill-forwards-expected.html (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/LayoutTests/animations/stacking-context-fill-forwards-expected.html        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/LayoutTests/animations/stacking-context-fill-forwards-expected.html        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -1,42 +0,0 @@
</span><del>-<!DOCTYPE html>
-
-<html>
-<head>
- <style>
- .box {
- position: absolute;
- top: 0;
- left: 0;
- height: 100px;
- width: 100px;
- }
-
- #animating {
- background-color: orange;
- z-index: 0;
- }
-
- .behind {
- background-color: blue;
- top: 50px;
- left: 50px;
- z-index: 1;
- }
-
- .front {
- top: 25px;
- left: 25px;
- background-color: green;
- z-index: 2;
- }
- </style>
-</head>
-<body>
-
-<div class="behind box"></div>
-<div id="animating" class="box">
- <div class="front box"></div>
-</div>
-
-</body>
-</html>
</del></span></pre></div>
<a id="branchessafari602branchLayoutTestsanimationsstackingcontextfillforwardshtml"></a>
<div class="delfile"><h4>Deleted: branches/safari-602-branch/LayoutTests/animations/stacking-context-fill-forwards.html (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/LayoutTests/animations/stacking-context-fill-forwards.html        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/LayoutTests/animations/stacking-context-fill-forwards.html        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -1,62 +0,0 @@
</span><del>-<!DOCTYPE html>
-
-<html>
-<head>
- <title>Tests that a fill-forwards animation causes stacking context when finished.</title>
- <style>
- .box {
- position: absolute;
- top: 0;
- left: 0;
- height: 100px;
- width: 100px;
- }
-
- #animating {
- animation: move 0.1s 1 forwards;
- background-color: orange;
- }
-
- @keyframes move {
- from { transform: translateX(400px); }
- to { transform: none }
- }
-
- .behind {
- background-color: blue;
- top: 50px;
- left: 50px;
- z-index: 1;
- }
-
- .front {
- top: 25px;
- left: 25px;
- background-color: green;
- z-index: 2;
- }
- </style>
- <script>
- if (window.testRunner)
- testRunner.waitUntilDone();
-
- window.addEventListener('load', function() {
- document.getElementById('animating').addEventListener('animationend', function() {
- // Wait until filling forwards.
- window.setTimeout(function() {
- if (window.testRunner)
- testRunner.notifyDone();
- }, 0);
- })
- }, false);
- </script>
-</head>
-<body>
-
-<div class="behind box"></div>
-<div id="animating" class="box">
- <div class="front box"></div>
-</div>
-
-</body>
-</html>
</del></span></pre></div>
<a id="branchessafari602branchLayoutTestsanimationsstackingcontextnotfillforwardsexpectedhtml"></a>
<div class="delfile"><h4>Deleted: branches/safari-602-branch/LayoutTests/animations/stacking-context-not-fill-forwards-expected.html (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/LayoutTests/animations/stacking-context-not-fill-forwards-expected.html        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/LayoutTests/animations/stacking-context-not-fill-forwards-expected.html        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -1,41 +0,0 @@
</span><del>-<!DOCTYPE html>
-
-<html>
-<head>
- <style>
- .box {
- position: absolute;
- top: 0;
- left: 0;
- height: 100px;
- width: 100px;
- }
-
- #animating {
- background-color: orange;
- }
-
- .behind {
- background-color: blue;
- top: 50px;
- left: 50px;
- z-index: 1;
- }
-
- .front {
- top: 25px;
- left: 25px;
- background-color: green;
- z-index: 2;
- }
- </style>
-</head>
-<body>
-
-<div class="behind box"></div>
-<div id="animating" class="box">
- <div class="front box"></div>
-</div>
-
-</body>
-</html>
</del></span></pre></div>
<a id="branchessafari602branchLayoutTestsanimationsstackingcontextnotfillforwardshtml"></a>
<div class="delfile"><h4>Deleted: branches/safari-602-branch/LayoutTests/animations/stacking-context-not-fill-forwards.html (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/LayoutTests/animations/stacking-context-not-fill-forwards.html        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/LayoutTests/animations/stacking-context-not-fill-forwards.html        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -1,62 +0,0 @@
</span><del>-<!DOCTYPE html>
-
-<html>
-<head>
- <title>Tests that a non-fill-forwards animation doesn't cause stacking context once finished.</title>
- <style>
- .box {
- position: absolute;
- top: 0;
- left: 0;
- height: 100px;
- width: 100px;
- }
-
- #animating {
- animation: move 0.1s 1;
- background-color: orange;
- }
-
- @keyframes move {
- from { transform: translateX(400px); }
- to { transform: none }
- }
-
- .behind {
- background-color: blue;
- top: 50px;
- left: 50px;
- z-index: 1;
- }
-
- .front {
- top: 25px;
- left: 25px;
- background-color: green;
- z-index: 2;
- }
- </style>
- <script>
- if (window.testRunner)
- testRunner.waitUntilDone();
-
- window.addEventListener('load', function() {
- document.getElementById('animating').addEventListener('animationend', function() {
- // Wait until filling forwards.
- window.setTimeout(function() {
- if (window.testRunner)
- testRunner.notifyDone();
- }, 0);
- })
- }, false);
- </script>
-</head>
-<body>
-
-<div class="behind box"></div>
-<div id="animating" class="box">
- <div class="front box"></div>
-</div>
-
-</body>
-</html>
</del></span></pre></div>
<a id="branchessafari602branchLayoutTestsanimationsstackingcontextunchangedwhilerunningexpectedhtml"></a>
<div class="delfile"><h4>Deleted: branches/safari-602-branch/LayoutTests/animations/stacking-context-unchanged-while-running-expected.html (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/LayoutTests/animations/stacking-context-unchanged-while-running-expected.html        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/LayoutTests/animations/stacking-context-unchanged-while-running-expected.html        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -1,41 +0,0 @@
</span><del>-<!DOCTYPE html>
-
-<html>
-<head>
- <style>
- .box {
- position: absolute;
- top: 0;
- left: 0;
- height: 100px;
- width: 100px;
- }
-
- #animating {
- background-color: orange;
- z-index: 0;
- }
-
- .behind {
- background-color: blue;
- top: 25px;
- left: 50px;
- z-index: 1;
- }
-
- .front {
- top: 50px;
- left: 25px;
- background-color: green;
- z-index: 2;
- }
- </style>
-</head>
-<body>
-
-<div class="behind box"></div>
-<div id="animating" class="box">
- <div class="front box"></div>
-</div>
-</body>
-</html>
</del></span></pre></div>
<a id="branchessafari602branchLayoutTestsanimationsstackingcontextunchangedwhilerunninghtml"></a>
<div class="delfile"><h4>Deleted: branches/safari-602-branch/LayoutTests/animations/stacking-context-unchanged-while-running.html (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/LayoutTests/animations/stacking-context-unchanged-while-running.html        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/LayoutTests/animations/stacking-context-unchanged-while-running.html        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -1,65 +0,0 @@
</span><del>-<!DOCTYPE html>
-
-<html>
-<head>
- <title>Tests that an element with an animation whose keyframes affect stacking context creates stacking context throughout the animation.</title>
- <style>
- .box {
- position: absolute;
- top: 0;
- left: 0;
- height: 100px;
- width: 100px;
- }
-
- #animating {
- animation: move 2s 1;
- background-color: orange;
- }
-
- @keyframes move {
- from { z-index: 1; }
- 10% { z-index: auto; }
- 90% { z-index: auto; }
- to { z-index: 2; }
- }
-
- .behind {
- background-color: blue;
- top: 25px;
- left: 50px;
- z-index: 1;
- }
-
- .front {
- top: 50px;
- left: 25px;
- background-color: green;
- z-index: 2;
- }
-
- #result {
- opacity: 0;
- }
- </style>
- <script src="resources/animation-test-helpers.js" type="text/javascript"></script>
- <script type="text/javascript">
- const expectedValues = [
- // [animation-name, time, element-id, property, expected-value, tolerance]
- ["move", 0.5, "animating", "z-index", 0, 0],
- ];
-
- var doPixelTest = true;
- var disablePauseAPI = false;
- runAnimationTest(expectedValues, null, undefined, disablePauseAPI, doPixelTest);
- </script>
-</head>
-<body>
-
-<div class="behind box"></div>
-<div id="animating" class="box">
- <div class="front box"></div>
-</div>
-<div id="result"></div>
-</body>
-</html>
</del></span></pre></div>
<a id="branchessafari602branchSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: branches/safari-602-branch/Source/WebCore/ChangeLog (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/Source/WebCore/ChangeLog        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/Source/WebCore/ChangeLog        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -1,5 +1,9 @@
</span><span class="cx"> 2016-11-01 Matthew Hanson <matthew_hanson@apple.com>
</span><span class="cx">
</span><ins>+ Rollout r208167. rdar://problem/28216240
+
+2016-11-01 Matthew Hanson <matthew_hanson@apple.com>
+
</ins><span class="cx"> Rollout r208255. rdar://problem/28962886
</span><span class="cx">
</span><span class="cx"> 2016-11-01 Matthew Hanson <matthew_hanson@apple.com>
</span></span></pre></div>
<a id="branchessafari602branchSourceWebCorepageanimationAnimationControllercpp"></a>
<div class="modfile"><h4>Modified: branches/safari-602-branch/Source/WebCore/page/animation/AnimationController.cpp (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/Source/WebCore/page/animation/AnimationController.cpp        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/Source/WebCore/page/animation/AnimationController.cpp        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -624,6 +624,13 @@
</span><span class="cx"> #endif
</span><span class="cx"> }
</span><span class="cx">
</span><ins>+ if (animatedStyle) {
+ // If the animations/transitions change opacity or transform, we need to update
+ // the style to impose the stacking rules. Note that this is also
+ // done in StyleResolver::adjustRenderStyle().
+ if (animatedStyle->hasAutoZIndex() && (animatedStyle->opacity() < 1.0f || animatedStyle->hasTransform()))
+ animatedStyle->setZIndex(0);
+ }
</ins><span class="cx"> return animationStateChanged;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="branchessafari602branchSourceWebCorepageanimationCSSPropertyAnimationcpp"></a>
<div class="modfile"><h4>Modified: branches/safari-602-branch/Source/WebCore/page/animation/CSSPropertyAnimation.cpp (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/Source/WebCore/page/animation/CSSPropertyAnimation.cpp        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/Source/WebCore/page/animation/CSSPropertyAnimation.cpp        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -599,7 +599,10 @@
</span><span class="cx">
</span><span class="cx"> void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
</span><span class="cx"> {
</span><del>- dst->setOpacity(blendFunc(anim, a->opacity(), b->opacity(), progress));
</del><ins>+ float fromOpacity = a->opacity();
+
+ // This makes sure we put the object being animated into a RenderLayer during the animation
+ dst->setOpacity(blendFunc(anim, (fromOpacity == 1) ? 0.999999f : fromOpacity, b->opacity(), progress));
</ins><span class="cx"> }
</span><span class="cx"> };
</span><span class="cx">
</span></span></pre></div>
<a id="branchessafari602branchSourceWebCorepageanimationCompositeAnimationcpp"></a>
<div class="modfile"><h4>Modified: branches/safari-602-branch/Source/WebCore/page/animation/CompositeAnimation.cpp (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/Source/WebCore/page/animation/CompositeAnimation.cpp        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/Source/WebCore/page/animation/CompositeAnimation.cpp        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -292,34 +292,14 @@
</span><span class="cx"> m_keyframeAnimations.checkConsistency();
</span><span class="cx">
</span><span class="cx"> bool animationStateChanged = false;
</span><del>- bool forceStackingContext = false;
</del><span class="cx">
</span><span class="cx"> if (currentStyle) {
</span><span class="cx"> // Now that we have transition objects ready, let them know about the new goal state. We want them
</span><span class="cx"> // to fill in a RenderStyle*& only if needed.
</span><del>- bool checkForStackingContext = false;
</del><span class="cx"> for (auto& transition : m_transitions.values()) {
</span><span class="cx"> if (transition->animate(this, &renderer, currentStyle, &targetStyle, blendedStyle))
</span><span class="cx"> animationStateChanged = true;
</span><del>-
- checkForStackingContext |= WillChangeData::propertyCreatesStackingContext(transition->animatingProperty());
</del><span class="cx"> }
</span><del>-
- if (blendedStyle && checkForStackingContext) {
- // Note that this is similar to code in StyleResolver::adjustRenderStyle() but only needs to consult
- // animatable properties that can trigger stacking context.
- if (blendedStyle->opacity() < 1.0f
- || blendedStyle->hasTransformRelatedProperty()
- || blendedStyle->hasMask()
- || blendedStyle->clipPath()
- || blendedStyle->boxReflect()
- || blendedStyle->hasFilter()
-#if ENABLE(FILTERS_LEVEL_2)
- || blendedStyle->hasBackdropFilter()
-#endif
- )
- forceStackingContext = true;
- }
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> // Now that we have animation objects ready, let them know about the new goal state. We want them
</span><span class="lines">@@ -326,24 +306,10 @@
</span><span class="cx"> // to fill in a RenderStyle*& only if needed.
</span><span class="cx"> for (auto& name : m_keyframeAnimationOrderMap) {
</span><span class="cx"> RefPtr<KeyframeAnimation> keyframeAnim = m_keyframeAnimations.get(name);
</span><del>- if (keyframeAnim) {
- if (keyframeAnim->animate(this, &renderer, currentStyle, &targetStyle, blendedStyle))
- animationStateChanged = true;
-
- bool runningOrFillingForwards = !keyframeAnim->waitingToStart() && !keyframeAnim->postActive();
- forceStackingContext |= runningOrFillingForwards && keyframeAnim->triggersStackingContext();
- }
</del><ins>+ if (keyframeAnim && keyframeAnim->animate(this, &renderer, currentStyle, &targetStyle, blendedStyle))
+ animationStateChanged = true;
</ins><span class="cx"> }
</span><span class="cx">
</span><del>- // https://drafts.csswg.org/css-animations-1/
- // While an animation is applied but has not finished, or has finished but has an animation-fill-mode of forwards or both,
- // the user agent must act as if the will-change property ([css-will-change-1]) on the element additionally
- // includes all the properties animated by the animation.
- if (forceStackingContext && blendedStyle) {
- if (blendedStyle->hasAutoZIndex())
- blendedStyle->setZIndex(0);
- }
-
</del><span class="cx"> return animationStateChanged;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="branchessafari602branchSourceWebCorepageanimationKeyframeAnimationcpp"></a>
<div class="modfile"><h4>Modified: branches/safari-602-branch/Source/WebCore/page/animation/KeyframeAnimation.cpp (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/Source/WebCore/page/animation/KeyframeAnimation.cpp        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/Source/WebCore/page/animation/KeyframeAnimation.cpp        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -38,7 +38,6 @@
</span><span class="cx"> #include "RenderBox.h"
</span><span class="cx"> #include "RenderStyle.h"
</span><span class="cx"> #include "StyleResolver.h"
</span><del>-#include "WillChangeData.h"
</del><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><span class="lines">@@ -57,7 +56,6 @@
</span><span class="cx"> #if ENABLE(FILTERS_LEVEL_2)
</span><span class="cx"> checkForMatchingBackdropFilterFunctionLists();
</span><span class="cx"> #endif
</span><del>- computeStackingContextImpact();
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> KeyframeAnimation::~KeyframeAnimation()
</span><span class="lines">@@ -67,16 +65,6 @@
</span><span class="cx"> endAnimation();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void KeyframeAnimation::computeStackingContextImpact()
-{
- for (auto propertyID : m_keyframes.properties()) {
- if (WillChangeData::propertyCreatesStackingContext(propertyID)) {
- m_triggersStackingContext = true;
- break;
- }
- }
-}
-
</del><span class="cx"> void KeyframeAnimation::fetchIntervalEndpointsForProperty(CSSPropertyID property, const RenderStyle*& fromStyle, const RenderStyle*& toStyle, double& prog) const
</span><span class="cx"> {
</span><span class="cx"> size_t numKeyframes = m_keyframes.size();
</span><span class="lines">@@ -160,7 +148,6 @@
</span><span class="cx"> return false;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- // FIXME: the code below never changes the state, so this function always returns false.
</del><span class="cx"> AnimationState oldState = state();
</span><span class="cx">
</span><span class="cx"> // Run a cycle of animation.
</span></span></pre></div>
<a id="branchessafari602branchSourceWebCorepageanimationKeyframeAnimationh"></a>
<div class="modfile"><h4>Modified: branches/safari-602-branch/Source/WebCore/page/animation/KeyframeAnimation.h (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/Source/WebCore/page/animation/KeyframeAnimation.h        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/Source/WebCore/page/animation/KeyframeAnimation.h        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -55,8 +55,6 @@
</span><span class="cx"> const AtomicString& name() const { return m_keyframes.animationName(); }
</span><span class="cx">
</span><span class="cx"> bool hasAnimationForProperty(CSSPropertyID) const;
</span><del>-
- bool triggersStackingContext() const { return m_triggersStackingContext; }
</del><span class="cx">
</span><span class="cx"> void setUnanimatedStyle(std::unique_ptr<RenderStyle> style) { m_unanimatedStyle = WTFMove(style); }
</span><span class="cx"> RenderStyle* unanimatedStyle() const { return m_unanimatedStyle.get(); }
</span><span class="lines">@@ -83,7 +81,6 @@
</span><span class="cx">
</span><span class="cx"> bool computeExtentOfAnimationForMatchingTransformLists(const FloatRect& rendererBox, LayoutRect&) const;
</span><span class="cx">
</span><del>- void computeStackingContextImpact();
</del><span class="cx"> void validateTransformFunctionList();
</span><span class="cx"> void checkForMatchingFilterFunctionLists();
</span><span class="cx"> #if ENABLE(FILTERS_LEVEL_2)
</span><span class="lines">@@ -101,7 +98,6 @@
</span><span class="cx"> std::unique_ptr<RenderStyle> m_unanimatedStyle; // The style just before we started animation
</span><span class="cx">
</span><span class="cx"> bool m_startEventDispatched { false };
</span><del>- bool m_triggersStackingContext { false };
</del><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> } // namespace WebCore
</span></span></pre></div>
<a id="branchessafari602branchSourceWebCorerenderingRenderLayercpp"></a>
<div class="modfile"><h4>Modified: branches/safari-602-branch/Source/WebCore/rendering/RenderLayer.cpp (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/Source/WebCore/rendering/RenderLayer.cpp        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/Source/WebCore/rendering/RenderLayer.cpp        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -995,7 +995,6 @@
</span><span class="cx">
</span><span class="cx"> RenderBox* box = renderBox();
</span><span class="cx"> ASSERT(box);
</span><del>- // FIXME: replace with call to AnimationController::isRunningAcceleratedAnimationOnRenderer() and remove RenderStyle::isRunningAcceleratedAnimation().
</del><span class="cx"> if (renderer().style().isRunningAcceleratedAnimation()) {
</span><span class="cx"> TransformationMatrix currTransform;
</span><span class="cx"> FloatRect pixelSnappedBorderRect = snapRectToDevicePixels(box->borderBoxRect(), box->document().deviceScaleFactor());
</span></span></pre></div>
<a id="branchessafari602branchSourceWebCorerenderingstyleWillChangeDatacpp"></a>
<div class="modfile"><h4>Modified: branches/safari-602-branch/Source/WebCore/rendering/style/WillChangeData.cpp (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/Source/WebCore/rendering/style/WillChangeData.cpp        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/Source/WebCore/rendering/style/WillChangeData.cpp        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -62,7 +62,7 @@
</span><span class="cx">
</span><span class="cx"> // "If any non-initial value of a property would create a stacking context on the element,
</span><span class="cx"> // specifying that property in will-change must create a stacking context on the element."
</span><del>-bool WillChangeData::propertyCreatesStackingContext(CSSPropertyID property)
</del><ins>+static bool propertyCreatesStackingContext(CSSPropertyID property)
</ins><span class="cx"> {
</span><span class="cx"> switch (property) {
</span><span class="cx"> case CSSPropertyPerspective:
</span></span></pre></div>
<a id="branchessafari602branchSourceWebCorerenderingstyleWillChangeDatah"></a>
<div class="modfile"><h4>Modified: branches/safari-602-branch/Source/WebCore/rendering/style/WillChangeData.h (208262 => 208263)</h4>
<pre class="diff"><span>
<span class="info">--- branches/safari-602-branch/Source/WebCore/rendering/style/WillChangeData.h        2016-11-02 00:25:51 UTC (rev 208262)
+++ branches/safari-602-branch/Source/WebCore/rendering/style/WillChangeData.h        2016-11-02 00:25:57 UTC (rev 208263)
</span><span class="lines">@@ -70,8 +70,6 @@
</span><span class="cx"> typedef std::pair<Feature, CSSPropertyID> FeaturePropertyPair;
</span><span class="cx"> FeaturePropertyPair featureAt(size_t) const;
</span><span class="cx">
</span><del>- static bool propertyCreatesStackingContext(CSSPropertyID);
-
</del><span class="cx"> private:
</span><span class="cx"> WillChangeData()
</span><span class="cx"> {
</span></span></pre>
</div>
</div>
</body>
</html>