<!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>[201715] trunk</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/201715">201715</a></dd>
<dt>Author</dt> <dd>ryanhaddad@apple.com</dd>
<dt>Date</dt> <dd>2016-06-06 10:56:33 -0700 (Mon, 06 Jun 2016)</dd>
</dl>
<h3>Log Message</h3>
<pre>Unreviewed, rolling out <a href="http://trac.webkit.org/projects/webkit/changeset/201706">r201706</a>.
https://bugs.webkit.org/show_bug.cgi?id=158427
This change broke the Yosemite build and its LayoutTest is
failing on Mac and iOS testers (Requested by ryanhaddad on
#webkit).
Reverted changeset:
"Add experimental support for spring based CSS animations"
https://bugs.webkit.org/show_bug.cgi?id=158403
http://trac.webkit.org/changeset/201706
Patch by Commit Queue <commit-queue@webkit.org> on 2016-06-06</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsChangeLog">trunk/LayoutTests/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreWebCorexcodeprojprojectpbxproj">trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj</a></li>
<li><a href="#trunkSourceWebCorecssCSSComputedStyleDeclarationcpp">trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp</a></li>
<li><a href="#trunkSourceWebCorecssCSSParsercpp">trunk/Source/WebCore/css/CSSParser.cpp</a></li>
<li><a href="#trunkSourceWebCorecssCSSParserh">trunk/Source/WebCore/css/CSSParser.h</a></li>
<li><a href="#trunkSourceWebCorecssCSSParserModeh">trunk/Source/WebCore/css/CSSParserMode.h</a></li>
<li><a href="#trunkSourceWebCorecssCSSTimingFunctionValuecpp">trunk/Source/WebCore/css/CSSTimingFunctionValue.cpp</a></li>
<li><a href="#trunkSourceWebCorecssCSSTimingFunctionValueh">trunk/Source/WebCore/css/CSSTimingFunctionValue.h</a></li>
<li><a href="#trunkSourceWebCorecssCSSToStyleMapcpp">trunk/Source/WebCore/css/CSSToStyleMap.cpp</a></li>
<li><a href="#trunkSourceWebCorecssCSSValuecpp">trunk/Source/WebCore/css/CSSValue.cpp</a></li>
<li><a href="#trunkSourceWebCorecssCSSValueh">trunk/Source/WebCore/css/CSSValue.h</a></li>
<li><a href="#trunkSourceWebCorepageSettingsin">trunk/Source/WebCore/page/Settings.in</a></li>
<li><a href="#trunkSourceWebCorepageanimationAnimationBasecpp">trunk/Source/WebCore/page/animation/AnimationBase.cpp</a></li>
<li><a href="#trunkSourceWebCoreplatformanimationTimingFunctioncpp">trunk/Source/WebCore/platform/animation/TimingFunction.cpp</a></li>
<li><a href="#trunkSourceWebCoreplatformanimationTimingFunctionh">trunk/Source/WebCore/platform/animation/TimingFunction.h</a></li>
<li><a href="#trunkSourceWebCoreplatformgraphicscaGraphicsLayerCAcpp">trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp</a></li>
<li><a href="#trunkSourceWebCoreplatformgraphicscaGraphicsLayerCAh">trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.h</a></li>
<li><a href="#trunkSourceWebCoreplatformgraphicscaPlatformCAAnimationcpp">trunk/Source/WebCore/platform/graphics/ca/PlatformCAAnimation.cpp</a></li>
<li><a href="#trunkSourceWebCoreplatformgraphicscaPlatformCAAnimationh">trunk/Source/WebCore/platform/graphics/ca/PlatformCAAnimation.h</a></li>
<li><a href="#trunkSourceWebCoreplatformgraphicscacocoaPlatformCAAnimationCocoamm">trunk/Source/WebCore/platform/graphics/ca/cocoa/PlatformCAAnimationCocoa.mm</a></li>
<li><a href="#trunkSourceWebKit2ChangeLog">trunk/Source/WebKit2/ChangeLog</a></li>
<li><a href="#trunkSourceWebKit2SharedCoordinatedGraphicsCoordinatedGraphicsArgumentCoderscpp">trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp</a></li>
<li><a href="#trunkSourceWebKit2SharedWebCoreArgumentCoderscpp">trunk/Source/WebKit2/Shared/WebCoreArgumentCoders.cpp</a></li>
<li><a href="#trunkSourceWebKit2SharedWebCoreArgumentCodersh">trunk/Source/WebKit2/Shared/WebCoreArgumentCoders.h</a></li>
<li><a href="#trunkSourceWebKit2SharedWebPreferencesDefinitionsh">trunk/Source/WebKit2/Shared/WebPreferencesDefinitions.h</a></li>
<li><a href="#trunkSourceWebKit2WebProcessWebPageWebPagecpp">trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp</a></li>
<li><a href="#trunkSourceWebKit2WebProcessWebPagemacPlatformCAAnimationRemotemm">trunk/Source/WebKit2/WebProcess/WebPage/mac/PlatformCAAnimationRemote.mm</a></li>
</ul>
<h3>Removed Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsanimationsscripttestsspringcomputedstylejs">trunk/LayoutTests/animations/script-tests/spring-computed-style.js</a></li>
<li><a href="#trunkLayoutTestsanimationsscripttestsspringparsingjs">trunk/LayoutTests/animations/script-tests/spring-parsing.js</a></li>
<li><a href="#trunkLayoutTestsanimationsspringcomputedstyleexpectedtxt">trunk/LayoutTests/animations/spring-computed-style-expected.txt</a></li>
<li><a href="#trunkLayoutTestsanimationsspringcomputedstylehtml">trunk/LayoutTests/animations/spring-computed-style.html</a></li>
<li><a href="#trunkLayoutTestsanimationsspringfunctionexpectedtxt">trunk/LayoutTests/animations/spring-function-expected.txt</a></li>
<li><a href="#trunkLayoutTestsanimationsspringfunctionhtml">trunk/LayoutTests/animations/spring-function.html</a></li>
<li><a href="#trunkLayoutTestsanimationsspringparsingexpectedtxt">trunk/LayoutTests/animations/spring-parsing-expected.txt</a></li>
<li><a href="#trunkLayoutTestsanimationsspringparsinghtml">trunk/LayoutTests/animations/spring-parsing.html</a></li>
<li><a href="#trunkSourceWebCoreplatformgraphicsSpringSolverh">trunk/Source/WebCore/platform/graphics/SpringSolver.h</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkLayoutTestsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/ChangeLog (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/ChangeLog 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/LayoutTests/ChangeLog 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,3 +1,18 @@
</span><ins>+2016-06-06 Commit Queue <commit-queue@webkit.org>
+
+ Unreviewed, rolling out r201706.
+ https://bugs.webkit.org/show_bug.cgi?id=158427
+
+ This change broke the Yosemite build and its LayoutTest is
+ failing on Mac and iOS testers (Requested by ryanhaddad on
+ #webkit).
+
+ Reverted changeset:
+
+ "Add experimental support for spring based CSS animations"
+ https://bugs.webkit.org/show_bug.cgi?id=158403
+ http://trac.webkit.org/changeset/201706
+
</ins><span class="cx"> 2016-06-03 Oliver Hunt <oliver@apple.com>
</span><span class="cx">
</span><span class="cx"> RegExp unicode parsing reads an extra character before failing
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsscripttestsspringcomputedstylejs"></a>
<div class="delfile"><h4>Deleted: trunk/LayoutTests/animations/script-tests/spring-computed-style.js (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/script-tests/spring-computed-style.js 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/LayoutTests/animations/script-tests/spring-computed-style.js 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,48 +0,0 @@
</span><del>-description("Test the computed style of the spring() function on the transition-timing-function property.");
-
-if (window.internals)
- internals.settings.setSpringTimingFunctionEnabled(true);
-
-// These have to be global for the test helpers to see them.
-var stylesheet, springStyle;
-var styleElement = document.createElement("style");
-document.head.appendChild(styleElement);
-stylesheet = styleElement.sheet;
-
-function testComputedSpring(description, spring, expectedValue)
-{
- debug("");
- debug(description + " : " + spring);
-
- stylesheet.insertRule("body { transition-timing-function: " + spring + "; }", 0);
-
- springStyle = window.getComputedStyle(document.body).getPropertyCSSValue("transition-timing-function");
- shouldBe("springStyle.cssText", "'" + expectedValue + "'");
-
- stylesheet.deleteRule(0);
-}
-
-debug("")
-debug("Valid spring tests");
-debug("")
-
-testComputedSpring("Basic", "spring(1 100 10 0)", "spring(1 100 10 0)");
-testComputedSpring("Negative Velocity", "spring(1 100 10 -10)", "spring(1 100 10 -10)");
-testComputedSpring("Positive Velocity", "spring(1 100 10 10)", "spring(1 100 10 10)");
-testComputedSpring("Zero Damping", "spring(1 100 0 10)", "spring(1 100 0 10)");
-testComputedSpring("Minimum Values", "spring(1 1 0 -999999)", "spring(1 1 0 -999999)");
-
-debug("")
-debug("Invalid spring tests");
-debug("")
-
-testComputedSpring("No parameters", "spring()", "ease");
-testComputedSpring("Not enough parameters", "spring(1 100 10)", "ease");
-testComputedSpring("Too many parameters", "spring(1 100 10 0 0)", "ease");
-testComputedSpring("Illegal Mass (< 0)", "spring(-1 100 10 0)", "ease");
-testComputedSpring("Illegal Mass (== 0)", "spring(0 100 10 0)", "ease");
-testComputedSpring("Illegal Stiffness (< 0)", "spring(1 -1 10 0)", "ease");
-testComputedSpring("Illegal Stiffness (== 0)", "spring(1 0 10 0)", "ease");
-testComputedSpring("Illegal Damping (< 0)", "spring(1 100 -1 0)", "ease");
-
-successfullyParsed = true;
</del></span></pre></div>
<a id="trunkLayoutTestsanimationsscripttestsspringparsingjs"></a>
<div class="delfile"><h4>Deleted: trunk/LayoutTests/animations/script-tests/spring-parsing.js (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/script-tests/spring-parsing.js 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/LayoutTests/animations/script-tests/spring-parsing.js 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,61 +0,0 @@
</span><del>-description("Test the parsing of the spring() function on the transition-timing-function property.");
-
-if (window.internals)
- internals.settings.setSpringTimingFunctionEnabled(true);
-
-// These have to be global for the test helpers to see them.
-var stylesheet, cssRule, declaration, propertyValue, subRule;
-var styleElement = document.createElement("style");
-document.head.appendChild(styleElement);
-stylesheet = styleElement.sheet;
-
-function testSpring(description, spring, expectedValue)
-{
- debug("");
- debug(description + " : " + spring);
-
- stylesheet.insertRule("body { transition-timing-function: " + spring + "; }", 0);
- cssRule = stylesheet.cssRules.item(0);
-
- shouldBe("cssRule.type", "1");
-
- declaration = cssRule.style;
- if (!expectedValue) {
- shouldBe("declaration.length", "0");
- shouldBeEqualToString("declaration.getPropertyValue('transition-timing-function')", "");
- return;
- }
-
- shouldBe("declaration.length", "1");
- shouldBe("declaration.getPropertyValue('transition-timing-function')", "'" + expectedValue + "'");
-
- propertyValue = declaration.getPropertyCSSValue("transition-timing-function");
- shouldBe("propertyValue.cssText", "'" + expectedValue + "'");
-
- stylesheet.deleteRule(0);
-}
-
-debug("")
-debug("Valid spring tests");
-debug("")
-
-testSpring("Basic", "spring(1 100 10 0)", "spring(1 100 10 0)");
-testSpring("Negative Velocity", "spring(1 100 10 -10)", "spring(1 100 10 -10)");
-testSpring("Positive Velocity", "spring(1 100 10 10)", "spring(1 100 10 10)");
-testSpring("Zero Damping", "spring(1 100 0 10)", "spring(1 100 0 10)");
-testSpring("Minimum Values", "spring(1 1 0 -999999)", "spring(1 1 0 -999999)");
-
-debug("")
-debug("Invalid spring tests");
-debug("")
-
-testSpring("No parameters", "spring()", null);
-testSpring("Not enough parameters", "spring(1 100 10)", null);
-testSpring("Too many parameters", "spring(1 100 10 0 0)", null);
-testSpring("Illegal Mass (< 0)", "spring(-1 100 10 0)", null);
-testSpring("Illegal Mass (== 0)", "spring(0 100 10 0)", null);
-testSpring("Illegal Stiffness (< 0)", "spring(1 -1 10 0)", null);
-testSpring("Illegal Stiffness (== 0)", "spring(1 0 10 0)", null);
-testSpring("Illegal Damping (< 0)", "spring(1 100 -1 0)", null);
-
-successfullyParsed = true;
</del></span></pre></div>
<a id="trunkLayoutTestsanimationsspringcomputedstyleexpectedtxt"></a>
<div class="delfile"><h4>Deleted: trunk/LayoutTests/animations/spring-computed-style-expected.txt (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/spring-computed-style-expected.txt 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/LayoutTests/animations/spring-computed-style-expected.txt 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,54 +0,0 @@
</span><del>-Test the computed style of the spring() function on the transition-timing-function property.
-
-On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
-
-
-
-Valid spring tests
-
-
-Basic : spring(1 100 10 0)
-PASS springStyle.cssText is 'spring(1 100 10 0)'
-
-Negative Velocity : spring(1 100 10 -10)
-PASS springStyle.cssText is 'spring(1 100 10 -10)'
-
-Positive Velocity : spring(1 100 10 10)
-PASS springStyle.cssText is 'spring(1 100 10 10)'
-
-Zero Damping : spring(1 100 0 10)
-PASS springStyle.cssText is 'spring(1 100 0 10)'
-
-Minimum Values : spring(1 1 0 -999999)
-PASS springStyle.cssText is 'spring(1 1 0 -999999)'
-
-Invalid spring tests
-
-
-No parameters : spring()
-PASS springStyle.cssText is 'ease'
-
-Not enough parameters : spring(1 100 10)
-PASS springStyle.cssText is 'ease'
-
-Too many parameters : spring(1 100 10 0 0)
-PASS springStyle.cssText is 'ease'
-
-Illegal Mass (< 0) : spring(-1 100 10 0)
-PASS springStyle.cssText is 'ease'
-
-Illegal Mass (== 0) : spring(0 100 10 0)
-PASS springStyle.cssText is 'ease'
-
-Illegal Stiffness (< 0) : spring(1 -1 10 0)
-PASS springStyle.cssText is 'ease'
-
-Illegal Stiffness (== 0) : spring(1 0 10 0)
-PASS springStyle.cssText is 'ease'
-
-Illegal Damping (< 0) : spring(1 100 -1 0)
-PASS springStyle.cssText is 'ease'
-PASS successfullyParsed is true
-
-TEST COMPLETE
-
</del></span></pre></div>
<a id="trunkLayoutTestsanimationsspringcomputedstylehtml"></a>
<div class="delfile"><h4>Deleted: trunk/LayoutTests/animations/spring-computed-style.html (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/spring-computed-style.html 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/LayoutTests/animations/spring-computed-style.html 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,6 +0,0 @@
</span><del>-<!DOCTYPE HTML>
-<script src="../resources/js-test-pre.js"></script>
-<p id="description"></p>
-<div id="console"></div>
-<script src="script-tests/spring-computed-style.js"></script>
-<script src="../resources/js-test-post.js"></script>
</del></span></pre></div>
<a id="trunkLayoutTestsanimationsspringfunctionexpectedtxt"></a>
<div class="delfile"><h4>Deleted: trunk/LayoutTests/animations/spring-function-expected.txt (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/spring-function-expected.txt 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/LayoutTests/animations/spring-function-expected.txt 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,5 +0,0 @@
</span><del>-This test performs an animation of the left property. It animates over 1 second. It takes 3 snapshots and expects each result to be within a specified range.
-PASS - "left" property for "box1" element at 0.25s saw something close to: 512
-PASS - "left" property for "box1" element at 0.5s saw something close to: 537
-PASS - "left" property for "box1" element at 0.75s saw something close to: 487
-
</del></span></pre></div>
<a id="trunkLayoutTestsanimationsspringfunctionhtml"></a>
<div class="delfile"><h4>Deleted: trunk/LayoutTests/animations/spring-function.html (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/spring-function.html 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/LayoutTests/animations/spring-function.html 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,51 +0,0 @@
</span><del>-<!DOCTYPE html>
-<html lang="en">
-<head>
- <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
- <title>Test timing functions</title>
- <script>
- if (window.internals)
- internals.settings.setSpringTimingFunctionEnabled(true);
- </script>
- <style type="text/css" media="screen">
- .box {
- position: relative;
- left: 100px;
- top: 0px;
- height: 5px;
- width: 5px;
- background-color: blue;
- animation-duration: 1s;
- animation-name: anim;
- }
- @keyframes anim {
- from { left: 0px; }
- to { left: 500px; }
- }
- #box1 {
- animation-timing-function: spring(1 100 10 0);
- }
-
- </style>
- <script src="resources/animation-test-helpers.js" type="text/javascript" charset="utf-8"></script>
- <script type="text/javascript" charset="utf-8">
-
- const expectedValues = [
- // [animation-name, time, element-id, property, expected-value, tolerance]
- ["anim", 0.25, "box1", "left", 512, 5],
- ["anim", 0.50, "box1", "left", 537, 5],
- ["anim", 0.75, "box1", "left", 487, 5],
- ];
-
- runAnimationTest(expectedValues);
-
- </script>
-</head>
-<body>
-This test performs an animation of the left property. It animates over 1 second.
-It takes 3 snapshots and expects each result to be within a specified range.
-<div class="box" id="box1"></div>
-<div id="result">
-</div>
-</body>
-</html>
</del></span></pre></div>
<a id="trunkLayoutTestsanimationsspringparsingexpectedtxt"></a>
<div class="delfile"><h4>Deleted: trunk/LayoutTests/animations/spring-parsing-expected.txt (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/spring-parsing-expected.txt 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/LayoutTests/animations/spring-parsing-expected.txt 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,85 +0,0 @@
</span><del>-Test the parsing of the spring() function on the transition-timing-function property.
-
-On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
-
-
-
-Valid spring tests
-
-
-Basic : spring(1 100 10 0)
-PASS cssRule.type is 1
-PASS declaration.length is 1
-PASS declaration.getPropertyValue('transition-timing-function') is 'spring(1 100 10 0)'
-PASS propertyValue.cssText is 'spring(1 100 10 0)'
-
-Negative Velocity : spring(1 100 10 -10)
-PASS cssRule.type is 1
-PASS declaration.length is 1
-PASS declaration.getPropertyValue('transition-timing-function') is 'spring(1 100 10 -10)'
-PASS propertyValue.cssText is 'spring(1 100 10 -10)'
-
-Positive Velocity : spring(1 100 10 10)
-PASS cssRule.type is 1
-PASS declaration.length is 1
-PASS declaration.getPropertyValue('transition-timing-function') is 'spring(1 100 10 10)'
-PASS propertyValue.cssText is 'spring(1 100 10 10)'
-
-Zero Damping : spring(1 100 0 10)
-PASS cssRule.type is 1
-PASS declaration.length is 1
-PASS declaration.getPropertyValue('transition-timing-function') is 'spring(1 100 0 10)'
-PASS propertyValue.cssText is 'spring(1 100 0 10)'
-
-Minimum Values : spring(1 1 0 -999999)
-PASS cssRule.type is 1
-PASS declaration.length is 1
-PASS declaration.getPropertyValue('transition-timing-function') is 'spring(1 1 0 -999999)'
-PASS propertyValue.cssText is 'spring(1 1 0 -999999)'
-
-Invalid spring tests
-
-
-No parameters : spring()
-PASS cssRule.type is 1
-PASS declaration.length is 0
-PASS declaration.getPropertyValue('transition-timing-function') is ""
-
-Not enough parameters : spring(1 100 10)
-PASS cssRule.type is 1
-PASS declaration.length is 0
-PASS declaration.getPropertyValue('transition-timing-function') is ""
-
-Too many parameters : spring(1 100 10 0 0)
-PASS cssRule.type is 1
-PASS declaration.length is 0
-PASS declaration.getPropertyValue('transition-timing-function') is ""
-
-Illegal Mass (< 0) : spring(-1 100 10 0)
-PASS cssRule.type is 1
-PASS declaration.length is 0
-PASS declaration.getPropertyValue('transition-timing-function') is ""
-
-Illegal Mass (== 0) : spring(0 100 10 0)
-PASS cssRule.type is 1
-PASS declaration.length is 0
-PASS declaration.getPropertyValue('transition-timing-function') is ""
-
-Illegal Stiffness (< 0) : spring(1 -1 10 0)
-PASS cssRule.type is 1
-PASS declaration.length is 0
-PASS declaration.getPropertyValue('transition-timing-function') is ""
-
-Illegal Stiffness (== 0) : spring(1 0 10 0)
-PASS cssRule.type is 1
-PASS declaration.length is 0
-PASS declaration.getPropertyValue('transition-timing-function') is ""
-
-Illegal Damping (< 0) : spring(1 100 -1 0)
-PASS cssRule.type is 1
-PASS declaration.length is 0
-PASS declaration.getPropertyValue('transition-timing-function') is ""
-PASS successfullyParsed is true
-
-TEST COMPLETE
-
</del></span></pre></div>
<a id="trunkLayoutTestsanimationsspringparsinghtml"></a>
<div class="delfile"><h4>Deleted: trunk/LayoutTests/animations/spring-parsing.html (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/spring-parsing.html 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/LayoutTests/animations/spring-parsing.html 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,6 +0,0 @@
</span><del>-<!DOCTYPE HTML>
-<script src="../resources/js-test-pre.js"></script>
-<p id="description"></p>
-<div id="console"></div>
-<script src="script-tests/spring-parsing.js"></script>
-<script src="../resources/js-test-post.js"></script>
</del></span></pre></div>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/ChangeLog 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,3 +1,18 @@
</span><ins>+2016-06-06 Commit Queue <commit-queue@webkit.org>
+
+ Unreviewed, rolling out r201706.
+ https://bugs.webkit.org/show_bug.cgi?id=158427
+
+ This change broke the Yosemite build and its LayoutTest is
+ failing on Mac and iOS testers (Requested by ryanhaddad on
+ #webkit).
+
+ Reverted changeset:
+
+ "Add experimental support for spring based CSS animations"
+ https://bugs.webkit.org/show_bug.cgi?id=158403
+ http://trac.webkit.org/changeset/201706
+
</ins><span class="cx"> 2016-06-06 Javier Fernandez <jfernandez@igalia.com>
</span><span class="cx">
</span><span class="cx"> [css-grid] Horizontal scroll must account for grid container's height
</span></span></pre></div>
<a id="trunkSourceWebCoreWebCorexcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -2764,7 +2764,6 @@
</span><span class="cx"> 7C74D43C1882400400E5ED57 /* UTextProviderUTF16.h in Headers */ = {isa = PBXBuildFile; fileRef = 7C74D43A1882400400E5ED57 /* UTextProviderUTF16.h */; };
</span><span class="cx"> 7C7941E41C56C29300A4C58E /* DataDetectorsCoreSoftLink.mm in Sources */ = {isa = PBXBuildFile; fileRef = 7C7941E21C56C29300A4C58E /* DataDetectorsCoreSoftLink.mm */; };
</span><span class="cx"> 7C7941E51C56C29300A4C58E /* DataDetectorsCoreSoftLink.h in Headers */ = {isa = PBXBuildFile; fileRef = 7C7941E31C56C29300A4C58E /* DataDetectorsCoreSoftLink.h */; };
</span><del>- 7C83DE861D04CC5D00FEBCF3 /* SpringSolver.h in Headers */ = {isa = PBXBuildFile; fileRef = 7C83DE851D04CBD400FEBCF3 /* SpringSolver.h */; };
</del><span class="cx"> 7C91A38F1B498ABE003F9EFA /* JSNodeOrString.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 7C91A38D1B498ABE003F9EFA /* JSNodeOrString.cpp */; };
</span><span class="cx"> 7C91A3901B498ABE003F9EFA /* JSNodeOrString.h in Headers */ = {isa = PBXBuildFile; fileRef = 7C91A38E1B498ABE003F9EFA /* JSNodeOrString.h */; };
</span><span class="cx"> 7C93F3491AA6BA5E00A98BAB /* CompiledContentExtension.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 7C93F3471AA6BA5E00A98BAB /* CompiledContentExtension.cpp */; };
</span><span class="lines">@@ -10431,7 +10430,6 @@
</span><span class="cx"> 7C74D43A1882400400E5ED57 /* UTextProviderUTF16.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = UTextProviderUTF16.h; sourceTree = "<group>"; };
</span><span class="cx"> 7C7941E21C56C29300A4C58E /* DataDetectorsCoreSoftLink.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = DataDetectorsCoreSoftLink.mm; sourceTree = "<group>"; };
</span><span class="cx"> 7C7941E31C56C29300A4C58E /* DataDetectorsCoreSoftLink.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = DataDetectorsCoreSoftLink.h; sourceTree = "<group>"; };
</span><del>- 7C83DE851D04CBD400FEBCF3 /* SpringSolver.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = SpringSolver.h; sourceTree = "<group>"; };
</del><span class="cx"> 7C91A38D1B498ABE003F9EFA /* JSNodeOrString.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSNodeOrString.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 7C91A38E1B498ABE003F9EFA /* JSNodeOrString.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSNodeOrString.h; sourceTree = "<group>"; };
</span><span class="cx"> 7C93F3471AA6BA5E00A98BAB /* CompiledContentExtension.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = CompiledContentExtension.cpp; sourceTree = "<group>"; };
</span><span class="lines">@@ -22066,7 +22064,6 @@
</span><span class="cx"> 0F3DD44E12F5EA1B000D9190 /* ShadowBlur.h */,
</span><span class="cx"> CD641EB21818F5ED00EE4C41 /* SourceBufferPrivate.h */,
</span><span class="cx"> CDC8B5AC1804AE5D0016E685 /* SourceBufferPrivateClient.h */,
</span><del>- 7C83DE851D04CBD400FEBCF3 /* SpringSolver.h */,
</del><span class="cx"> B23540F00D00782E002382FA /* StringTruncator.cpp */,
</span><span class="cx"> B23540F10D00782E002382FA /* StringTruncator.h */,
</span><span class="cx"> 849F77750EFEC6200090849D /* StrokeStyleApplier.h */,
</span><span class="lines">@@ -27843,7 +27840,6 @@
</span><span class="cx"> BC8B854B0E7C7F5600AB6984 /* ScrollbarTheme.h in Headers */,
</span><span class="cx"> BC14028B0E83680800319717 /* ScrollbarThemeComposite.h in Headers */,
</span><span class="cx"> 44C991860F3D1EBE00586670 /* ScrollbarThemeIOS.h in Headers */,
</span><del>- 7C83DE861D04CC5D00FEBCF3 /* SpringSolver.h in Headers */,
</del><span class="cx"> BC8B853E0E7C7F1100AB6984 /* ScrollbarThemeMac.h in Headers */,
</span><span class="cx"> 0FE71406142170B800DB33BA /* ScrollbarThemeMock.h in Headers */,
</span><span class="cx"> 5D925B680F64D4DD00B847F0 /* ScrollBehavior.h in Headers */,
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSComputedStyleDeclarationcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1498,10 +1498,10 @@
</span><span class="cx"> {
</span><span class="cx"> switch (timingFunction->type()) {
</span><span class="cx"> case TimingFunction::CubicBezierFunction: {
</span><del>- auto& function = *static_cast<const CubicBezierTimingFunction*>(timingFunction);
- if (function.timingFunctionPreset() != CubicBezierTimingFunction::Custom) {
</del><ins>+ const CubicBezierTimingFunction* bezierTimingFunction = static_cast<const CubicBezierTimingFunction*>(timingFunction);
+ if (bezierTimingFunction->timingFunctionPreset() != CubicBezierTimingFunction::Custom) {
</ins><span class="cx"> CSSValueID valueId = CSSValueInvalid;
</span><del>- switch (function.timingFunctionPreset()) {
</del><ins>+ switch (bezierTimingFunction->timingFunctionPreset()) {
</ins><span class="cx"> case CubicBezierTimingFunction::Ease:
</span><span class="cx"> valueId = CSSValueEase;
</span><span class="cx"> break;
</span><span class="lines">@@ -1512,22 +1512,18 @@
</span><span class="cx"> valueId = CSSValueEaseOut;
</span><span class="cx"> break;
</span><span class="cx"> default:
</span><del>- ASSERT(function.timingFunctionPreset() == CubicBezierTimingFunction::EaseInOut);
</del><ins>+ ASSERT(bezierTimingFunction->timingFunctionPreset() == CubicBezierTimingFunction::EaseInOut);
</ins><span class="cx"> valueId = CSSValueEaseInOut;
</span><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> return CSSValuePool::singleton().createIdentifierValue(valueId);
</span><span class="cx"> }
</span><del>- return CSSCubicBezierTimingFunctionValue::create(function.x1(), function.y1(), function.x2(), function.y2());
</del><ins>+ return CSSCubicBezierTimingFunctionValue::create(bezierTimingFunction->x1(), bezierTimingFunction->y1(), bezierTimingFunction->x2(), bezierTimingFunction->y2());
</ins><span class="cx"> }
</span><span class="cx"> case TimingFunction::StepsFunction: {
</span><del>- auto& function = *static_cast<const StepsTimingFunction*>(timingFunction);
- return CSSStepsTimingFunctionValue::create(function.numberOfSteps(), function.stepAtStart());
</del><ins>+ const StepsTimingFunction* stepsTimingFunction = static_cast<const StepsTimingFunction*>(timingFunction);
+ return CSSStepsTimingFunctionValue::create(stepsTimingFunction->numberOfSteps(), stepsTimingFunction->stepAtStart());
</ins><span class="cx"> }
</span><del>- case TimingFunction::SpringFunction: {
- auto& function = *static_cast<const SpringTimingFunction*>(timingFunction);
- return CSSSpringTimingFunctionValue::create(function.mass(), function.stiffness(), function.damping(), function.initialVelocity());
- }
</del><span class="cx"> default:
</span><span class="cx"> ASSERT(timingFunction->type() == TimingFunction::LinearFunction);
</span><span class="cx"> return CSSValuePool::singleton().createIdentifierValue(CSSValueLinear);
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSParsercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSParser.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSParser.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/css/CSSParser.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -270,7 +270,6 @@
</span><span class="cx"> #if ENABLE(IOS_TEXT_AUTOSIZING)
</span><span class="cx"> textAutosizingEnabled = settings->textAutosizingEnabled();
</span><span class="cx"> #endif
</span><del>- springTimingFunctionEnabled = settings->springTimingFunctionEnabled();
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> #if PLATFORM(IOS)
</span><span class="lines">@@ -292,8 +291,7 @@
</span><span class="cx"> #endif
</span><span class="cx"> && a.needsSiteSpecificQuirks == b.needsSiteSpecificQuirks
</span><span class="cx"> && a.enforcesCSSMIMETypeInNoQuirksMode == b.enforcesCSSMIMETypeInNoQuirksMode
</span><del>- && a.useLegacyBackgroundSizeShorthandBehavior == b.useLegacyBackgroundSizeShorthandBehavior
- && a.springTimingFunctionEnabled == b.springTimingFunctionEnabled;
</del><ins>+ && a.useLegacyBackgroundSizeShorthandBehavior == b.useLegacyBackgroundSizeShorthandBehavior;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> CSSParser::CSSParser(const CSSParserContext& context)
</span><span class="lines">@@ -5159,38 +5157,23 @@
</span><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-bool CSSParser::isSpringTimingFunctionEnabled() const
</del><ins>+bool CSSParser::parseCubicBezierTimingFunctionValue(CSSParserValueList& args, double& result)
</ins><span class="cx"> {
</span><del>- return m_context.springTimingFunctionEnabled;
-}
-
-Optional<double> CSSParser::parseCubicBezierTimingFunctionValue(CSSParserValueList& args)
-{
</del><span class="cx"> ValueWithCalculation argumentWithCalculation(*args.current());
</span><span class="cx"> if (!validateUnit(argumentWithCalculation, FNumber))
</span><del>- return Nullopt;
- Optional<double> result = parsedDouble(argumentWithCalculation);
</del><ins>+ return false;
+ result = parsedDouble(argumentWithCalculation);
</ins><span class="cx"> CSSParserValue* nextValue = args.next();
</span><span class="cx"> if (!nextValue) {
</span><span class="cx"> // The last number in the function has no comma after it, so we're done.
</span><del>- return result;
</del><ins>+ return true;
</ins><span class="cx"> }
</span><span class="cx"> if (!isComma(nextValue))
</span><del>- return Nullopt;
</del><ins>+ return false;
</ins><span class="cx"> args.next();
</span><del>- return result;
</del><ins>+ return true;
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-Optional<double> CSSParser::parseSpringTimingFunctionValue(CSSParserValueList& args)
-{
- ValueWithCalculation argumentWithCalculation(*args.current());
- if (!validateUnit(argumentWithCalculation, FNumber))
- return Nullopt;
- Optional<double> result = parsedDouble(argumentWithCalculation);
- args.next();
- return result;
-}
-
</del><span class="cx"> RefPtr<CSSValue> CSSParser::parseAnimationTimingFunction()
</span><span class="cx"> {
</span><span class="cx"> CSSParserValue& value = *m_valueList->current();
</span><span class="lines">@@ -5236,69 +5219,29 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if (equalLettersIgnoringASCIICase(value.function->name, "cubic-bezier(")) {
</span><del>- // For cubic bezier, 4 values must be specified (comma-separated).
</del><ins>+ // For cubic bezier, 4 values must be specified.
</ins><span class="cx"> if (!args || args->size() != 7)
</span><span class="cx"> return nullptr;
</span><span class="cx">
</span><span class="cx"> // There are two points specified. The x values must be between 0 and 1 but the y values can exceed this range.
</span><ins>+ double x1, y1, x2, y2;
</ins><span class="cx">
</span><del>- auto x1 = parseCubicBezierTimingFunctionValue(*args);
- if (!x1)
</del><ins>+ if (!parseCubicBezierTimingFunctionValue(*args, x1))
</ins><span class="cx"> return nullptr;
</span><del>- if (x1.value() < 0 || x1.value() > 1)
</del><ins>+ if (x1 < 0 || x1 > 1)
</ins><span class="cx"> return nullptr;
</span><del>-
- auto y1 = parseCubicBezierTimingFunctionValue(*args);
- if (!y1)
</del><ins>+ if (!parseCubicBezierTimingFunctionValue(*args, y1))
</ins><span class="cx"> return nullptr;
</span><del>-
- auto x2 = parseCubicBezierTimingFunctionValue(*args);
- if (!x2)
</del><ins>+ if (!parseCubicBezierTimingFunctionValue(*args, x2))
</ins><span class="cx"> return nullptr;
</span><del>- if (x2.value() < 0 || x2.value() > 1)
</del><ins>+ if (x2 < 0 || x2 > 1)
</ins><span class="cx"> return nullptr;
</span><del>-
- auto y2 = parseCubicBezierTimingFunctionValue(*args);
- if (!y2)
</del><ins>+ if (!parseCubicBezierTimingFunctionValue(*args, y2))
</ins><span class="cx"> return nullptr;
</span><span class="cx">
</span><del>- return CSSCubicBezierTimingFunctionValue::create(x1.value(), y1.value(), x2.value(), y2.value());
</del><ins>+ return CSSCubicBezierTimingFunctionValue::create(x1, y1, x2, y2);
</ins><span class="cx"> }
</span><span class="cx">
</span><del>- if (isSpringTimingFunctionEnabled() && equalLettersIgnoringASCIICase(value.function->name, "spring(")) {
- // For a spring, 4 values must be specified (space-separated).
- if (!args || args->size() != 4)
- return nullptr;
-
- // Mass must be greater than 0.
- auto mass = parseSpringTimingFunctionValue(*args);
- if (!mass)
- return nullptr;
- if (mass.value() <= 0)
- return nullptr;
-
- // Stiffness must be greater than 0.
- auto stiffness = parseSpringTimingFunctionValue(*args);
- if (!stiffness)
- return nullptr;
- if (stiffness.value() <= 0)
- return nullptr;
-
- // Damping coefficient must be greater than or equal to 0.
- auto damping = parseSpringTimingFunctionValue(*args);
- if (!damping)
- return nullptr;
- if (damping.value() < 0)
- return nullptr;
-
- // Initial velocity may have any value.
- auto initialVelocity = parseSpringTimingFunctionValue(*args);
- if (!initialVelocity)
- return nullptr;
-
- return CSSSpringTimingFunctionValue::create(mass.value(), stiffness.value(), damping.value(), initialVelocity.value());
- }
-
</del><span class="cx"> return nullptr;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSParserh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSParser.h (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSParser.h 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/css/CSSParser.h 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -214,14 +214,11 @@
</span><span class="cx"> static Vector<double> parseKeyframeSelector(const String&);
</span><span class="cx">
</span><span class="cx"> bool parseTransformOriginShorthand(RefPtr<CSSPrimitiveValue>&, RefPtr<CSSPrimitiveValue>&, RefPtr<CSSValue>&);
</span><del>- Optional<double> parseCubicBezierTimingFunctionValue(CSSParserValueList&);
- Optional<double> parseSpringTimingFunctionValue(CSSParserValueList&);
</del><ins>+ bool parseCubicBezierTimingFunctionValue(CSSParserValueList& args, double& result);
</ins><span class="cx"> bool parseAnimationProperty(CSSPropertyID, RefPtr<CSSValue>&, AnimationParseContext&);
</span><span class="cx"> bool parseTransitionShorthand(CSSPropertyID, bool important);
</span><span class="cx"> bool parseAnimationShorthand(CSSPropertyID, bool important);
</span><span class="cx">
</span><del>- bool isSpringTimingFunctionEnabled() const;
-
</del><span class="cx"> RefPtr<CSSPrimitiveValue> parseColumnWidth();
</span><span class="cx"> RefPtr<CSSPrimitiveValue> parseColumnCount();
</span><span class="cx"> bool parseColumnsShorthand(bool important);
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSParserModeh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSParserMode.h (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSParserMode.h 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/css/CSSParserMode.h 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -73,7 +73,6 @@
</span><span class="cx"> bool needsSiteSpecificQuirks { false };
</span><span class="cx"> bool enforcesCSSMIMETypeInNoQuirksMode { true };
</span><span class="cx"> bool useLegacyBackgroundSizeShorthandBehavior { false };
</span><del>- bool springTimingFunctionEnabled { false };
</del><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> bool operator==(const CSSParserContext&, const CSSParserContext&);
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSTimingFunctionValuecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSTimingFunctionValue.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSTimingFunctionValue.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/css/CSSTimingFunctionValue.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,5 +1,5 @@
</span><span class="cx"> /*
</span><del>- * Copyright (C) 2007, 2013, 2016 Apple Inc. All rights reserved.
</del><ins>+ * Copyright (C) 2007, 2013 Apple Inc. All rights reserved.
</ins><span class="cx"> *
</span><span class="cx"> * Redistribution and use in source and binary forms, with or without
</span><span class="cx"> * modification, are permitted provided that the following conditions
</span><span class="lines">@@ -26,23 +26,17 @@
</span><span class="cx"> #include "config.h"
</span><span class="cx"> #include "CSSTimingFunctionValue.h"
</span><span class="cx">
</span><del>-#include <wtf/text/StringBuilder.h>
</del><ins>+#include <wtf/text/WTFString.h>
</ins><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><span class="cx"> String CSSCubicBezierTimingFunctionValue::customCSSText() const
</span><span class="cx"> {
</span><del>- StringBuilder builder;
- builder.appendLiteral("cubic-bezier(");
- builder.appendNumber(m_x1);
- builder.appendLiteral(", ");
- builder.appendNumber(m_y1);
- builder.appendLiteral(", ");
- builder.appendNumber(m_x2);
- builder.appendLiteral(", ");
- builder.appendNumber(m_y2);
- builder.append(')');
- return builder.toString();
</del><ins>+ return "cubic-bezier("
+ + String::number(m_x1) + ", "
+ + String::number(m_y1) + ", "
+ + String::number(m_x2) + ", "
+ + String::number(m_y2) + ')';
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool CSSCubicBezierTimingFunctionValue::equals(const CSSCubicBezierTimingFunctionValue& other) const
</span><span class="lines">@@ -50,16 +44,10 @@
</span><span class="cx"> return m_x1 == other.m_x1 && m_x2 == other.m_x2 && m_y1 == other.m_y1 && m_y2 == other.m_y2;
</span><span class="cx"> }
</span><span class="cx">
</span><ins>+
</ins><span class="cx"> String CSSStepsTimingFunctionValue::customCSSText() const
</span><span class="cx"> {
</span><del>- StringBuilder builder;
- builder.appendLiteral("steps(");
- builder.appendNumber(m_steps);
- if (m_stepAtStart)
- builder.appendLiteral(", start)");
- else
- builder.appendLiteral(", end)");
- return builder.toString();
</del><ins>+ return "steps(" + String::number(m_steps) + ", " + (m_stepAtStart ? "start" : "end") + ')';
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool CSSStepsTimingFunctionValue::equals(const CSSStepsTimingFunctionValue& other) const
</span><span class="lines">@@ -67,25 +55,4 @@
</span><span class="cx"> return m_steps == other.m_steps && m_stepAtStart == other.m_stepAtStart;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-String CSSSpringTimingFunctionValue::customCSSText() const
-{
- StringBuilder builder;
- builder.appendLiteral("spring(");
- builder.appendNumber(m_mass);
- builder.append(' ');
- builder.appendNumber(m_stiffness);
- builder.append(' ');
- builder.appendNumber(m_damping);
- builder.append(' ');
- builder.appendNumber(m_initialVelocity);
- builder.append(')');
- return builder.toString();
-}
-
-bool CSSSpringTimingFunctionValue::equals(const CSSSpringTimingFunctionValue& other) const
-{
- return m_mass == other.m_mass && m_stiffness == other.m_stiffness && m_damping == other.m_damping && m_initialVelocity == other.m_initialVelocity;
-}
-
-
</del><span class="cx"> } // namespace WebCore
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSTimingFunctionValueh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSTimingFunctionValue.h (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSTimingFunctionValue.h 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/css/CSSTimingFunctionValue.h 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,5 +1,5 @@
</span><span class="cx"> /*
</span><del>- * Copyright (C) 2007, 2008, 2012, 2016 Apple Inc. All rights reserved.
</del><ins>+ * Copyright (C) 2007, 2008, 2012 Apple Inc. All rights reserved.
</ins><span class="cx"> *
</span><span class="cx"> * Redistribution and use in source and binary forms, with or without
</span><span class="cx"> * modification, are permitted provided that the following conditions
</span><span class="lines">@@ -89,42 +89,9 @@
</span><span class="cx"> bool m_stepAtStart;
</span><span class="cx"> };
</span><span class="cx">
</span><del>-class CSSSpringTimingFunctionValue : public CSSValue {
-public:
- static Ref<CSSSpringTimingFunctionValue> create(double mass, double stiffness, double damping, double initialVelocity)
- {
- return adoptRef(*new CSSSpringTimingFunctionValue(mass, stiffness, damping, initialVelocity));
- }
-
- double mass() const { return m_mass; }
- double stiffness() const { return m_stiffness; }
- double damping() const { return m_damping; }
- double initialVelocity() const { return m_initialVelocity; }
-
- String customCSSText() const;
-
- bool equals(const CSSSpringTimingFunctionValue&) const;
-
-private:
- CSSSpringTimingFunctionValue(double mass, double stiffness, double damping, double initialVelocity)
- : CSSValue(SpringTimingFunctionClass)
- , m_mass(mass)
- , m_stiffness(stiffness)
- , m_damping(damping)
- , m_initialVelocity(initialVelocity)
- {
- }
-
- double m_mass;
- double m_stiffness;
- double m_damping;
- double m_initialVelocity;
-};
-
</del><span class="cx"> } // namespace WebCore
</span><span class="cx">
</span><span class="cx"> SPECIALIZE_TYPE_TRAITS_CSS_VALUE(CSSCubicBezierTimingFunctionValue, isCubicBezierTimingFunctionValue())
</span><span class="cx"> SPECIALIZE_TYPE_TRAITS_CSS_VALUE(CSSStepsTimingFunctionValue, isStepsTimingFunctionValue())
</span><del>-SPECIALIZE_TYPE_TRAITS_CSS_VALUE(CSSSpringTimingFunctionValue, isSpringTimingFunctionValue())
</del><span class="cx">
</span><span class="cx"> #endif // CSSTimingFunctionValue_h
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSToStyleMapcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSToStyleMap.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSToStyleMap.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/css/CSSToStyleMap.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -510,9 +510,6 @@
</span><span class="cx"> } else if (is<CSSStepsTimingFunctionValue>(value)) {
</span><span class="cx"> auto& stepsTimingFunction = downcast<CSSStepsTimingFunctionValue>(value);
</span><span class="cx"> animation.setTimingFunction(StepsTimingFunction::create(stepsTimingFunction.numberOfSteps(), stepsTimingFunction.stepAtStart()));
</span><del>- } else if (is<CSSSpringTimingFunctionValue>(value)) {
- auto& springTimingFunction = downcast<CSSSpringTimingFunctionValue>(value);
- animation.setTimingFunction(SpringTimingFunction::create(springTimingFunction.mass(), springTimingFunction.stiffness(), springTimingFunction.damping(), springTimingFunction.initialVelocity()));
</del><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSValuecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSValue.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSValue.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/css/CSSValue.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -226,8 +226,6 @@
</span><span class="cx"> return compareCSSValues<CSSCubicBezierTimingFunctionValue>(*this, other);
</span><span class="cx"> case StepsTimingFunctionClass:
</span><span class="cx"> return compareCSSValues<CSSStepsTimingFunctionValue>(*this, other);
</span><del>- case SpringTimingFunctionClass:
- return compareCSSValues<CSSSpringTimingFunctionValue>(*this, other);
</del><span class="cx"> case UnicodeRangeClass:
</span><span class="cx"> return compareCSSValues<CSSUnicodeRangeValue>(*this, other);
</span><span class="cx"> case ValueListClass:
</span><span class="lines">@@ -334,8 +332,6 @@
</span><span class="cx"> return downcast<CSSCubicBezierTimingFunctionValue>(*this).customCSSText();
</span><span class="cx"> case StepsTimingFunctionClass:
</span><span class="cx"> return downcast<CSSStepsTimingFunctionValue>(*this).customCSSText();
</span><del>- case SpringTimingFunctionClass:
- return downcast<CSSSpringTimingFunctionValue>(*this).customCSSText();
</del><span class="cx"> case UnicodeRangeClass:
</span><span class="cx"> return downcast<CSSUnicodeRangeValue>(*this).customCSSText();
</span><span class="cx"> case ValueListClass:
</span><span class="lines">@@ -461,9 +457,6 @@
</span><span class="cx"> case StepsTimingFunctionClass:
</span><span class="cx"> delete downcast<CSSStepsTimingFunctionValue>(this);
</span><span class="cx"> return;
</span><del>- case SpringTimingFunctionClass:
- delete downcast<CSSSpringTimingFunctionValue>(this);
- return;
</del><span class="cx"> case UnicodeRangeClass:
</span><span class="cx"> delete downcast<CSSUnicodeRangeValue>(this);
</span><span class="cx"> return;
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSValueh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSValue.h (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSValue.h 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/css/CSSValue.h 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -104,7 +104,6 @@
</span><span class="cx"> bool isShadowValue() const { return m_classType == ShadowClass; }
</span><span class="cx"> bool isCubicBezierTimingFunctionValue() const { return m_classType == CubicBezierTimingFunctionClass; }
</span><span class="cx"> bool isStepsTimingFunctionValue() const { return m_classType == StepsTimingFunctionClass; }
</span><del>- bool isSpringTimingFunctionValue() const { return m_classType == SpringTimingFunctionClass; }
</del><span class="cx"> bool isWebKitCSSTransformValue() const { return m_classType == WebKitCSSTransformClass; }
</span><span class="cx"> bool isLineBoxContainValue() const { return m_classType == LineBoxContainClass; }
</span><span class="cx"> bool isCalcValue() const {return m_classType == CalculationClass; }
</span><span class="lines">@@ -162,7 +161,6 @@
</span><span class="cx"> // Timing function classes.
</span><span class="cx"> CubicBezierTimingFunctionClass,
</span><span class="cx"> StepsTimingFunctionClass,
</span><del>- SpringTimingFunctionClass,
</del><span class="cx">
</span><span class="cx"> // Other class types.
</span><span class="cx"> AspectRatioClass,
</span></span></pre></div>
<a id="trunkSourceWebCorepageSettingsin"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/Settings.in (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/Settings.in 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/page/Settings.in 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -267,5 +267,3 @@
</span><span class="cx"> selectionPaintingWithoutSelectionGapsEnabled initial=false
</span><span class="cx">
</span><span class="cx"> shouldConvertInvalidURLsToBlank initial=true
</span><del>-
-springTimingFunctionEnabled initial=false
</del></span></pre></div>
<a id="trunkSourceWebCorepageanimationAnimationBasecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/animation/AnimationBase.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/animation/AnimationBase.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/page/animation/AnimationBase.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -41,7 +41,6 @@
</span><span class="cx"> #include "RenderBox.h"
</span><span class="cx"> #include "RenderStyle.h"
</span><span class="cx"> #include "RenderView.h"
</span><del>-#include "SpringSolver.h"
</del><span class="cx"> #include "UnitBezier.h"
</span><span class="cx"> #include <algorithm>
</span><span class="cx"> #include <wtf/CurrentTime.h>
</span><span class="lines">@@ -71,12 +70,6 @@
</span><span class="cx"> return floor(numSteps * t) / numSteps;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-static inline double solveSpringFunction(double mass, double stiffness, double damping, double initialVelocity, double t, double duration)
-{
- SpringSolver solver(mass, stiffness, damping, initialVelocity);
- return solver.solve(t * duration);
-}
-
</del><span class="cx"> AnimationBase::AnimationBase(const Animation& animation, RenderElement* renderer, CompositeAnimation* compositeAnimation)
</span><span class="cx"> : m_object(renderer)
</span><span class="cx"> , m_compositeAnimation(compositeAnimation)
</span><span class="lines">@@ -655,17 +648,13 @@
</span><span class="cx">
</span><span class="cx"> switch (timingFunction->type()) {
</span><span class="cx"> case TimingFunction::CubicBezierFunction: {
</span><del>- auto& function = *static_cast<const CubicBezierTimingFunction*>(timingFunction);
- return solveCubicBezierFunction(function.x1(), function.y1(), function.x2(), function.y2(), fractionalTime, m_animation->duration());
</del><ins>+ const CubicBezierTimingFunction* function = static_cast<const CubicBezierTimingFunction*>(timingFunction);
+ return solveCubicBezierFunction(function->x1(), function->y1(), function->x2(), function->y2(), fractionalTime, m_animation->duration());
</ins><span class="cx"> }
</span><span class="cx"> case TimingFunction::StepsFunction: {
</span><del>- auto& function = *static_cast<const StepsTimingFunction*>(timingFunction);
- return solveStepsFunction(function.numberOfSteps(), function.stepAtStart(), fractionalTime);
</del><ins>+ const StepsTimingFunction* stepsTimingFunction = static_cast<const StepsTimingFunction*>(timingFunction);
+ return solveStepsFunction(stepsTimingFunction->numberOfSteps(), stepsTimingFunction->stepAtStart(), fractionalTime);
</ins><span class="cx"> }
</span><del>- case TimingFunction::SpringFunction: {
- auto& function = *static_cast<const SpringTimingFunction*>(timingFunction);
- return solveSpringFunction(function.mass(), function.stiffness(), function.damping(), function.initialVelocity(), fractionalTime, m_animation->duration());
- }
</del><span class="cx"> case TimingFunction::LinearFunction:
</span><span class="cx"> return fractionalTime;
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformanimationTimingFunctioncpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/animation/TimingFunction.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/animation/TimingFunction.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/platform/animation/TimingFunction.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -37,21 +37,16 @@
</span><span class="cx"> ts << "linear";
</span><span class="cx"> break;
</span><span class="cx"> case TimingFunction::CubicBezierFunction: {
</span><del>- auto& function = static_cast<const CubicBezierTimingFunction&>(timingFunction);
- ts << "cubic-bezier(" << function.x1() << ", " << function.y1() << ", " << function.x2() << ", " << function.y2() << ")";
</del><ins>+ const CubicBezierTimingFunction& cubicBezierFunction = static_cast<const CubicBezierTimingFunction&>(timingFunction);
+ ts << "cubic-bezier(" << cubicBezierFunction.x1() << ", " << cubicBezierFunction.y1() << ", " << cubicBezierFunction.x2() << ", " << cubicBezierFunction.y2() << ")";
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> case TimingFunction::StepsFunction: {
</span><del>- auto& function = static_cast<const StepsTimingFunction&>(timingFunction);
- ts << "steps(" << function.numberOfSteps() << ", " << (function.stepAtStart() ? "start" : "end") << ")";
</del><ins>+ const StepsTimingFunction& stepsFunction = static_cast<const StepsTimingFunction&>(timingFunction);
+ ts << "steps(" << stepsFunction.numberOfSteps() << ", " << (stepsFunction.stepAtStart() ? "start" : "end") << ")";
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case TimingFunction::SpringFunction: {
- auto& function = static_cast<const SpringTimingFunction&>(timingFunction);
- ts << "spring(" << function.mass() << " " << function.stiffness() << " " << function.damping() << " " << function.initialVelocity() << ")";
- break;
</del><span class="cx"> }
</span><del>- }
</del><span class="cx"> return ts;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformanimationTimingFunctionh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/animation/TimingFunction.h (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/animation/TimingFunction.h 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/platform/animation/TimingFunction.h 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -36,7 +36,7 @@
</span><span class="cx"> virtual PassRefPtr<TimingFunction> clone() const = 0;
</span><span class="cx">
</span><span class="cx"> enum TimingFunctionType {
</span><del>- LinearFunction, CubicBezierFunction, StepsFunction, SpringFunction
</del><ins>+ LinearFunction, CubicBezierFunction, StepsFunction
</ins><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> virtual ~TimingFunction() { }
</span><span class="lines">@@ -46,7 +46,6 @@
</span><span class="cx"> bool isLinearTimingFunction() const { return m_type == LinearFunction; }
</span><span class="cx"> bool isCubicBezierTimingFunction() const { return m_type == CubicBezierFunction; }
</span><span class="cx"> bool isStepsTimingFunction() const { return m_type == StepsFunction; }
</span><del>- bool isSpringTimingFunction() const { return m_type == SpringFunction; }
</del><span class="cx">
</span><span class="cx"> virtual bool operator==(const TimingFunction& other) = 0;
</span><span class="cx">
</span><span class="lines">@@ -59,7 +58,7 @@
</span><span class="cx"> TimingFunctionType m_type;
</span><span class="cx"> };
</span><span class="cx">
</span><del>-class LinearTimingFunction final : public TimingFunction {
</del><ins>+class LinearTimingFunction : public TimingFunction {
</ins><span class="cx"> public:
</span><span class="cx"> static PassRefPtr<LinearTimingFunction> create()
</span><span class="cx"> {
</span><span class="lines">@@ -85,7 +84,7 @@
</span><span class="cx"> }
</span><span class="cx"> };
</span><span class="cx">
</span><del>-class CubicBezierTimingFunction final : public TimingFunction {
</del><ins>+class CubicBezierTimingFunction : public TimingFunction {
</ins><span class="cx"> public:
</span><span class="cx"> enum TimingFunctionPreset {
</span><span class="cx"> Ease,
</span><span class="lines">@@ -186,7 +185,7 @@
</span><span class="cx"> TimingFunctionPreset m_timingFunctionPreset;
</span><span class="cx"> };
</span><span class="cx">
</span><del>-class StepsTimingFunction final : public TimingFunction {
</del><ins>+class StepsTimingFunction : public TimingFunction {
</ins><span class="cx"> public:
</span><span class="cx">
</span><span class="cx"> static PassRefPtr<StepsTimingFunction> create(int steps, bool stepAtStart)
</span><span class="lines">@@ -232,64 +231,6 @@
</span><span class="cx"> bool m_stepAtStart;
</span><span class="cx"> };
</span><span class="cx">
</span><del>-class SpringTimingFunction final : public TimingFunction {
-public:
- static Ref<SpringTimingFunction> create(double mass, double stiffness, double damping, double initialVelocity)
- {
- return adoptRef(*new SpringTimingFunction(mass, stiffness, damping, initialVelocity));
- }
-
- static Ref<SpringTimingFunction> create()
- {
- // This create() function should only be used by the argument decoders, and it is expected that
- // real values will be filled in using setValues().
- return create(0, 0, 0, 0);
- }
-
- bool operator==(const TimingFunction& other) override
- {
- if (other.isSpringTimingFunction()) {
- const SpringTimingFunction& otherString = *static_cast<const SpringTimingFunction*>(&other);
-
- return m_mass == otherString.m_mass && m_stiffness == otherString.m_stiffness && m_damping == otherString.m_damping && m_initialVelocity == otherString.m_initialVelocity;
- }
- return false;
- }
-
- double mass() const { return m_mass; }
- double stiffness() const { return m_stiffness; }
- double damping() const { return m_damping; }
- double initialVelocity() const { return m_initialVelocity; }
-
- void setValues(double mass, double stiffness, double damping, double initialVelocity)
- {
- m_mass = mass;
- m_stiffness = stiffness;
- m_damping = damping;
- m_initialVelocity = initialVelocity;
- }
-
-private:
- explicit SpringTimingFunction(double mass, double stiffness, double damping, double initialVelocity)
- : TimingFunction(SpringFunction)
- , m_mass(mass)
- , m_stiffness(stiffness)
- , m_damping(damping)
- , m_initialVelocity(initialVelocity)
- {
- }
-
- PassRefPtr<TimingFunction> clone() const override
- {
- return adoptRef(new SpringTimingFunction(m_mass, m_stiffness, m_damping, m_initialVelocity));
- }
-
- double m_mass;
- double m_stiffness;
- double m_damping;
- double m_initialVelocity;
-};
-
</del><span class="cx"> class TextStream;
</span><span class="cx"> WEBCORE_EXPORT TextStream& operator<<(TextStream&, const TimingFunction&);
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformgraphicsSpringSolverh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebCore/platform/graphics/SpringSolver.h (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/graphics/SpringSolver.h 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/platform/graphics/SpringSolver.h 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,71 +0,0 @@
</span><del>-/*
- * Copyright (C) 2016 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#pragma once
-
-namespace WebCore {
-
-class SpringSolver {
-public:
- SpringSolver(double mass, double stiffness, double damping, double initialVelocity)
- {
- m_w0 = std::sqrt(stiffness / mass);
- m_zeta = damping / (2 * std::sqrt(stiffness * mass));
-
- if (m_zeta < 1) {
- // Under-damped.
- m_wd = m_w0 * std::sqrt(1 - m_zeta * m_zeta);
- m_A = 1;
- m_B = (m_zeta * m_w0 + -initialVelocity) / m_wd;
- } else {
- // Critically damped (ignoring over-damped case for now).
- m_A = 1;
- m_B = -initialVelocity + m_w0;
- }
- }
-
- double solve(double t)
- {
- if (m_zeta < 1) {
- // Under-damped
- t = std::exp(-t * m_zeta * m_w0) * (m_A * std::cos(m_wd * t) + m_B * std::sin(m_wd * t));
- } else {
- // Critically damped (ignoring over-damped case for now).
- t = (m_A + m_B * t) * std::exp(-t * m_w0);
- }
-
- // Map range from [1..0] to [0..1].
- return 1 - t;
- }
-
-private:
- double m_w0;
- double m_zeta;
- double m_wd;
- double m_A;
- double m_B;
-};
-
-} // namespace WebCore
</del></span></pre></div>
<a id="trunkSourceWebCoreplatformgraphicscaGraphicsLayerCAcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -2743,10 +2743,7 @@
</span><span class="cx"> caAnimation = createKeyframeAnimation(animation, propertyIdToString(valueList.property()), additive);
</span><span class="cx"> valuesOK = setAnimationKeyframes(valueList, animation, caAnimation.get());
</span><span class="cx"> } else {
</span><del>- if (animation->timingFunction()->isSpringTimingFunction())
- caAnimation = createSpringAnimation(animation, propertyIdToString(valueList.property()), additive);
- else
- caAnimation = createBasicAnimation(animation, propertyIdToString(valueList.property()), additive);
</del><ins>+ caAnimation = createBasicAnimation(animation, propertyIdToString(valueList.property()), additive);
</ins><span class="cx"> valuesOK = setAnimationEndpoints(valueList, animation, caAnimation.get());
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -2770,10 +2767,7 @@
</span><span class="cx"> caAnimation = createKeyframeAnimation(animation, propertyIdToString(valueList.property()), additive);
</span><span class="cx"> validMatrices = setTransformAnimationKeyframes(valueList, animation, caAnimation.get(), animationIndex, transformOp, isMatrixAnimation, boxSize);
</span><span class="cx"> } else {
</span><del>- if (animation->timingFunction()->isSpringTimingFunction())
- caAnimation = createSpringAnimation(animation, propertyIdToString(valueList.property()), additive);
- else
- caAnimation = createBasicAnimation(animation, propertyIdToString(valueList.property()), additive);
</del><ins>+ caAnimation = createBasicAnimation(animation, propertyIdToString(valueList.property()), additive);
</ins><span class="cx"> validMatrices = setTransformAnimationEndpoints(valueList, animation, caAnimation.get(), animationIndex, transformOp, isMatrixAnimation, boxSize);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -2907,20 +2901,13 @@
</span><span class="cx"> return basicAnim;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-PassRefPtr<PlatformCAAnimation> GraphicsLayerCA::createKeyframeAnimation(const Animation* anim, const String& keyPath, bool additive)
</del><ins>+PassRefPtr<PlatformCAAnimation>GraphicsLayerCA::createKeyframeAnimation(const Animation* anim, const String& keyPath, bool additive)
</ins><span class="cx"> {
</span><span class="cx"> RefPtr<PlatformCAAnimation> keyframeAnim = createPlatformCAAnimation(PlatformCAAnimation::Keyframe, keyPath);
</span><span class="cx"> setupAnimation(keyframeAnim.get(), anim, additive);
</span><span class="cx"> return keyframeAnim;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-RefPtr<PlatformCAAnimation> GraphicsLayerCA::createSpringAnimation(const Animation* anim, const String& keyPath, bool additive)
-{
- auto basicAnim = createPlatformCAAnimation(PlatformCAAnimation::Spring, keyPath);
- setupAnimation(basicAnim.get(), anim, additive);
- return basicAnim;
-}
-
</del><span class="cx"> void GraphicsLayerCA::setupAnimation(PlatformCAAnimation* propertyAnim, const Animation* anim, bool additive)
</span><span class="cx"> {
</span><span class="cx"> double duration = anim->duration();
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformgraphicscaGraphicsLayerCAh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.h (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.h 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.h 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -242,7 +242,6 @@
</span><span class="cx"> // Return autoreleased animation (use RetainPtr?)
</span><span class="cx"> PassRefPtr<PlatformCAAnimation> createBasicAnimation(const Animation*, const String& keyPath, bool additive);
</span><span class="cx"> PassRefPtr<PlatformCAAnimation> createKeyframeAnimation(const Animation*, const String&, bool additive);
</span><del>- RefPtr<PlatformCAAnimation> createSpringAnimation(const Animation*, const String&, bool additive);
</del><span class="cx"> void setupAnimation(PlatformCAAnimation*, const Animation*, bool additive);
</span><span class="cx">
</span><span class="cx"> const TimingFunction* timingFunctionForAnimationValue(const AnimationValue&, const Animation&);
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformgraphicscaPlatformCAAnimationcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/graphics/ca/PlatformCAAnimation.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/graphics/ca/PlatformCAAnimation.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/platform/graphics/ca/PlatformCAAnimation.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -35,7 +35,6 @@
</span><span class="cx"> switch (type) {
</span><span class="cx"> case PlatformCAAnimation::Basic: ts << "basic"; break;
</span><span class="cx"> case PlatformCAAnimation::Keyframe: ts << "keyframe"; break;
</span><del>- case PlatformCAAnimation::Spring: ts << "spring"; break;
</del><span class="cx"> }
</span><span class="cx"> return ts;
</span><span class="cx"> }
</span><span class="lines">@@ -70,9 +69,4 @@
</span><span class="cx"> return ts;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-bool PlatformCAAnimation::isBasicAnimation() const
-{
- return animationType() == Basic || animationType() == Spring;
-}
-
</del><span class="cx"> } // namespace WebCore
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformgraphicscaPlatformCAAnimationh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/graphics/ca/PlatformCAAnimation.h (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/graphics/ca/PlatformCAAnimation.h 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/platform/graphics/ca/PlatformCAAnimation.h 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -41,7 +41,7 @@
</span><span class="cx">
</span><span class="cx"> class PlatformCAAnimation : public RefCounted<PlatformCAAnimation> {
</span><span class="cx"> public:
</span><del>- enum AnimationType { Basic, Keyframe, Spring };
</del><ins>+ enum AnimationType { Basic, Keyframe };
</ins><span class="cx"> enum FillModeType { NoFillMode, Forwards, Backwards, Both };
</span><span class="cx"> enum ValueFunctionType { NoValueFunction, RotateX, RotateY, RotateZ, ScaleX, ScaleY, ScaleZ, Scale, TranslateX, TranslateY, TranslateZ, Translate };
</span><span class="cx">
</span><span class="lines">@@ -123,8 +123,6 @@
</span><span class="cx"> if (beginTime() <= 0)
</span><span class="cx"> setBeginTime(t);
</span><span class="cx"> }
</span><del>-
- bool isBasicAnimation() const;
</del><span class="cx">
</span><span class="cx"> protected:
</span><span class="cx"> PlatformCAAnimation(AnimationType type = Basic)
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformgraphicscacocoaPlatformCAAnimationCocoamm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/graphics/ca/cocoa/PlatformCAAnimationCocoa.mm (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/graphics/ca/cocoa/PlatformCAAnimationCocoa.mm 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebCore/platform/graphics/ca/cocoa/PlatformCAAnimationCocoa.mm 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -161,30 +161,20 @@
</span><span class="cx"> PlatformCAAnimationCocoa::PlatformCAAnimationCocoa(AnimationType type, const String& keyPath)
</span><span class="cx"> : PlatformCAAnimation(type)
</span><span class="cx"> {
</span><del>- switch (type) {
- case Basic:
</del><ins>+ if (type == Basic)
</ins><span class="cx"> m_animation = [CABasicAnimation animationWithKeyPath:keyPath];
</span><del>- break;
- case Keyframe:
</del><ins>+ else
</ins><span class="cx"> m_animation = [CAKeyframeAnimation animationWithKeyPath:keyPath];
</span><del>- break;
- case Spring:
- m_animation = [CASpringAnimation animationWithKeyPath:keyPath];
- break;
- }
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> PlatformCAAnimationCocoa::PlatformCAAnimationCocoa(PlatformAnimationRef animation)
</span><span class="cx"> {
</span><del>- if ([static_cast<CAAnimation*>(animation) isKindOfClass:[CABasicAnimation class]]) {
- if ([static_cast<CAAnimation*>(animation) isKindOfClass:[CASpringAnimation class]])
- setType(Spring);
- else
- setType(Basic);
- } else if ([static_cast<CAAnimation*>(animation) isKindOfClass:[CAKeyframeAnimation class]])
</del><ins>+ if ([static_cast<CAAnimation*>(animation) isKindOfClass:[CABasicAnimation class]])
+ setType(Basic);
+ else if ([static_cast<CAAnimation*>(animation) isKindOfClass:[CAKeyframeAnimation class]])
</ins><span class="cx"> setType(Keyframe);
</span><span class="cx"> else {
</span><del>- ASSERT_NOT_REACHED();
</del><ins>+ ASSERT(0);
</ins><span class="cx"> return;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -315,23 +305,7 @@
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::setTimingFunction(const TimingFunction* value, bool reverse)
</span><span class="cx"> {
</span><del>- switch (animationType()) {
- case Basic:
- case Keyframe:
- [m_animation setTimingFunction:toCAMediaTimingFunction(value, reverse)];
- break;
- case Spring:
- if (value->isSpringTimingFunction()) {
- // FIXME: Handle reverse.
- auto& function = *static_cast<const SpringTimingFunction*>(value);
- CASpringAnimation *springAnimation = (CASpringAnimation *)m_animation.get();
- springAnimation.mass = function.mass();
- springAnimation.stiffness = function.stiffness();
- springAnimation.damping = function.damping();
- springAnimation.initialVelocity = function.initialVelocity();
- }
- break;
- }
</del><ins>+ [m_animation setTimingFunction:toCAMediaTimingFunction(value, reverse)];
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::copyTimingFunctionFrom(const PlatformCAAnimation& value)
</span><span class="lines">@@ -372,14 +346,14 @@
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::setFromValue(float value)
</span><span class="cx"> {
</span><del>- if (!isBasicAnimation())
</del><ins>+ if (animationType() != Basic)
</ins><span class="cx"> return;
</span><span class="cx"> [static_cast<CABasicAnimation*>(m_animation.get()) setFromValue:[NSNumber numberWithDouble:value]];
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::setFromValue(const WebCore::TransformationMatrix& value)
</span><span class="cx"> {
</span><del>- if (!isBasicAnimation())
</del><ins>+ if (animationType() != Basic)
</ins><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> [static_cast<CABasicAnimation*>(m_animation.get()) setFromValue:[NSValue valueWithCATransform3D:value]];
</span><span class="lines">@@ -387,7 +361,7 @@
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::setFromValue(const FloatPoint3D& value)
</span><span class="cx"> {
</span><del>- if (!isBasicAnimation())
</del><ins>+ if (animationType() != Basic)
</ins><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> NSArray* array = [NSArray arrayWithObjects:
</span><span class="lines">@@ -400,7 +374,7 @@
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::setFromValue(const WebCore::Color& value)
</span><span class="cx"> {
</span><del>- if (!isBasicAnimation())
</del><ins>+ if (animationType() != Basic)
</ins><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> NSArray* array = [NSArray arrayWithObjects:
</span><span class="lines">@@ -420,7 +394,7 @@
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::copyFromValueFrom(const PlatformCAAnimation& value)
</span><span class="cx"> {
</span><del>- if (!isBasicAnimation() || !value.isBasicAnimation())
</del><ins>+ if (animationType() != Basic || value.animationType() != Basic)
</ins><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> CABasicAnimation* otherAnimation = static_cast<CABasicAnimation*>(downcast<PlatformCAAnimationCocoa>(value).m_animation.get());
</span><span class="lines">@@ -429,14 +403,14 @@
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::setToValue(float value)
</span><span class="cx"> {
</span><del>- if (!isBasicAnimation())
</del><ins>+ if (animationType() != Basic)
</ins><span class="cx"> return;
</span><span class="cx"> [static_cast<CABasicAnimation*>(m_animation.get()) setToValue:[NSNumber numberWithDouble:value]];
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::setToValue(const WebCore::TransformationMatrix& value)
</span><span class="cx"> {
</span><del>- if (!isBasicAnimation())
</del><ins>+ if (animationType() != Basic)
</ins><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> [static_cast<CABasicAnimation*>(m_animation.get()) setToValue:[NSValue valueWithCATransform3D:value]];
</span><span class="lines">@@ -444,7 +418,7 @@
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::setToValue(const FloatPoint3D& value)
</span><span class="cx"> {
</span><del>- if (!isBasicAnimation())
</del><ins>+ if (animationType() != Basic)
</ins><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> NSArray* array = [NSArray arrayWithObjects:
</span><span class="lines">@@ -457,7 +431,7 @@
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::setToValue(const WebCore::Color& value)
</span><span class="cx"> {
</span><del>- if (!isBasicAnimation())
</del><ins>+ if (animationType() != Basic)
</ins><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> NSArray* array = [NSArray arrayWithObjects:
</span><span class="lines">@@ -477,7 +451,7 @@
</span><span class="cx">
</span><span class="cx"> void PlatformCAAnimationCocoa::copyToValueFrom(const PlatformCAAnimation& value)
</span><span class="cx"> {
</span><del>- if (!isBasicAnimation() || !value.isBasicAnimation())
</del><ins>+ if (animationType() != Basic || value.animationType() != Basic)
</ins><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> CABasicAnimation* otherAnimation = static_cast<CABasicAnimation*>(downcast<PlatformCAAnimationCocoa>(value).m_animation.get());
</span></span></pre></div>
<a id="trunkSourceWebKit2ChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/ChangeLog (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/ChangeLog 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebKit2/ChangeLog 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -1,3 +1,18 @@
</span><ins>+2016-06-06 Commit Queue <commit-queue@webkit.org>
+
+ Unreviewed, rolling out r201706.
+ https://bugs.webkit.org/show_bug.cgi?id=158427
+
+ This change broke the Yosemite build and its LayoutTest is
+ failing on Mac and iOS testers (Requested by ryanhaddad on
+ #webkit).
+
+ Reverted changeset:
+
+ "Add experimental support for spring based CSS animations"
+ https://bugs.webkit.org/show_bug.cgi?id=158403
+ http://trac.webkit.org/changeset/201706
+
</ins><span class="cx"> 2016-06-06 Alex Christensen <achristensen@webkit.org>
</span><span class="cx">
</span><span class="cx"> Reduce ResourceRequest copying in loading code
</span></span></pre></div>
<a id="trunkSourceWebKit2SharedCoordinatedGraphicsCoordinatedGraphicsArgumentCoderscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -365,15 +365,7 @@
</span><span class="cx"> encoder << steps->stepAtStart();
</span><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case TimingFunction::SpringFunction: {
- const SpringTimingFunction* spring = static_cast<const SpringTimingFunction*>(timingFunction);
- encoder << spring->mass();
- encoder << spring->stiffness();
- encoder << spring->damping();
- encoder << spring->initialVelocity();
- break;
</del><span class="cx"> }
</span><del>- }
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool decodeTimingFunction(ArgumentDecoder& decoder, RefPtr<TimingFunction>& timingFunction)
</span><span class="lines">@@ -421,24 +413,7 @@
</span><span class="cx"> timingFunction = StepsTimingFunction::create(numberOfSteps, stepAtStart);
</span><span class="cx"> return true;
</span><span class="cx"> }
</span><del>- case TimingFunction::SpringFunction: {
- double mass;
- if (!decoder.decode(mass))
- return false;
- double stiffness;
- if (!decoder.decode(stiffness))
- return false;
- double damping;
- if (!decoder.decode(damping))
- return false;
- double initialVelocity;
- if (!decoder.decode(initialVelocity))
- return false;
-
- timingFunction = SpringTimingFunction::create(mass, stiffness, damping, initialVelocity);
- return true;
</del><span class="cx"> }
</span><del>- }
</del><span class="cx">
</span><span class="cx"> return false;
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebKit2SharedWebCoreArgumentCoderscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/Shared/WebCoreArgumentCoders.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/Shared/WebCoreArgumentCoders.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebKit2/Shared/WebCoreArgumentCoders.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -265,40 +265,6 @@
</span><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void ArgumentCoder<SpringTimingFunction>::encode(ArgumentEncoder& encoder, const SpringTimingFunction& timingFunction)
-{
- encoder.encodeEnum(timingFunction.type());
-
- encoder << timingFunction.mass();
- encoder << timingFunction.stiffness();
- encoder << timingFunction.damping();
- encoder << timingFunction.initialVelocity();
-}
-
-bool ArgumentCoder<SpringTimingFunction>::decode(ArgumentDecoder& decoder, SpringTimingFunction& timingFunction)
-{
- // Type is decoded by the caller.
- double mass;
- if (!decoder.decode(mass))
- return false;
-
- double stiffness;
- if (!decoder.decode(stiffness))
- return false;
-
- double damping;
- if (!decoder.decode(damping))
- return false;
-
- double initialVelocity;
- if (!decoder.decode(initialVelocity))
- return false;
-
- timingFunction.setValues(mass, stiffness, damping, initialVelocity);
-
- return true;
-}
-
</del><span class="cx"> void ArgumentCoder<FloatPoint>::encode(ArgumentEncoder& encoder, const FloatPoint& floatPoint)
</span><span class="cx"> {
</span><span class="cx"> SimpleArgumentCoder<FloatPoint>::encode(encoder, floatPoint);
</span></span></pre></div>
<a id="trunkSourceWebKit2SharedWebCoreArgumentCodersh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/Shared/WebCoreArgumentCoders.h (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/Shared/WebCoreArgumentCoders.h 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebKit2/Shared/WebCoreArgumentCoders.h 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -60,7 +60,6 @@
</span><span class="cx"> class ResourceRequest;
</span><span class="cx"> class ResourceResponse;
</span><span class="cx"> class SessionID;
</span><del>-class SpringTimingFunction;
</del><span class="cx"> class StepsTimingFunction;
</span><span class="cx"> class StickyPositionViewportConstraints;
</span><span class="cx"> class TextCheckingRequestData;
</span><span class="lines">@@ -152,11 +151,6 @@
</span><span class="cx"> static bool decode(ArgumentDecoder&, WebCore::StepsTimingFunction&);
</span><span class="cx"> };
</span><span class="cx">
</span><del>-template<> struct ArgumentCoder<WebCore::SpringTimingFunction> {
- static void encode(ArgumentEncoder&, const WebCore::SpringTimingFunction&);
- static bool decode(ArgumentDecoder&, WebCore::SpringTimingFunction&);
-};
-
</del><span class="cx"> template<> struct ArgumentCoder<WebCore::CertificateInfo> {
</span><span class="cx"> static void encode(ArgumentEncoder&, const WebCore::CertificateInfo&);
</span><span class="cx"> static bool decode(ArgumentDecoder&, WebCore::CertificateInfo&);
</span></span></pre></div>
<a id="trunkSourceWebKit2SharedWebPreferencesDefinitionsh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/Shared/WebPreferencesDefinitions.h (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/Shared/WebPreferencesDefinitions.h 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebKit2/Shared/WebPreferencesDefinitions.h 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -293,7 +293,6 @@
</span><span class="cx"> macro(CSSGridLayoutEnabled, cssGridLayoutEnabled, Bool, bool, true, "CSS Grid", "CSS Grid Layout Module support") \
</span><span class="cx"> macro(CustomElementsEnabled, customElementsEnabled, Bool, bool, true, "Custom Elements", "HTML Custom Elements prototype") \
</span><span class="cx"> macro(WebGL2Enabled, webGL2Enabled, Bool, bool, true, "WebGL 2.0", "WebGL 2 prototype") \
</span><del>- macro(SpringTimingFunctionEnabled, springTimingFunctionEnabled, Bool, bool, true, "CSS Spring Animations", "CSS Spring Animation prototype") \
</del><span class="cx"> \
</span><span class="cx">
</span><span class="cx"> #if PLATFORM(COCOA)
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessWebPageWebPagecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -3150,8 +3150,6 @@
</span><span class="cx"> RuntimeEnabledFeatures::sharedFeatures().setWebGL2Enabled(store.getBoolValueForKey(WebPreferencesKey::webGL2EnabledKey()));
</span><span class="cx"> #endif
</span><span class="cx">
</span><del>- settings.setSpringTimingFunctionEnabled(store.getBoolValueForKey(WebPreferencesKey::springTimingFunctionEnabledKey()));
-
</del><span class="cx"> bool processSuppressionEnabled = store.getBoolValueForKey(WebPreferencesKey::pageVisibilityBasedProcessSuppressionEnabledKey());
</span><span class="cx"> if (m_processSuppressionEnabled != processSuppressionEnabled) {
</span><span class="cx"> m_processSuppressionEnabled = processSuppressionEnabled;
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessWebPagemacPlatformCAAnimationRemotemm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/WebPage/mac/PlatformCAAnimationRemote.mm (201714 => 201715)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebPage/mac/PlatformCAAnimationRemote.mm 2016-06-06 17:31:28 UTC (rev 201714)
+++ trunk/Source/WebKit2/WebProcess/WebPage/mac/PlatformCAAnimationRemote.mm 2016-06-06 17:56:33 UTC (rev 201715)
</span><span class="lines">@@ -196,10 +196,6 @@
</span><span class="cx"> case TimingFunction::StepsFunction:
</span><span class="cx"> encoder << *static_cast<StepsTimingFunction*>(timingFunction.get());
</span><span class="cx"> break;
</span><del>-
- case TimingFunction::SpringFunction:
- encoder << *static_cast<SpringTimingFunction*>(timingFunction.get());
- break;
</del><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="lines">@@ -286,12 +282,6 @@
</span><span class="cx"> if (!decoder.decode(*static_cast<StepsTimingFunction*>(timingFunction.get())))
</span><span class="cx"> return false;
</span><span class="cx"> break;
</span><del>-
- case TimingFunction::SpringFunction:
- timingFunction = SpringTimingFunction::create();
- if (!decoder.decode(*static_cast<SpringTimingFunction*>(timingFunction.get())))
- return false;
- break;
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> properties.timingFunctions.uncheckedAppend(timingFunction.release());
</span><span class="lines">@@ -764,29 +754,7 @@
</span><span class="cx"> caAnimation = keyframeAnimation;
</span><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case PlatformCAAnimation::Spring: {
- RetainPtr<CASpringAnimation> springAnimation;
- springAnimation = [CASpringAnimation animationWithKeyPath:properties.keyPath];
-
- if (properties.keyValues.size() > 1) {
- [springAnimation setFromValue:animationValueFromKeyframeValue(properties.keyValues[0])];
- [springAnimation setToValue:animationValueFromKeyframeValue(properties.keyValues[1])];
- }
-
- if (properties.timingFunctions.size()) {
- auto& timingFunction = properties.timingFunctions[0];
- if (timingFunction->isSpringTimingFunction()) {
- auto& function = *static_cast<const SpringTimingFunction*>(timingFunction.get());
- [springAnimation setMass:function.mass()];
- [springAnimation setStiffness:function.stiffness()];
- [springAnimation setDamping:function.damping()];
- [springAnimation setInitialVelocity:function.initialVelocity()];
- }
- }
- caAnimation = springAnimation;
- break;
</del><span class="cx"> }
</span><del>- }
</del><span class="cx">
</span><span class="cx"> [caAnimation setBeginTime:properties.beginTime];
</span><span class="cx"> [caAnimation setDuration:properties.duration];
</span></span></pre>
</div>
</div>
</body>
</html>