<!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>[209805] 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/209805">209805</a></dd>
<dt>Author</dt> <dd>hyatt@apple.com</dd>
<dt>Date</dt> <dd>2016-12-14 08:34:48 -0800 (Wed, 14 Dec 2016)</dd>
</dl>
<h3>Log Message</h3>
<pre>[CSS Parser] Remove WebkitCSSTransformValue
https://bugs.webkit.org/show_bug.cgi?id=165844
Reviewed by Zalan Bujtas.
Source/WebCore:
Remove WebkitCSSTransformValue and just use a CSSFunctionValue instead. No other
browsers support it, and the information it provides is easily obtainable from cssText.
* CMakeLists.txt:
* DerivedSources.cpp:
* DerivedSources.make:
* WebCore.xcodeproj/project.pbxproj:
* bindings/js/JSCSSValueCustom.cpp:
(WebCore::toJSNewlyCreated):
* css/CSSComputedStyleDeclaration.cpp:
(WebCore::matrixTransformValue):
* css/CSSValue.cpp:
(WebCore::CSSValue::equals):
(WebCore::CSSValue::cssText):
(WebCore::CSSValue::destroy):
(WebCore::CSSValue::cloneForCSSOM):
* css/CSSValue.h:
(WebCore::CSSValue::isSpringTimingFunctionValue):
(WebCore::CSSValue::isWebKitCSSTransformValue): Deleted.
* css/StyleResolver.cpp:
* css/TransformFunctions.cpp:
(WebCore::transformOperationType):
(WebCore::transformsForValue):
* css/TransformFunctions.h:
* css/WebKitCSSTransformValue.cpp: Removed.
* css/WebKitCSSTransformValue.h: Removed.
* css/WebKitCSSTransformValue.idl: Removed.
* css/parser/CSSParserFastPaths.cpp:
(WebCore::parseTransformTranslateArguments):
(WebCore::parseTransformNumberArguments):
(WebCore::parseSimpleTransformValue):
(WebCore::parseSimpleTransformList):
* css/parser/CSSPropertyParser.cpp:
(WebCore::consumeTranslate3d):
(WebCore::consumeNumbers):
(WebCore::consumePerspective):
(WebCore::consumeTransformValue):
(WebCore::transformOperationForCSSValueID): Deleted.
LayoutTests:
* fast/css/getFloatValueForUnit-expected.txt:
* fast/css/getFloatValueForUnit.html:
* platform/mac-wk1/js/dom/global-constructors-attributes-expected.txt:
* platform/mac-yosemite/js/dom/global-constructors-attributes-expected.txt:
* platform/mac/fast/dom/wrapper-classes-objc-expected.txt:
* platform/mac/fast/dom/wrapper-classes-objc.html:
* platform/mac/js/dom/global-constructors-attributes-expected.txt:
* transforms/2d/transform-value-types-expected.txt:
* transforms/2d/transform-value-types.html:</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsChangeLog">trunk/LayoutTests/ChangeLog</a></li>
<li><a href="#trunkLayoutTestsfastcssgetFloatValueForUnitexpectedtxt">trunk/LayoutTests/fast/css/getFloatValueForUnit-expected.txt</a></li>
<li><a href="#trunkLayoutTestsfastcssgetFloatValueForUnithtml">trunk/LayoutTests/fast/css/getFloatValueForUnit.html</a></li>
<li><a href="#trunkLayoutTestsplatformmacfastdomwrapperclassesobjcexpectedtxt">trunk/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc-expected.txt</a></li>
<li><a href="#trunkLayoutTestsplatformmacfastdomwrapperclassesobjchtml">trunk/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc.html</a></li>
<li><a href="#trunkLayoutTestsplatformmacjsdomglobalconstructorsattributesexpectedtxt">trunk/LayoutTests/platform/mac/js/dom/global-constructors-attributes-expected.txt</a></li>
<li><a href="#trunkLayoutTestsplatformmacwk1jsdomglobalconstructorsattributesexpectedtxt">trunk/LayoutTests/platform/mac-wk1/js/dom/global-constructors-attributes-expected.txt</a></li>
<li><a href="#trunkLayoutTestsplatformmacyosemitejsdomglobalconstructorsattributesexpectedtxt">trunk/LayoutTests/platform/mac-yosemite/js/dom/global-constructors-attributes-expected.txt</a></li>
<li><a href="#trunkLayoutTeststransforms2dtransformvaluetypesexpectedtxt">trunk/LayoutTests/transforms/2d/transform-value-types-expected.txt</a></li>
<li><a href="#trunkLayoutTeststransforms2dtransformvaluetypeshtml">trunk/LayoutTests/transforms/2d/transform-value-types.html</a></li>
<li><a href="#trunkSourceWebCoreCMakeListstxt">trunk/Source/WebCore/CMakeLists.txt</a></li>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreDerivedSourcescpp">trunk/Source/WebCore/DerivedSources.cpp</a></li>
<li><a href="#trunkSourceWebCoreDerivedSourcesmake">trunk/Source/WebCore/DerivedSources.make</a></li>
<li><a href="#trunkSourceWebCoreWebCorexcodeprojprojectpbxproj">trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj</a></li>
<li><a href="#trunkSourceWebCorebindingsjsJSCSSValueCustomcpp">trunk/Source/WebCore/bindings/js/JSCSSValueCustom.cpp</a></li>
<li><a href="#trunkSourceWebCorecssCSSComputedStyleDeclarationcpp">trunk/Source/WebCore/css/CSSComputedStyleDeclaration.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="#trunkSourceWebCorecssStyleResolvercpp">trunk/Source/WebCore/css/StyleResolver.cpp</a></li>
<li><a href="#trunkSourceWebCorecssTransformFunctionscpp">trunk/Source/WebCore/css/TransformFunctions.cpp</a></li>
<li><a href="#trunkSourceWebCorecssTransformFunctionsh">trunk/Source/WebCore/css/TransformFunctions.h</a></li>
<li><a href="#trunkSourceWebCorecssparserCSSParserFastPathscpp">trunk/Source/WebCore/css/parser/CSSParserFastPaths.cpp</a></li>
<li><a href="#trunkSourceWebCorecssparserCSSPropertyParsercpp">trunk/Source/WebCore/css/parser/CSSPropertyParser.cpp</a></li>
</ul>
<h3>Removed Paths</h3>
<ul>
<li><a href="#trunkSourceWebCorecssWebKitCSSTransformValuecpp">trunk/Source/WebCore/css/WebKitCSSTransformValue.cpp</a></li>
<li><a href="#trunkSourceWebCorecssWebKitCSSTransformValueh">trunk/Source/WebCore/css/WebKitCSSTransformValue.h</a></li>
<li><a href="#trunkSourceWebCorecssWebKitCSSTransformValueidl">trunk/Source/WebCore/css/WebKitCSSTransformValue.idl</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkLayoutTestsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/ChangeLog (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/ChangeLog 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/LayoutTests/ChangeLog 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -1,3 +1,20 @@
</span><ins>+2016-12-14 Dave Hyatt <hyatt@apple.com>
+
+ [CSS Parser] Remove WebkitCSSTransformValue
+ https://bugs.webkit.org/show_bug.cgi?id=165844
+
+ Reviewed by Zalan Bujtas.
+
+ * fast/css/getFloatValueForUnit-expected.txt:
+ * fast/css/getFloatValueForUnit.html:
+ * platform/mac-wk1/js/dom/global-constructors-attributes-expected.txt:
+ * platform/mac-yosemite/js/dom/global-constructors-attributes-expected.txt:
+ * platform/mac/fast/dom/wrapper-classes-objc-expected.txt:
+ * platform/mac/fast/dom/wrapper-classes-objc.html:
+ * platform/mac/js/dom/global-constructors-attributes-expected.txt:
+ * transforms/2d/transform-value-types-expected.txt:
+ * transforms/2d/transform-value-types.html:
+
</ins><span class="cx"> 2016-12-14 Yusuke Suzuki <utatane.tea@gmail.com>
</span><span class="cx">
</span><span class="cx"> Unreviewed, LayoutTest http/tests/misc/module-script-async.html is a flaky timeout
</span></span></pre></div>
<a id="trunkLayoutTestsfastcssgetFloatValueForUnitexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/fast/css/getFloatValueForUnit-expected.txt (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/css/getFloatValueForUnit-expected.txt 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/LayoutTests/fast/css/getFloatValueForUnit-expected.txt 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -10,10 +10,6 @@
</span><span class="cx"> PASS getFloatValue('font-size', CSSPrimitiveValue.CSS_PC) is 0.833333
</span><span class="cx"> PASS getFloatValue('font-size', CSSPrimitiveValue.CSS_PX) is 13.333333
</span><span class="cx"> PASS getFloatValue('font-size', CSSPrimitiveValue.CSS_NUMBER) is 13.333333
</span><del>-PASS getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_DEG) is 90
-PASS getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_RAD) is 1.570796
-PASS getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_GRAD) is 100
-PASS getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_NUMBER) is 90
</del><span class="cx"> PASS getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_MS) is 200
</span><span class="cx"> PASS getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_S) is 0.2
</span><span class="cx"> PASS getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_NUMBER) is 200
</span><span class="lines">@@ -22,11 +18,6 @@
</span><span class="cx"> PASS getFloatValue('font-size', CSSPrimitiveValue.CSS_HZ) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
</span><span class="cx"> PASS getFloatValue('font-size', CSSPrimitiveValue.CSS_DEG) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
</span><span class="cx"> PASS getFloatValue('font-size', CSSPrimitiveValue.CSS_RGBCOLOR) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
</span><del>-PASS getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_PERCENTAGE) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
-PASS getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_PX) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
-PASS getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_MS) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
-PASS getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_HZ) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
-PASS getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_RGBCOLOR) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
</del><span class="cx"> PASS getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_PERCENTAGE) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
</span><span class="cx"> PASS getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_PX) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
</span><span class="cx"> PASS getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_DEG) threw exception InvalidAccessError (DOM Exception 15): The object does not support the operation or argument..
</span></span></pre></div>
<a id="trunkLayoutTestsfastcssgetFloatValueForUnithtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/fast/css/getFloatValueForUnit.html (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/css/getFloatValueForUnit.html 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/LayoutTests/fast/css/getFloatValueForUnit.html 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -32,10 +32,6 @@
</span><span class="cx"> shouldBe("getFloatValue('font-size', CSSPrimitiveValue.CSS_PC)", "0.833333");
</span><span class="cx"> shouldBe("getFloatValue('font-size', CSSPrimitiveValue.CSS_PX)", "13.333333");
</span><span class="cx"> shouldBe("getFloatValue('font-size', CSSPrimitiveValue.CSS_NUMBER)", "13.333333");
</span><del>-shouldBe("getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_DEG)", "90");
-shouldBe("getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_RAD)", "1.570796");
-shouldBe("getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_GRAD)", "100");
-shouldBe("getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_NUMBER)", "90");
</del><span class="cx"> shouldBe("getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_MS)", "200");
</span><span class="cx"> shouldBe("getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_S)", "0.2");
</span><span class="cx"> shouldBe("getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_NUMBER)", "200");
</span><span class="lines">@@ -46,12 +42,6 @@
</span><span class="cx"> shouldThrow("getFloatValue('font-size', CSSPrimitiveValue.CSS_DEG)", "invalidAccessError");
</span><span class="cx"> shouldThrow("getFloatValue('font-size', CSSPrimitiveValue.CSS_RGBCOLOR)", "invalidAccessError");
</span><span class="cx">
</span><del>-shouldThrow("getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_PERCENTAGE)", "invalidAccessError");
-shouldThrow("getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_PX)", "invalidAccessError");
-shouldThrow("getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_MS)", "invalidAccessError");
-shouldThrow("getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_HZ)", "invalidAccessError");
-shouldThrow("getFirstFloatValueInList('-webkit-transform', CSSPrimitiveValue.CSS_RGBCOLOR)", "invalidAccessError");
-
</del><span class="cx"> shouldThrow("getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_PERCENTAGE)", "invalidAccessError");
</span><span class="cx"> shouldThrow("getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_PX)", "invalidAccessError");
</span><span class="cx"> shouldThrow("getFloatValue('-webkit-animation-delay', CSSPrimitiveValue.CSS_DEG)", "invalidAccessError");
</span></span></pre></div>
<a id="trunkLayoutTestsplatformmacfastdomwrapperclassesobjcexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc-expected.txt (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc-expected.txt 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc-expected.txt 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -46,8 +46,6 @@
</span><span class="cx"> PASS objCWrapperClass(stylesheet.cssRules.item(0).style.getPropertyCSSValue('clip').getRectValue()) is 'DOMRect'
</span><span class="cx"> PASS objCWrapperClass(stylesheet.cssRules.item(0).style.getPropertyCSSValue('content')) is 'DOMCSSValueList'
</span><span class="cx"> PASS objCWrapperClass(stylesheet.cssRules.item(0).style.getPropertyCSSValue('content').item(0).getCounterValue()) is 'DOMCounter'
</span><del>-PASS objCWrapperClass(stylesheet.cssRules.item(0).style.getPropertyCSSValue('-webkit-transform')) is 'DOMCSSValueList'
-PASS objCWrapperClass(stylesheet.cssRules.item(0).style.getPropertyCSSValue('-webkit-transform').item(0)) is 'DOMCSSValueList'
</del><span class="cx"> PASS objCWrapperClass(stylesheet.media) is 'DOMMediaList'
</span><span class="cx">
</span><span class="cx"> XPath
</span></span></pre></div>
<a id="trunkLayoutTestsplatformmacfastdomwrapperclassesobjchtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc.html (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc.html 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc.html 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -6,7 +6,6 @@
</span><span class="cx"> color: rgb(0, 0, 0);
</span><span class="cx"> clip: rect(0, 0, 1, 1);
</span><span class="cx"> content: counter(dummy, square);
</span><del>- -webkit-transform: translate(10px, 20px);
</del><span class="cx"> }
</span><span class="cx"> </style>
</span><span class="cx"> </head>
</span><span class="lines">@@ -132,8 +131,6 @@
</span><span class="cx"> test("stylesheet.cssRules.item(0).style.getPropertyCSSValue('clip').getRectValue()", "Rect");
</span><span class="cx"> test("stylesheet.cssRules.item(0).style.getPropertyCSSValue('content')", "CSSValueList");
</span><span class="cx"> test("stylesheet.cssRules.item(0).style.getPropertyCSSValue('content').item(0).getCounterValue()", "Counter");
</span><del>- test("stylesheet.cssRules.item(0).style.getPropertyCSSValue('-webkit-transform')", "CSSValueList");
- test("stylesheet.cssRules.item(0).style.getPropertyCSSValue('-webkit-transform').item(0)", "CSSValueList");
</del><span class="cx"> test("stylesheet.media", "MediaList");
</span><span class="cx">
</span><span class="cx"> debug('');
</span></span></pre></div>
<a id="trunkLayoutTestsplatformmacjsdomglobalconstructorsattributesexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/platform/mac/js/dom/global-constructors-attributes-expected.txt (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/platform/mac/js/dom/global-constructors-attributes-expected.txt 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/LayoutTests/platform/mac/js/dom/global-constructors-attributes-expected.txt 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -2393,11 +2393,6 @@
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSRegionRule').hasOwnProperty('set') is false
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSRegionRule').enumerable is false
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSRegionRule').configurable is true
</span><del>-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').value is WebKitCSSTransformValue
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').enumerable is false
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').configurable is true
</del><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitDataCue').value is WebKitDataCue
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitDataCue').hasOwnProperty('get') is false
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitDataCue').hasOwnProperty('set') is false
</span></span></pre></div>
<a id="trunkLayoutTestsplatformmacwk1jsdomglobalconstructorsattributesexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/platform/mac-wk1/js/dom/global-constructors-attributes-expected.txt (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/platform/mac-wk1/js/dom/global-constructors-attributes-expected.txt 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/LayoutTests/platform/mac-wk1/js/dom/global-constructors-attributes-expected.txt 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -2363,11 +2363,6 @@
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSRegionRule').hasOwnProperty('set') is false
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSRegionRule').enumerable is false
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSRegionRule').configurable is true
</span><del>-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').value is WebKitCSSTransformValue
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').enumerable is false
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').configurable is true
</del><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitDataCue').value is WebKitDataCue
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitDataCue').hasOwnProperty('get') is false
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitDataCue').hasOwnProperty('set') is false
</span></span></pre></div>
<a id="trunkLayoutTestsplatformmacyosemitejsdomglobalconstructorsattributesexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/platform/mac-yosemite/js/dom/global-constructors-attributes-expected.txt (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/platform/mac-yosemite/js/dom/global-constructors-attributes-expected.txt 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/LayoutTests/platform/mac-yosemite/js/dom/global-constructors-attributes-expected.txt 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -2388,11 +2388,6 @@
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSRegionRule').hasOwnProperty('set') is false
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSRegionRule').enumerable is false
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSRegionRule').configurable is true
</span><del>-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').value is WebKitCSSTransformValue
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').enumerable is false
-PASS Object.getOwnPropertyDescriptor(global, 'WebKitCSSTransformValue').configurable is true
</del><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitDataCue').value is WebKitDataCue
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitDataCue').hasOwnProperty('get') is false
</span><span class="cx"> PASS Object.getOwnPropertyDescriptor(global, 'WebKitDataCue').hasOwnProperty('set') is false
</span></span></pre></div>
<a id="trunkLayoutTeststransforms2dtransformvaluetypesexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/transforms/2d/transform-value-types-expected.txt (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/transforms/2d/transform-value-types-expected.txt 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/LayoutTests/transforms/2d/transform-value-types-expected.txt 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -1,36 +1,22 @@
</span><span class="cx"> PASS jsWrapperClass(transformRule) is 'CSSValueList'
</span><span class="cx"> PASS jsWrapperClass(transformRule.__proto__) is 'CSSValueListPrototype'
</span><span class="cx"> PASS jsWrapperClass(transformRule.constructor) is 'Function'
</span><del>-PASS jsWrapperClass(transformRule[0]) is 'WebKitCSSTransformValue'
-PASS jsWrapperClass(transformRule[0].__proto__) is 'WebKitCSSTransformValuePrototype'
</del><ins>+PASS jsWrapperClass(transformRule[0]) is 'CSSValue'
+PASS jsWrapperClass(transformRule[0].__proto__) is 'CSSValuePrototype'
</ins><span class="cx"> PASS jsWrapperClass(transformRule[0].constructor) is 'Function'
</span><del>-PASS transformRule[0].operationType is WebKitCSSTransformValue.CSS_TRANSLATE
</del><span class="cx"> PASS transformRule[0].cssText is 'translate(10px)'
</span><del>-PASS transformRule[1].operationType is WebKitCSSTransformValue.CSS_TRANSLATEX
-PASS transformRule[1].cssText is 'translateX(10px)'
-PASS transformRule[2].operationType is WebKitCSSTransformValue.CSS_TRANSLATEY
-PASS transformRule[2].cssText is 'translateY(10px)'
-PASS transformRule[3].operationType is WebKitCSSTransformValue.CSS_ROTATE
</del><ins>+PASS transformRule[1].cssText is 'translatex(10px)'
+PASS transformRule[2].cssText is 'translatey(10px)'
</ins><span class="cx"> PASS transformRule[3].cssText is 'rotate(10deg)'
</span><del>-PASS transformRule[4].operationType is WebKitCSSTransformValue.CSS_ROTATE
</del><span class="cx"> PASS transformRule[4].cssText is 'rotate(11rad)'
</span><del>-PASS transformRule[5].operationType is WebKitCSSTransformValue.CSS_ROTATE
</del><span class="cx"> PASS transformRule[5].cssText is 'rotate(12grad)'
</span><del>-PASS transformRule[6].operationType is WebKitCSSTransformValue.CSS_ROTATE
</del><span class="cx"> PASS transformRule[6].cssText is 'rotate(13turn)'
</span><del>-PASS transformRule[7].operationType is WebKitCSSTransformValue.CSS_SCALE
</del><span class="cx"> PASS transformRule[7].cssText is 'scale(2)'
</span><del>-PASS transformRule[8].operationType is WebKitCSSTransformValue.CSS_SCALEX
-PASS transformRule[8].cssText is 'scaleX(1.2)'
-PASS transformRule[9].operationType is WebKitCSSTransformValue.CSS_SCALEY
-PASS transformRule[9].cssText is 'scaleY(1.2)'
-PASS transformRule[10].operationType is WebKitCSSTransformValue.CSS_SKEW
</del><ins>+PASS transformRule[8].cssText is 'scalex(1.2)'
+PASS transformRule[9].cssText is 'scaley(1.2)'
</ins><span class="cx"> PASS transformRule[10].cssText is 'skew(10deg, 10rad)'
</span><del>-PASS transformRule[11].operationType is WebKitCSSTransformValue.CSS_SKEWX
-PASS transformRule[11].cssText is 'skewX(10deg)'
-PASS transformRule[12].operationType is WebKitCSSTransformValue.CSS_SKEWY
-PASS transformRule[12].cssText is 'skewY(11grad)'
-PASS transformRule[13].operationType is WebKitCSSTransformValue.CSS_MATRIX
</del><ins>+PASS transformRule[11].cssText is 'skewx(10deg)'
+PASS transformRule[12].cssText is 'skewy(11grad)'
</ins><span class="cx"> PASS transformRule[13].cssText is 'matrix(1, 0, 0, 1, 0, 0)'
</span><span class="cx"> PASS successfullyParsed is true
</span><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTeststransforms2dtransformvaluetypeshtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/transforms/2d/transform-value-types.html (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/transforms/2d/transform-value-types.html 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/LayoutTests/transforms/2d/transform-value-types.html 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -51,48 +51,34 @@
</span><span class="cx"> var transformRule = stylesheet.cssRules.item(0).style.getPropertyCSSValue('-webkit-transform');
</span><span class="cx"> test("transformRule", "CSSValueList");
</span><span class="cx">
</span><del>- test("transformRule[0]", "WebKitCSSTransformValue");
</del><ins>+ test("transformRule[0]", "CSSValue");
</ins><span class="cx">
</span><del>- shouldBe("transformRule[0].operationType", "WebKitCSSTransformValue.CSS_TRANSLATE");
</del><span class="cx"> shouldBe("transformRule[0].cssText", "'translate(10px)'");
</span><span class="cx">
</span><del>- shouldBe("transformRule[1].operationType", "WebKitCSSTransformValue.CSS_TRANSLATEX");
- shouldBe("transformRule[1].cssText", "'translateX(10px)'");
</del><ins>+ shouldBe("transformRule[1].cssText", "'translatex(10px)'");
</ins><span class="cx">
</span><del>- shouldBe("transformRule[2].operationType", "WebKitCSSTransformValue.CSS_TRANSLATEY");
- shouldBe("transformRule[2].cssText", "'translateY(10px)'");
</del><ins>+ shouldBe("transformRule[2].cssText", "'translatey(10px)'");
</ins><span class="cx">
</span><del>- shouldBe("transformRule[3].operationType", "WebKitCSSTransformValue.CSS_ROTATE");
</del><span class="cx"> shouldBe("transformRule[3].cssText", "'rotate(10deg)'");
</span><span class="cx">
</span><del>- shouldBe("transformRule[4].operationType", "WebKitCSSTransformValue.CSS_ROTATE");
</del><span class="cx"> shouldBe("transformRule[4].cssText", "'rotate(11rad)'");
</span><span class="cx">
</span><del>- shouldBe("transformRule[5].operationType", "WebKitCSSTransformValue.CSS_ROTATE");
</del><span class="cx"> shouldBe("transformRule[5].cssText", "'rotate(12grad)'");
</span><span class="cx">
</span><del>- shouldBe("transformRule[6].operationType", "WebKitCSSTransformValue.CSS_ROTATE");
</del><span class="cx"> shouldBe("transformRule[6].cssText", "'rotate(13turn)'");
</span><span class="cx">
</span><del>- shouldBe("transformRule[7].operationType", "WebKitCSSTransformValue.CSS_SCALE");
</del><span class="cx"> shouldBe("transformRule[7].cssText", "'scale(2)'");
</span><span class="cx">
</span><del>- shouldBe("transformRule[8].operationType", "WebKitCSSTransformValue.CSS_SCALEX");
- shouldBe("transformRule[8].cssText", "'scaleX(1.2)'");
</del><ins>+ shouldBe("transformRule[8].cssText", "'scalex(1.2)'");
</ins><span class="cx">
</span><del>- shouldBe("transformRule[9].operationType", "WebKitCSSTransformValue.CSS_SCALEY");
- shouldBe("transformRule[9].cssText", "'scaleY(1.2)'");
</del><ins>+ shouldBe("transformRule[9].cssText", "'scaley(1.2)'");
</ins><span class="cx">
</span><del>- shouldBe("transformRule[10].operationType", "WebKitCSSTransformValue.CSS_SKEW");
</del><span class="cx"> shouldBe("transformRule[10].cssText", "'skew(10deg, 10rad)'");
</span><span class="cx">
</span><del>- shouldBe("transformRule[11].operationType", "WebKitCSSTransformValue.CSS_SKEWX");
- shouldBe("transformRule[11].cssText", "'skewX(10deg)'");
</del><ins>+ shouldBe("transformRule[11].cssText", "'skewx(10deg)'");
</ins><span class="cx">
</span><del>- shouldBe("transformRule[12].operationType", "WebKitCSSTransformValue.CSS_SKEWY");
- shouldBe("transformRule[12].cssText", "'skewY(11grad)'");
</del><ins>+ shouldBe("transformRule[12].cssText", "'skewy(11grad)'");
</ins><span class="cx">
</span><del>- shouldBe("transformRule[13].operationType", "WebKitCSSTransformValue.CSS_MATRIX");
</del><span class="cx"> shouldBe("transformRule[13].cssText", "'matrix(1, 0, 0, 1, 0, 0)'");
</span><span class="cx">
</span><span class="cx"> </script>
</span></span></pre></div>
<a id="trunkSourceWebCoreCMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/CMakeLists.txt (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/CMakeLists.txt 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/CMakeLists.txt 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -371,7 +371,6 @@
</span><span class="cx"> css/StyleSheetList.idl
</span><span class="cx"> css/WebKitCSSMatrix.idl
</span><span class="cx"> css/WebKitCSSRegionRule.idl
</span><del>- css/WebKitCSSTransformValue.idl
</del><span class="cx"> css/WebKitCSSViewportRule.idl
</span><span class="cx">
</span><span class="cx"> dom/AnimationEvent.idl
</span><span class="lines">@@ -1366,7 +1365,6 @@
</span><span class="cx"> css/ViewportStyleResolver.cpp
</span><span class="cx"> css/WebKitCSSMatrix.cpp
</span><span class="cx"> css/WebKitCSSRegionRule.cpp
</span><del>- css/WebKitCSSTransformValue.cpp
</del><span class="cx"> css/WebKitCSSViewportRule.cpp
</span><span class="cx">
</span><span class="cx"> css/parser/CSSAtRuleID.cpp
</span></span></pre></div>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/ChangeLog 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -1,3 +1,49 @@
</span><ins>+2016-12-14 Dave Hyatt <hyatt@apple.com>
+
+ [CSS Parser] Remove WebkitCSSTransformValue
+ https://bugs.webkit.org/show_bug.cgi?id=165844
+
+ Reviewed by Zalan Bujtas.
+
+ Remove WebkitCSSTransformValue and just use a CSSFunctionValue instead. No other
+ browsers support it, and the information it provides is easily obtainable from cssText.
+
+ * CMakeLists.txt:
+ * DerivedSources.cpp:
+ * DerivedSources.make:
+ * WebCore.xcodeproj/project.pbxproj:
+ * bindings/js/JSCSSValueCustom.cpp:
+ (WebCore::toJSNewlyCreated):
+ * css/CSSComputedStyleDeclaration.cpp:
+ (WebCore::matrixTransformValue):
+ * css/CSSValue.cpp:
+ (WebCore::CSSValue::equals):
+ (WebCore::CSSValue::cssText):
+ (WebCore::CSSValue::destroy):
+ (WebCore::CSSValue::cloneForCSSOM):
+ * css/CSSValue.h:
+ (WebCore::CSSValue::isSpringTimingFunctionValue):
+ (WebCore::CSSValue::isWebKitCSSTransformValue): Deleted.
+ * css/StyleResolver.cpp:
+ * css/TransformFunctions.cpp:
+ (WebCore::transformOperationType):
+ (WebCore::transformsForValue):
+ * css/TransformFunctions.h:
+ * css/WebKitCSSTransformValue.cpp: Removed.
+ * css/WebKitCSSTransformValue.h: Removed.
+ * css/WebKitCSSTransformValue.idl: Removed.
+ * css/parser/CSSParserFastPaths.cpp:
+ (WebCore::parseTransformTranslateArguments):
+ (WebCore::parseTransformNumberArguments):
+ (WebCore::parseSimpleTransformValue):
+ (WebCore::parseSimpleTransformList):
+ * css/parser/CSSPropertyParser.cpp:
+ (WebCore::consumeTranslate3d):
+ (WebCore::consumeNumbers):
+ (WebCore::consumePerspective):
+ (WebCore::consumeTransformValue):
+ (WebCore::transformOperationForCSSValueID): Deleted.
+
</ins><span class="cx"> 2016-12-14 Antti Koivisto <antti@apple.com>
</span><span class="cx">
</span><span class="cx"> Make Style::Update const in RenderTreeUpdater
</span></span></pre></div>
<a id="trunkSourceWebCoreDerivedSourcescpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/DerivedSources.cpp (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/DerivedSources.cpp 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/DerivedSources.cpp 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -642,7 +642,6 @@
</span><span class="cx"> #include "JSVoidCallback.cpp"
</span><span class="cx"> #include "JSWaveShaperNode.cpp"
</span><span class="cx"> #include "JSWebKitAnimationEvent.cpp"
</span><del>-#include "JSWebKitCSSTransformValue.cpp"
</del><span class="cx"> #include "JSWebKitCSSMatrix.cpp"
</span><span class="cx"> #include "JSWebKitCSSRegionRule.cpp"
</span><span class="cx"> #include "JSWebKitCSSViewportRule.cpp"
</span></span></pre></div>
<a id="trunkSourceWebCoreDerivedSourcesmake"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/DerivedSources.make (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/DerivedSources.make 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/DerivedSources.make 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -296,7 +296,6 @@
</span><span class="cx"> $(WebCore)/css/StyleSheetList.idl \
</span><span class="cx"> $(WebCore)/css/WebKitCSSMatrix.idl \
</span><span class="cx"> $(WebCore)/css/WebKitCSSRegionRule.idl \
</span><del>- $(WebCore)/css/WebKitCSSTransformValue.idl \
</del><span class="cx"> $(WebCore)/css/WebKitCSSViewportRule.idl \
</span><span class="cx"> $(WebCore)/dom/AnimationEvent.idl \
</span><span class="cx"> $(WebCore)/dom/Attr.idl \
</span></span></pre></div>
<a id="trunkSourceWebCoreWebCorexcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -1361,8 +1361,6 @@
</span><span class="cx"> 314BE3A61B3103FB00141982 /* NamedImageGeneratedImage.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 314BE3A41B3103FB00141982 /* NamedImageGeneratedImage.cpp */; };
</span><span class="cx"> 314BE3A71B3103FB00141982 /* NamedImageGeneratedImage.h in Headers */ = {isa = PBXBuildFile; fileRef = 314BE3A51B3103FB00141982 /* NamedImageGeneratedImage.h */; };
</span><span class="cx"> 316023F01532C40C00D50FF4 /* Dictionary.h in Headers */ = {isa = PBXBuildFile; fileRef = 316023EF1532C40C00D50FF4 /* Dictionary.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><del>- 31611E5A0E1C4DE000F6A579 /* JSWebKitCSSTransformValue.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 31611E580E1C4DE000F6A579 /* JSWebKitCSSTransformValue.cpp */; };
- 31611E5B0E1C4DE000F6A579 /* JSWebKitCSSTransformValue.h in Headers */ = {isa = PBXBuildFile; fileRef = 31611E590E1C4DE000F6A579 /* JSWebKitCSSTransformValue.h */; };
</del><span class="cx"> 316FE0710E6CCBEE00BF6088 /* JSCSSKeyframeRule.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 316FE06D0E6CCBEE00BF6088 /* JSCSSKeyframeRule.cpp */; };
</span><span class="cx"> 316FE0720E6CCBEE00BF6088 /* JSCSSKeyframeRule.h in Headers */ = {isa = PBXBuildFile; fileRef = 316FE06E0E6CCBEE00BF6088 /* JSCSSKeyframeRule.h */; };
</span><span class="cx"> 316FE0730E6CCBEE00BF6088 /* JSCSSKeyframesRule.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 316FE06F0E6CCBEE00BF6088 /* JSCSSKeyframesRule.cpp */; };
</span><span class="lines">@@ -5382,8 +5380,6 @@
</span><span class="cx"> BC97E413109154FA0010D361 /* JSHTMLAllCollection.h in Headers */ = {isa = PBXBuildFile; fileRef = BC97E411109154FA0010D361 /* JSHTMLAllCollection.h */; };
</span><span class="cx"> BC97E42C10915B060010D361 /* JSHTMLAllCollectionCustom.cpp in Sources */ = {isa = PBXBuildFile; fileRef = BC97E42B10915B060010D361 /* JSHTMLAllCollectionCustom.cpp */; };
</span><span class="cx"> BC98A27D0C0C9950004BEBF7 /* JSStyleSheetCustom.cpp in Sources */ = {isa = PBXBuildFile; fileRef = BC98A27C0C0C9950004BEBF7 /* JSStyleSheetCustom.cpp */; };
</span><del>- BC9ADD230CC4032600098C4C /* WebKitCSSTransformValue.h in Headers */ = {isa = PBXBuildFile; fileRef = BC9ADD220CC4032600098C4C /* WebKitCSSTransformValue.h */; settings = {ATTRIBUTES = (Private, ); }; };
- BC9ADD800CC4092200098C4C /* WebKitCSSTransformValue.cpp in Sources */ = {isa = PBXBuildFile; fileRef = BC9ADD7F0CC4092200098C4C /* WebKitCSSTransformValue.cpp */; };
</del><span class="cx"> BCA169A20BFD55B40019CA76 /* JSHTMLTableCaptionElement.cpp in Sources */ = {isa = PBXBuildFile; fileRef = BCA169A00BFD55B40019CA76 /* JSHTMLTableCaptionElement.cpp */; };
</span><span class="cx"> BCA169A30BFD55B40019CA76 /* JSHTMLTableCaptionElement.h in Headers */ = {isa = PBXBuildFile; fileRef = BCA169A10BFD55B40019CA76 /* JSHTMLTableCaptionElement.h */; };
</span><span class="cx"> BCA257151293C010007A263D /* VerticalPositionCache.h in Headers */ = {isa = PBXBuildFile; fileRef = BCA257141293C010007A263D /* VerticalPositionCache.h */; };
</span><span class="lines">@@ -8485,9 +8481,6 @@
</span><span class="cx"> 314BE3A41B3103FB00141982 /* NamedImageGeneratedImage.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = NamedImageGeneratedImage.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 314BE3A51B3103FB00141982 /* NamedImageGeneratedImage.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = NamedImageGeneratedImage.h; sourceTree = "<group>"; };
</span><span class="cx"> 316023EF1532C40C00D50FF4 /* Dictionary.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = Dictionary.h; sourceTree = "<group>"; };
</span><del>- 31611E540E1C4D4A00F6A579 /* WebKitCSSTransformValue.idl */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text; path = WebKitCSSTransformValue.idl; sourceTree = "<group>"; };
- 31611E580E1C4DE000F6A579 /* JSWebKitCSSTransformValue.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSWebKitCSSTransformValue.cpp; sourceTree = "<group>"; };
- 31611E590E1C4DE000F6A579 /* JSWebKitCSSTransformValue.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSWebKitCSSTransformValue.h; sourceTree = "<group>"; };
</del><span class="cx"> 316FE06D0E6CCBEE00BF6088 /* JSCSSKeyframeRule.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSCSSKeyframeRule.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 316FE06E0E6CCBEE00BF6088 /* JSCSSKeyframeRule.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSCSSKeyframeRule.h; sourceTree = "<group>"; };
</span><span class="cx"> 316FE06F0E6CCBEE00BF6088 /* JSCSSKeyframesRule.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSCSSKeyframesRule.cpp; sourceTree = "<group>"; };
</span><span class="lines">@@ -13248,8 +13241,6 @@
</span><span class="cx"> BC9A6144146859D9006057FD /* make_dom_exceptions.pl */ = {isa = PBXFileReference; lastKnownFileType = text.script.perl; path = make_dom_exceptions.pl; sourceTree = "<group>"; };
</span><span class="cx"> BC9A6145146859D9006057FD /* make_event_factory.pl */ = {isa = PBXFileReference; lastKnownFileType = text.script.perl; path = make_event_factory.pl; sourceTree = "<group>"; };
</span><span class="cx"> BC9A6146146859D9006057FD /* make_names.pl */ = {isa = PBXFileReference; lastKnownFileType = text.script.perl; path = make_names.pl; sourceTree = "<group>"; };
</span><del>- BC9ADD220CC4032600098C4C /* WebKitCSSTransformValue.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = WebKitCSSTransformValue.h; sourceTree = "<group>"; };
- BC9ADD7F0CC4092200098C4C /* WebKitCSSTransformValue.cpp */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.cpp; path = WebKitCSSTransformValue.cpp; sourceTree = "<group>"; };
</del><span class="cx"> BCA169A00BFD55B40019CA76 /* JSHTMLTableCaptionElement.cpp */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.cpp; path = JSHTMLTableCaptionElement.cpp; sourceTree = "<group>"; };
</span><span class="cx"> BCA169A10BFD55B40019CA76 /* JSHTMLTableCaptionElement.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = JSHTMLTableCaptionElement.h; sourceTree = "<group>"; };
</span><span class="cx"> BCA257141293C010007A263D /* VerticalPositionCache.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = VerticalPositionCache.h; sourceTree = "<group>"; };
</span><span class="lines">@@ -20365,8 +20356,6 @@
</span><span class="cx"> 4983913E0F1E767500C23782 /* JSWebKitCSSMatrix.h */,
</span><span class="cx"> 8ACC24CD148E24B200EFCC0D /* JSWebKitCSSRegionRule.cpp */,
</span><span class="cx"> 8ACC24CE148E24B200EFCC0D /* JSWebKitCSSRegionRule.h */,
</span><del>- 31611E580E1C4DE000F6A579 /* JSWebKitCSSTransformValue.cpp */,
- 31611E590E1C4DE000F6A579 /* JSWebKitCSSTransformValue.h */,
</del><span class="cx"> 3F2B33E9165AF15500E3987C /* JSWebKitCSSViewportRule.cpp */,
</span><span class="cx"> 3F2B33EA165AF15500E3987C /* JSWebKitCSSViewportRule.h */,
</span><span class="cx"> );
</span><span class="lines">@@ -23550,9 +23539,6 @@
</span><span class="cx"> 8AA61CFD144D595B00F37350 /* WebKitCSSRegionRule.cpp */,
</span><span class="cx"> 8AA61CFE144D595B00F37350 /* WebKitCSSRegionRule.h */,
</span><span class="cx"> 8AD0A55614C87425000D83C5 /* WebKitCSSRegionRule.idl */,
</span><del>- BC9ADD7F0CC4092200098C4C /* WebKitCSSTransformValue.cpp */,
- BC9ADD220CC4032600098C4C /* WebKitCSSTransformValue.h */,
- 31611E540E1C4D4A00F6A579 /* WebKitCSSTransformValue.idl */,
</del><span class="cx"> 3FFFF9A6159D9A550020BBD5 /* WebKitCSSViewportRule.cpp */,
</span><span class="cx"> 3FFFF9A7159D9A550020BBD5 /* WebKitCSSViewportRule.h */,
</span><span class="cx"> 3F2B33E3165ABD3500E3987C /* WebKitCSSViewportRule.idl */,
</span><span class="lines">@@ -26556,7 +26542,6 @@
</span><span class="cx"> 31C0FF3E0E4CEFAC007D6FE5 /* JSWebKitAnimationEvent.h in Headers */,
</span><span class="cx"> 498391400F1E767500C23782 /* JSWebKitCSSMatrix.h in Headers */,
</span><span class="cx"> 8AD0A59514C88336000D83C5 /* JSWebKitCSSRegionRule.h in Headers */,
</span><del>- 31611E5B0E1C4DE000F6A579 /* JSWebKitCSSTransformValue.h in Headers */,
</del><span class="cx"> 3F2B33EF165AF15600E3987C /* JSWebKitCSSViewportRule.h in Headers */,
</span><span class="cx"> 7CC69941191EC5F500AF2270 /* JSWebKitNamespace.h in Headers */,
</span><span class="cx"> 0FDA7C271883333200C954B5 /* JSWebKitPlaybackTargetAvailabilityEvent.h in Headers */,
</span><span class="lines">@@ -28147,7 +28132,6 @@
</span><span class="cx"> 976D6C7C122B8A3D001FD1F7 /* BlobBuilder.h in Headers */,
</span><span class="cx"> 498391590F1E776900C23782 /* WebKitCSSMatrix.h in Headers */,
</span><span class="cx"> 8AA61D00144D595B00F37350 /* WebKitCSSRegionRule.h in Headers */,
</span><del>- BC9ADD230CC4032600098C4C /* WebKitCSSTransformValue.h in Headers */,
</del><span class="cx"> 3FFFF9A9159D9A550020BBD5 /* WebKitCSSViewportRule.h in Headers */,
</span><span class="cx"> 2D0621451DA639BA00A7FB26 /* WebKitMediaKeyMessageEvent.h in Headers */,
</span><span class="cx"> 2D0621511DA63A9F00A7FB26 /* WebKitMediaKeyNeededEvent.h in Headers */,
</span><span class="lines">@@ -30358,7 +30342,6 @@
</span><span class="cx"> 31C0FF3D0E4CEFAC007D6FE5 /* JSWebKitAnimationEvent.cpp in Sources */,
</span><span class="cx"> 4983913F0F1E767500C23782 /* JSWebKitCSSMatrix.cpp in Sources */,
</span><span class="cx"> 8AD0A59414C88336000D83C5 /* JSWebKitCSSRegionRule.cpp in Sources */,
</span><del>- 31611E5A0E1C4DE000F6A579 /* JSWebKitCSSTransformValue.cpp in Sources */,
</del><span class="cx"> 3F2B33EE165AF15600E3987C /* JSWebKitCSSViewportRule.cpp in Sources */,
</span><span class="cx"> CDA98DA31601464100FEA3B1 /* JSWebKitMediaKeyError.cpp in Sources */,
</span><span class="cx"> CDA98DDF16026A3700FEA3B1 /* JSWebKitMediaKeyMessageEvent.cpp in Sources */,
</span><span class="lines">@@ -31637,7 +31620,6 @@
</span><span class="cx"> 976D6C7B122B8A3D001FD1F7 /* BlobBuilder.cpp in Sources */,
</span><span class="cx"> 498391580F1E776900C23782 /* WebKitCSSMatrix.cpp in Sources */,
</span><span class="cx"> 8AA61CFF144D595B00F37350 /* WebKitCSSRegionRule.cpp in Sources */,
</span><del>- BC9ADD800CC4092200098C4C /* WebKitCSSTransformValue.cpp in Sources */,
</del><span class="cx"> 3FFFF9A8159D9A550020BBD5 /* WebKitCSSViewportRule.cpp in Sources */,
</span><span class="cx"> 1A1414B513A0F0500019996C /* WebKitFontFamilyNames.cpp in Sources */,
</span><span class="cx"> 2D0621441DA639B600A7FB26 /* WebKitMediaKeyMessageEvent.cpp in Sources */,
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsjsJSCSSValueCustomcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/js/JSCSSValueCustom.cpp (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/js/JSCSSValueCustom.cpp 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/bindings/js/JSCSSValueCustom.cpp 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -31,8 +31,6 @@
</span><span class="cx"> #include "JSCSSPrimitiveValue.h"
</span><span class="cx"> #include "JSCSSValueList.h"
</span><span class="cx"> #include "JSNode.h"
</span><del>-#include "JSWebKitCSSTransformValue.h"
-#include "WebKitCSSTransformValue.h"
</del><span class="cx">
</span><span class="cx"> using namespace JSC;
</span><span class="cx">
</span><span class="lines">@@ -60,8 +58,6 @@
</span><span class="cx">
</span><span class="cx"> JSValue toJSNewlyCreated(ExecState*, JSDOMGlobalObject* globalObject, Ref<CSSValue>&& value)
</span><span class="cx"> {
</span><del>- if (value->isWebKitCSSTransformValue())
- return createWrapper<WebKitCSSTransformValue>(globalObject, WTFMove(value));
</del><span class="cx"> if (value->isValueList())
</span><span class="cx"> return createWrapper<CSSValueList>(globalObject, WTFMove(value));
</span><span class="cx"> if (value->isPrimitiveValue())
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSComputedStyleDeclarationcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -75,7 +75,6 @@
</span><span class="cx"> #include "StyleResolver.h"
</span><span class="cx"> #include "StyleScope.h"
</span><span class="cx"> #include "Text.h"
</span><del>-#include "WebKitCSSTransformValue.h"
</del><span class="cx"> #include "WebKitFontFamilyNames.h"
</span><span class="cx"> #include "WillChangeData.h"
</span><span class="cx"> #include <wtf/NeverDestroyed.h>
</span><span class="lines">@@ -818,12 +817,12 @@
</span><span class="cx"> return box.style().boxSizing() == BORDER_BOX ? box.borderBoxRect() : box.computedCSSContentBoxRect();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-static Ref<WebKitCSSTransformValue> matrixTransformValue(const TransformationMatrix& transform, const RenderStyle& style)
</del><ins>+static Ref<CSSFunctionValue> matrixTransformValue(const TransformationMatrix& transform, const RenderStyle& style)
</ins><span class="cx"> {
</span><del>- RefPtr<WebKitCSSTransformValue> transformValue;
</del><ins>+ RefPtr<CSSFunctionValue> transformValue;
</ins><span class="cx"> auto& cssValuePool = CSSValuePool::singleton();
</span><span class="cx"> if (transform.isAffine()) {
</span><del>- transformValue = WebKitCSSTransformValue::create(WebKitCSSTransformValue::MatrixTransformOperation);
</del><ins>+ transformValue = CSSFunctionValue::create(CSSValueMatrix);
</ins><span class="cx">
</span><span class="cx"> transformValue->append(cssValuePool.createValue(transform.a(), CSSPrimitiveValue::CSS_NUMBER));
</span><span class="cx"> transformValue->append(cssValuePool.createValue(transform.b(), CSSPrimitiveValue::CSS_NUMBER));
</span><span class="lines">@@ -832,7 +831,7 @@
</span><span class="cx"> transformValue->append(zoomAdjustedNumberValue(transform.e(), style));
</span><span class="cx"> transformValue->append(zoomAdjustedNumberValue(transform.f(), style));
</span><span class="cx"> } else {
</span><del>- transformValue = WebKitCSSTransformValue::create(WebKitCSSTransformValue::Matrix3DTransformOperation);
</del><ins>+ transformValue = CSSFunctionValue::create(CSSValueMatrix3d);
</ins><span class="cx">
</span><span class="cx"> transformValue->append(cssValuePool.createValue(transform.m11(), CSSPrimitiveValue::CSS_NUMBER));
</span><span class="cx"> transformValue->append(cssValuePool.createValue(transform.m12(), CSSPrimitiveValue::CSS_NUMBER));
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSValuecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSValue.cpp (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSValue.cpp 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/CSSValue.cpp 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -61,7 +61,6 @@
</span><span class="cx"> #include "CSSUnsetValue.h"
</span><span class="cx"> #include "CSSValueList.h"
</span><span class="cx"> #include "CSSVariableReferenceValue.h"
</span><del>-#include "WebKitCSSTransformValue.h"
</del><span class="cx">
</span><span class="cx"> #if ENABLE(CSS_GRID_LAYOUT)
</span><span class="cx"> #include "CSSGridAutoRepeatValue.h"
</span><span class="lines">@@ -218,8 +217,6 @@
</span><span class="cx"> return compareCSSValues<CSSUnicodeRangeValue>(*this, other);
</span><span class="cx"> case ValueListClass:
</span><span class="cx"> return compareCSSValues<CSSValueList>(*this, other);
</span><del>- case WebKitCSSTransformClass:
- return compareCSSValues<WebKitCSSTransformValue>(*this, other);
</del><span class="cx"> case LineBoxContainClass:
</span><span class="cx"> return compareCSSValues<CSSLineBoxContainValue>(*this, other);
</span><span class="cx"> case CalculationClass:
</span><span class="lines">@@ -322,8 +319,6 @@
</span><span class="cx"> return downcast<CSSUnicodeRangeValue>(*this).customCSSText();
</span><span class="cx"> case ValueListClass:
</span><span class="cx"> return downcast<CSSValueList>(*this).customCSSText();
</span><del>- case WebKitCSSTransformClass:
- return downcast<WebKitCSSTransformValue>(*this).customCSSText();
</del><span class="cx"> case LineBoxContainClass:
</span><span class="cx"> return downcast<CSSLineBoxContainValue>(*this).customCSSText();
</span><span class="cx"> case CalculationClass:
</span><span class="lines">@@ -451,9 +446,6 @@
</span><span class="cx"> case ValueListClass:
</span><span class="cx"> delete downcast<CSSValueList>(this);
</span><span class="cx"> return;
</span><del>- case WebKitCSSTransformClass:
- delete downcast<WebKitCSSTransformValue>(this);
- return;
</del><span class="cx"> case LineBoxContainClass:
</span><span class="cx"> delete downcast<CSSLineBoxContainValue>(this);
</span><span class="cx"> return;
</span><span class="lines">@@ -500,8 +492,6 @@
</span><span class="cx"> case ImageClass:
</span><span class="cx"> case CursorImageClass:
</span><span class="cx"> return downcast<CSSImageValue>(*this).cloneForCSSOM();
</span><del>- case WebKitCSSTransformClass:
- return downcast<WebKitCSSTransformValue>(*this).cloneForCSSOM();
</del><span class="cx"> case ImageSetClass:
</span><span class="cx"> return downcast<CSSImageSetValue>(*this).cloneForCSSOM();
</span><span class="cx"> default:
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSValueh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSValue.h (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSValue.h 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/CSSValue.h 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -105,7 +105,6 @@
</span><span class="cx"> bool isCubicBezierTimingFunctionValue() const { return m_classType == CubicBezierTimingFunctionClass; }
</span><span class="cx"> bool isStepsTimingFunctionValue() const { return m_classType == StepsTimingFunctionClass; }
</span><span class="cx"> bool isSpringTimingFunctionValue() const { return m_classType == SpringTimingFunctionClass; }
</span><del>- bool isWebKitCSSTransformValue() const { return m_classType == WebKitCSSTransformClass; }
</del><span class="cx"> bool isLineBoxContainValue() const { return m_classType == LineBoxContainClass; }
</span><span class="cx"> bool isCalcValue() const {return m_classType == CalculationClass; }
</span><span class="cx"> bool isFilterImageValue() const { return m_classType == FilterImageClass; }
</span><span class="lines">@@ -199,10 +198,11 @@
</span><span class="cx"> VariableReferenceClass,
</span><span class="cx"> PendingSubstitutionValueClass,
</span><span class="cx">
</span><del>- // List class types must appear after ValueListClass.
</del><ins>+ // List class types must appear after ValueListClass. Note CSSFunctionValue
+ // is deliberately excluded, since we don't want it exposed to the CSS OM
+ // as a list.
</ins><span class="cx"> ValueListClass,
</span><span class="cx"> ImageSetClass,
</span><del>- WebKitCSSTransformClass,
</del><span class="cx"> #if ENABLE(CSS_GRID_LAYOUT)
</span><span class="cx"> GridLineNamesClass,
</span><span class="cx"> GridAutoRepeatClass,
</span></span></pre></div>
<a id="trunkSourceWebCorecssStyleResolvercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/StyleResolver.cpp (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/StyleResolver.cpp 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/StyleResolver.cpp 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -134,7 +134,6 @@
</span><span class="cx"> #include "ViewportStyleResolver.h"
</span><span class="cx"> #include "VisitedLinkState.h"
</span><span class="cx"> #include "WebKitCSSRegionRule.h"
</span><del>-#include "WebKitCSSTransformValue.h"
</del><span class="cx"> #include "WebKitFontFamilyNames.h"
</span><span class="cx"> #include "XMLNames.h"
</span><span class="cx"> #include <bitset>
</span></span></pre></div>
<a id="trunkSourceWebCorecssTransformFunctionscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/TransformFunctions.cpp (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/TransformFunctions.cpp 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/TransformFunctions.cpp 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -32,6 +32,7 @@
</span><span class="cx"> #include "config.h"
</span><span class="cx"> #include "TransformFunctions.h"
</span><span class="cx">
</span><ins>+#include "CSSFunctionValue.h"
</ins><span class="cx"> #include "CSSPrimitiveValueMappings.h"
</span><span class="cx"> #include "CSSValueList.h"
</span><span class="cx"> #include "CSSValuePool.h"
</span><span class="lines">@@ -42,35 +43,56 @@
</span><span class="cx"> #include "ScaleTransformOperation.h"
</span><span class="cx"> #include "SkewTransformOperation.h"
</span><span class="cx"> #include "TranslateTransformOperation.h"
</span><del>-#include "WebKitCSSTransformValue.h"
</del><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><del>-static TransformOperation::OperationType transformOperationType(WebKitCSSTransformValue::TransformOperationType type)
</del><ins>+static TransformOperation::OperationType transformOperationType(CSSValueID type)
</ins><span class="cx"> {
</span><span class="cx"> switch (type) {
</span><del>- case WebKitCSSTransformValue::ScaleTransformOperation: return TransformOperation::SCALE;
- case WebKitCSSTransformValue::ScaleXTransformOperation: return TransformOperation::SCALE_X;
- case WebKitCSSTransformValue::ScaleYTransformOperation: return TransformOperation::SCALE_Y;
- case WebKitCSSTransformValue::ScaleZTransformOperation: return TransformOperation::SCALE_Z;
- case WebKitCSSTransformValue::Scale3DTransformOperation: return TransformOperation::SCALE_3D;
- case WebKitCSSTransformValue::TranslateTransformOperation: return TransformOperation::TRANSLATE;
- case WebKitCSSTransformValue::TranslateXTransformOperation: return TransformOperation::TRANSLATE_X;
- case WebKitCSSTransformValue::TranslateYTransformOperation: return TransformOperation::TRANSLATE_Y;
- case WebKitCSSTransformValue::TranslateZTransformOperation: return TransformOperation::TRANSLATE_Z;
- case WebKitCSSTransformValue::Translate3DTransformOperation: return TransformOperation::TRANSLATE_3D;
- case WebKitCSSTransformValue::RotateTransformOperation: return TransformOperation::ROTATE;
- case WebKitCSSTransformValue::RotateXTransformOperation: return TransformOperation::ROTATE_X;
- case WebKitCSSTransformValue::RotateYTransformOperation: return TransformOperation::ROTATE_Y;
- case WebKitCSSTransformValue::RotateZTransformOperation: return TransformOperation::ROTATE_Z;
- case WebKitCSSTransformValue::Rotate3DTransformOperation: return TransformOperation::ROTATE_3D;
- case WebKitCSSTransformValue::SkewTransformOperation: return TransformOperation::SKEW;
- case WebKitCSSTransformValue::SkewXTransformOperation: return TransformOperation::SKEW_X;
- case WebKitCSSTransformValue::SkewYTransformOperation: return TransformOperation::SKEW_Y;
- case WebKitCSSTransformValue::MatrixTransformOperation: return TransformOperation::MATRIX;
- case WebKitCSSTransformValue::Matrix3DTransformOperation: return TransformOperation::MATRIX_3D;
- case WebKitCSSTransformValue::PerspectiveTransformOperation: return TransformOperation::PERSPECTIVE;
- case WebKitCSSTransformValue::UnknownTransformOperation: return TransformOperation::NONE;
</del><ins>+ case CSSValueScale:
+ return TransformOperation::SCALE;
+ case CSSValueScalex:
+ return TransformOperation::SCALE_X;
+ case CSSValueScaley:
+ return TransformOperation::SCALE_Y;
+ case CSSValueScalez:
+ return TransformOperation::SCALE_Z;
+ case CSSValueScale3d:
+ return TransformOperation::SCALE_3D;
+ case CSSValueTranslate:
+ return TransformOperation::TRANSLATE;
+ case CSSValueTranslatex:
+ return TransformOperation::TRANSLATE_X;
+ case CSSValueTranslatey:
+ return TransformOperation::TRANSLATE_Y;
+ case CSSValueTranslatez:
+ return TransformOperation::TRANSLATE_Z;
+ case CSSValueTranslate3d:
+ return TransformOperation::TRANSLATE_3D;
+ case CSSValueRotate:
+ return TransformOperation::ROTATE;
+ case CSSValueRotatex:
+ return TransformOperation::ROTATE_X;
+ case CSSValueRotatey:
+ return TransformOperation::ROTATE_Y;
+ case CSSValueRotatez:
+ return TransformOperation::ROTATE_Z;
+ case CSSValueRotate3d:
+ return TransformOperation::ROTATE_3D;
+ case CSSValueSkew:
+ return TransformOperation::SKEW;
+ case CSSValueSkewx:
+ return TransformOperation::SKEW_X;
+ case CSSValueSkewy:
+ return TransformOperation::SKEW_Y;
+ case CSSValueMatrix:
+ return TransformOperation::MATRIX;
+ case CSSValueMatrix3d:
+ return TransformOperation::MATRIX_3D;
+ case CSSValuePerspective:
+ return TransformOperation::PERSPECTIVE;
+ default:
+ break;
</ins><span class="cx"> }
</span><span class="cx"> return TransformOperation::NONE;
</span><span class="cx"> }
</span><span class="lines">@@ -89,10 +111,10 @@
</span><span class="cx">
</span><span class="cx"> TransformOperations operations;
</span><span class="cx"> for (auto& currentValue : downcast<CSSValueList>(value)) {
</span><del>- if (!is<WebKitCSSTransformValue>(currentValue.get()))
</del><ins>+ if (!is<CSSFunctionValue>(currentValue.get()))
</ins><span class="cx"> continue;
</span><span class="cx">
</span><del>- auto& transformValue = downcast<WebKitCSSTransformValue>(currentValue.get());
</del><ins>+ auto& transformValue = downcast<CSSFunctionValue>(currentValue.get());
</ins><span class="cx"> if (!transformValue.length())
</span><span class="cx"> continue;
</span><span class="cx">
</span><span class="lines">@@ -108,17 +130,17 @@
</span><span class="cx">
</span><span class="cx"> auto& firstValue = downcast<CSSPrimitiveValue>(*transformValue.itemWithoutBoundsCheck(0));
</span><span class="cx">
</span><del>- switch (transformValue.operationType()) {
- case WebKitCSSTransformValue::ScaleTransformOperation:
- case WebKitCSSTransformValue::ScaleXTransformOperation:
- case WebKitCSSTransformValue::ScaleYTransformOperation: {
</del><ins>+ switch (transformValue.name()) {
+ case CSSValueScale:
+ case CSSValueScalex:
+ case CSSValueScaley: {
</ins><span class="cx"> double sx = 1.0;
</span><span class="cx"> double sy = 1.0;
</span><del>- if (transformValue.operationType() == WebKitCSSTransformValue::ScaleYTransformOperation)
</del><ins>+ if (transformValue.name() == CSSValueScaley)
</ins><span class="cx"> sy = firstValue.doubleValue();
</span><span class="cx"> else {
</span><span class="cx"> sx = firstValue.doubleValue();
</span><del>- if (transformValue.operationType() != WebKitCSSTransformValue::ScaleXTransformOperation) {
</del><ins>+ if (transformValue.name() != CSSValueScalex) {
</ins><span class="cx"> if (transformValue.length() > 1) {
</span><span class="cx"> auto& secondValue = downcast<CSSPrimitiveValue>(*transformValue.itemWithoutBoundsCheck(1));
</span><span class="cx"> sy = secondValue.doubleValue();
</span><span class="lines">@@ -126,21 +148,21 @@
</span><span class="cx"> sy = sx;
</span><span class="cx"> }
</span><span class="cx"> }
</span><del>- operations.operations().append(ScaleTransformOperation::create(sx, sy, 1.0, transformOperationType(transformValue.operationType())));
</del><ins>+ operations.operations().append(ScaleTransformOperation::create(sx, sy, 1.0, transformOperationType(transformValue.name())));
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::ScaleZTransformOperation:
- case WebKitCSSTransformValue::Scale3DTransformOperation: {
</del><ins>+ case CSSValueScalez:
+ case CSSValueScale3d: {
</ins><span class="cx"> double sx = 1.0;
</span><span class="cx"> double sy = 1.0;
</span><span class="cx"> double sz = 1.0;
</span><del>- if (transformValue.operationType() == WebKitCSSTransformValue::ScaleZTransformOperation)
</del><ins>+ if (transformValue.name() == CSSValueScalez)
</ins><span class="cx"> sz = firstValue.doubleValue();
</span><del>- else if (transformValue.operationType() == WebKitCSSTransformValue::ScaleYTransformOperation)
</del><ins>+ else if (transformValue.name() == CSSValueScaley)
</ins><span class="cx"> sy = firstValue.doubleValue();
</span><span class="cx"> else {
</span><span class="cx"> sx = firstValue.doubleValue();
</span><del>- if (transformValue.operationType() != WebKitCSSTransformValue::ScaleXTransformOperation) {
</del><ins>+ if (transformValue.name() != CSSValueScalex) {
</ins><span class="cx"> if (transformValue.length() > 2) {
</span><span class="cx"> auto& thirdValue = downcast<CSSPrimitiveValue>(*transformValue.itemWithoutBoundsCheck(2));
</span><span class="cx"> sz = thirdValue.doubleValue();
</span><span class="lines">@@ -152,19 +174,19 @@
</span><span class="cx"> sy = sx;
</span><span class="cx"> }
</span><span class="cx"> }
</span><del>- operations.operations().append(ScaleTransformOperation::create(sx, sy, sz, transformOperationType(transformValue.operationType())));
</del><ins>+ operations.operations().append(ScaleTransformOperation::create(sx, sy, sz, transformOperationType(transformValue.name())));
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::TranslateTransformOperation:
- case WebKitCSSTransformValue::TranslateXTransformOperation:
- case WebKitCSSTransformValue::TranslateYTransformOperation: {
</del><ins>+ case CSSValueTranslate:
+ case CSSValueTranslatex:
+ case CSSValueTranslatey: {
</ins><span class="cx"> Length tx = Length(0, Fixed);
</span><span class="cx"> Length ty = Length(0, Fixed);
</span><del>- if (transformValue.operationType() == WebKitCSSTransformValue::TranslateYTransformOperation)
</del><ins>+ if (transformValue.name() == CSSValueTranslatey)
</ins><span class="cx"> ty = convertToFloatLength(&firstValue, conversionData);
</span><span class="cx"> else {
</span><span class="cx"> tx = convertToFloatLength(&firstValue, conversionData);
</span><del>- if (transformValue.operationType() != WebKitCSSTransformValue::TranslateXTransformOperation) {
</del><ins>+ if (transformValue.name() != CSSValueTranslatex) {
</ins><span class="cx"> if (transformValue.length() > 1) {
</span><span class="cx"> auto& secondValue = downcast<CSSPrimitiveValue>(*transformValue.itemWithoutBoundsCheck(1));
</span><span class="cx"> ty = convertToFloatLength(&secondValue, conversionData);
</span><span class="lines">@@ -175,21 +197,21 @@
</span><span class="cx"> if (tx.isUndefined() || ty.isUndefined())
</span><span class="cx"> return false;
</span><span class="cx">
</span><del>- operations.operations().append(TranslateTransformOperation::create(tx, ty, Length(0, Fixed), transformOperationType(transformValue.operationType())));
</del><ins>+ operations.operations().append(TranslateTransformOperation::create(tx, ty, Length(0, Fixed), transformOperationType(transformValue.name())));
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::TranslateZTransformOperation:
- case WebKitCSSTransformValue::Translate3DTransformOperation: {
</del><ins>+ case CSSValueTranslatez:
+ case CSSValueTranslate3d: {
</ins><span class="cx"> Length tx = Length(0, Fixed);
</span><span class="cx"> Length ty = Length(0, Fixed);
</span><span class="cx"> Length tz = Length(0, Fixed);
</span><del>- if (transformValue.operationType() == WebKitCSSTransformValue::TranslateZTransformOperation)
</del><ins>+ if (transformValue.name() == CSSValueTranslatez)
</ins><span class="cx"> tz = convertToFloatLength(&firstValue, conversionData);
</span><del>- else if (transformValue.operationType() == WebKitCSSTransformValue::TranslateYTransformOperation)
</del><ins>+ else if (transformValue.name() == CSSValueTranslatey)
</ins><span class="cx"> ty = convertToFloatLength(&firstValue, conversionData);
</span><span class="cx"> else {
</span><span class="cx"> tx = convertToFloatLength(&firstValue, conversionData);
</span><del>- if (transformValue.operationType() != WebKitCSSTransformValue::TranslateXTransformOperation) {
</del><ins>+ if (transformValue.name() != CSSValueTranslatex) {
</ins><span class="cx"> if (transformValue.length() > 2) {
</span><span class="cx"> auto& thirdValue = downcast<CSSPrimitiveValue>(*transformValue.itemWithoutBoundsCheck(2));
</span><span class="cx"> tz = convertToFloatLength(&thirdValue, conversionData);
</span><span class="lines">@@ -204,32 +226,32 @@
</span><span class="cx"> if (tx.isUndefined() || ty.isUndefined() || tz.isUndefined())
</span><span class="cx"> return false;
</span><span class="cx">
</span><del>- operations.operations().append(TranslateTransformOperation::create(tx, ty, tz, transformOperationType(transformValue.operationType())));
</del><ins>+ operations.operations().append(TranslateTransformOperation::create(tx, ty, tz, transformOperationType(transformValue.name())));
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::RotateTransformOperation: {
</del><ins>+ case CSSValueRotate: {
</ins><span class="cx"> double angle = firstValue.computeDegrees();
</span><del>- operations.operations().append(RotateTransformOperation::create(0, 0, 1, angle, transformOperationType(transformValue.operationType())));
</del><ins>+ operations.operations().append(RotateTransformOperation::create(0, 0, 1, angle, transformOperationType(transformValue.name())));
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::RotateXTransformOperation:
- case WebKitCSSTransformValue::RotateYTransformOperation:
- case WebKitCSSTransformValue::RotateZTransformOperation: {
</del><ins>+ case CSSValueRotatex:
+ case CSSValueRotatey:
+ case CSSValueRotatez: {
</ins><span class="cx"> double x = 0;
</span><span class="cx"> double y = 0;
</span><span class="cx"> double z = 0;
</span><span class="cx"> double angle = firstValue.computeDegrees();
</span><span class="cx">
</span><del>- if (transformValue.operationType() == WebKitCSSTransformValue::RotateXTransformOperation)
</del><ins>+ if (transformValue.name() == CSSValueRotatex)
</ins><span class="cx"> x = 1;
</span><del>- else if (transformValue.operationType() == WebKitCSSTransformValue::RotateYTransformOperation)
</del><ins>+ else if (transformValue.name() == CSSValueRotatey)
</ins><span class="cx"> y = 1;
</span><span class="cx"> else
</span><span class="cx"> z = 1;
</span><del>- operations.operations().append(RotateTransformOperation::create(x, y, z, angle, transformOperationType(transformValue.operationType())));
</del><ins>+ operations.operations().append(RotateTransformOperation::create(x, y, z, angle, transformOperationType(transformValue.name())));
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::Rotate3DTransformOperation: {
</del><ins>+ case CSSValueRotate3d: {
</ins><span class="cx"> if (transformValue.length() < 4)
</span><span class="cx"> break;
</span><span class="cx"> auto& secondValue = downcast<CSSPrimitiveValue>(*transformValue.itemWithoutBoundsCheck(1));
</span><span class="lines">@@ -239,20 +261,20 @@
</span><span class="cx"> double y = secondValue.doubleValue();
</span><span class="cx"> double z = thirdValue.doubleValue();
</span><span class="cx"> double angle = fourthValue.computeDegrees();
</span><del>- operations.operations().append(RotateTransformOperation::create(x, y, z, angle, transformOperationType(transformValue.operationType())));
</del><ins>+ operations.operations().append(RotateTransformOperation::create(x, y, z, angle, transformOperationType(transformValue.name())));
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::SkewTransformOperation:
- case WebKitCSSTransformValue::SkewXTransformOperation:
- case WebKitCSSTransformValue::SkewYTransformOperation: {
</del><ins>+ case CSSValueSkew:
+ case CSSValueSkewx:
+ case CSSValueSkewy: {
</ins><span class="cx"> double angleX = 0;
</span><span class="cx"> double angleY = 0;
</span><span class="cx"> double angle = firstValue.computeDegrees();
</span><del>- if (transformValue.operationType() == WebKitCSSTransformValue::SkewYTransformOperation)
</del><ins>+ if (transformValue.name() == CSSValueSkewy)
</ins><span class="cx"> angleY = angle;
</span><span class="cx"> else {
</span><span class="cx"> angleX = angle;
</span><del>- if (transformValue.operationType() == WebKitCSSTransformValue::SkewTransformOperation) {
</del><ins>+ if (transformValue.name() == CSSValueSkew) {
</ins><span class="cx"> if (transformValue.length() > 1) {
</span><span class="cx"> auto& secondValue = downcast<CSSPrimitiveValue>(*transformValue.itemWithoutBoundsCheck(1));
</span><span class="cx"> angleY = secondValue.computeDegrees();
</span><span class="lines">@@ -259,10 +281,10 @@
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx"> }
</span><del>- operations.operations().append(SkewTransformOperation::create(angleX, angleY, transformOperationType(transformValue.operationType())));
</del><ins>+ operations.operations().append(SkewTransformOperation::create(angleX, angleY, transformOperationType(transformValue.name())));
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::MatrixTransformOperation: {
</del><ins>+ case CSSValueMatrix: {
</ins><span class="cx"> if (transformValue.length() < 6)
</span><span class="cx"> break;
</span><span class="cx"> double a = firstValue.doubleValue();
</span><span class="lines">@@ -274,7 +296,7 @@
</span><span class="cx"> operations.operations().append(MatrixTransformOperation::create(a, b, c, d, e, f));
</span><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::Matrix3DTransformOperation: {
</del><ins>+ case CSSValueMatrix3d: {
</ins><span class="cx"> if (transformValue.length() < 16)
</span><span class="cx"> break;
</span><span class="cx"> TransformationMatrix matrix(downcast<CSSPrimitiveValue>(*transformValue.itemWithoutBoundsCheck(0)).doubleValue(),
</span><span class="lines">@@ -296,7 +318,7 @@
</span><span class="cx"> operations.operations().append(Matrix3DTransformOperation::create(matrix));
</span><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::PerspectiveTransformOperation: {
</del><ins>+ case CSSValuePerspective: {
</ins><span class="cx"> Length p = Length(0, Fixed);
</span><span class="cx"> if (firstValue.isLength())
</span><span class="cx"> p = convertToFloatLength(&firstValue, conversionData);
</span><span class="lines">@@ -312,7 +334,7 @@
</span><span class="cx"> operations.operations().append(PerspectiveTransformOperation::create(p));
</span><span class="cx"> break;
</span><span class="cx"> }
</span><del>- case WebKitCSSTransformValue::UnknownTransformOperation:
</del><ins>+ default:
</ins><span class="cx"> ASSERT_NOT_REACHED();
</span><span class="cx"> break;
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorecssTransformFunctionsh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/TransformFunctions.h (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/TransformFunctions.h 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/TransformFunctions.h 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -37,7 +37,6 @@
</span><span class="cx"> class CSSToLengthConversionData;
</span><span class="cx"> class CSSValue;
</span><span class="cx"> class RenderStyle;
</span><del>-class WebKitCSSTransformValue;
</del><span class="cx">
</span><span class="cx"> struct Length;
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorecssWebKitCSSTransformValuecpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebCore/css/WebKitCSSTransformValue.cpp (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/WebKitCSSTransformValue.cpp 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/WebKitCSSTransformValue.cpp 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -1,91 +0,0 @@
</span><del>-/*
- * Copyright (C) 2007, 2008 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. ``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
- * 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.
- */
-
-#include "config.h"
-#include "WebKitCSSTransformValue.h"
-
-#include "CSSValueList.h"
-#include <wtf/text/WTFString.h>
-
-namespace WebCore {
-
-// These names must be kept in sync with TransformOperationType.
-const char* const transformNamePrefixes[] = {
- nullptr,
- "translate(",
- "translateX(",
- "translateY(",
- "rotate(",
- "scale(",
- "scaleX(",
- "scaleY(",
- "skew(",
- "skewX(",
- "skewY(",
- "matrix(",
- "translateZ(",
- "translate3d(",
- "rotateX(",
- "rotateY(",
- "rotateZ(",
- "rotate3d(",
- "scaleZ(",
- "scale3d(",
- "perspective(",
- "matrix3d("
-};
-
-static inline String transformValueToCssString(WebKitCSSTransformValue::TransformOperationType operation, const String& value)
-{
- if (operation != WebKitCSSTransformValue::UnknownTransformOperation) {
- ASSERT_WITH_SECURITY_IMPLICATION(static_cast<size_t>(operation) < WTF_ARRAY_LENGTH(transformNamePrefixes));
- return makeString(transformNamePrefixes[operation], value, ')');
- }
- return String();
-}
-
-WebKitCSSTransformValue::WebKitCSSTransformValue(TransformOperationType op)
- : CSSValueList(WebKitCSSTransformClass, CommaSeparator)
- , m_type(op)
-{
-}
-
-String WebKitCSSTransformValue::customCSSText() const
-{
- return transformValueToCssString(m_type, CSSValueList::customCSSText());
-}
-
-WebKitCSSTransformValue::WebKitCSSTransformValue(const WebKitCSSTransformValue& cloneFrom)
- : CSSValueList(cloneFrom)
- , m_type(cloneFrom.m_type)
-{
-}
-
-Ref<WebKitCSSTransformValue> WebKitCSSTransformValue::cloneForCSSOM() const
-{
- return adoptRef(*new WebKitCSSTransformValue(*this));
-}
-
-}
</del></span></pre></div>
<a id="trunkSourceWebCorecssWebKitCSSTransformValueh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebCore/css/WebKitCSSTransformValue.h (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/WebKitCSSTransformValue.h 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/WebKitCSSTransformValue.h 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -1,81 +0,0 @@
</span><del>-/*
- * Copyright (C) 2007, 2008 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. ``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
- * 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
-
-#include "CSSValueList.h"
-
-namespace WebCore {
-
-class WebKitCSSTransformValue final : public CSSValueList {
-public:
- // NOTE: these have to match the values in the IDL
- enum TransformOperationType {
- UnknownTransformOperation,
- TranslateTransformOperation,
- TranslateXTransformOperation,
- TranslateYTransformOperation,
- RotateTransformOperation,
- ScaleTransformOperation,
- ScaleXTransformOperation,
- ScaleYTransformOperation,
- SkewTransformOperation,
- SkewXTransformOperation,
- SkewYTransformOperation,
- MatrixTransformOperation,
- TranslateZTransformOperation,
- Translate3DTransformOperation,
- RotateXTransformOperation,
- RotateYTransformOperation,
- RotateZTransformOperation,
- Rotate3DTransformOperation,
- ScaleZTransformOperation,
- Scale3DTransformOperation,
- PerspectiveTransformOperation,
- Matrix3DTransformOperation
- };
-
- static Ref<WebKitCSSTransformValue> create(TransformOperationType type)
- {
- return adoptRef(*new WebKitCSSTransformValue(type));
- }
-
- String customCSSText() const;
- bool equals(const WebKitCSSTransformValue& other) const { return m_type == other.m_type && CSSValueList::equals(other); }
-
- TransformOperationType operationType() const { return m_type; }
-
- Ref<WebKitCSSTransformValue> cloneForCSSOM() const;
-
-private:
- WebKitCSSTransformValue(TransformOperationType);
- WebKitCSSTransformValue(const WebKitCSSTransformValue& cloneFrom);
-
- TransformOperationType m_type;
-};
-
-} // namespace WebCore
-
-SPECIALIZE_TYPE_TRAITS_CSS_VALUE(WebKitCSSTransformValue, isWebKitCSSTransformValue())
</del></span></pre></div>
<a id="trunkSourceWebCorecssWebKitCSSTransformValueidl"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebCore/css/WebKitCSSTransformValue.idl (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/WebKitCSSTransformValue.idl 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/WebKitCSSTransformValue.idl 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -1,62 +0,0 @@
</span><del>-/*
- * Copyright (C) 2008 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.
- * 3. Neither the name of Apple Inc. ("Apple") nor the names of
- * its contributors may be used to endorse or promote products derived
- * from this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE 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 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.
- */
-
-[
- DoNotCheckConstants,
- ImplementationLacksVTable,
-] interface WebKitCSSTransformValue : CSSValueList {
-
- // OperationTypes
-
- const unsigned short CSS_TRANSLATE = 1;
- const unsigned short CSS_TRANSLATEX = 2;
- const unsigned short CSS_TRANSLATEY = 3;
- const unsigned short CSS_ROTATE = 4;
- const unsigned short CSS_SCALE = 5;
- const unsigned short CSS_SCALEX = 6;
- const unsigned short CSS_SCALEY = 7;
- const unsigned short CSS_SKEW = 8;
- const unsigned short CSS_SKEWX = 9;
- const unsigned short CSS_SKEWY = 10;
- const unsigned short CSS_MATRIX = 11;
- const unsigned short CSS_TRANSLATEZ = 12;
- const unsigned short CSS_TRANSLATE3D = 13;
- const unsigned short CSS_ROTATEX = 14;
- const unsigned short CSS_ROTATEY = 15;
- const unsigned short CSS_ROTATEZ = 16;
- const unsigned short CSS_ROTATE3D = 17;
- const unsigned short CSS_SCALEZ = 18;
- const unsigned short CSS_SCALE3D = 19;
- const unsigned short CSS_PERSPECTIVE = 20;
- const unsigned short CSS_MATRIX3D = 21;
-
- readonly attribute unsigned short operationType;
-
- getter CSSValue (unsigned long index);
-};
-
</del></span></pre></div>
<a id="trunkSourceWebCorecssparserCSSParserFastPathscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/parser/CSSParserFastPaths.cpp (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/parser/CSSParserFastPaths.cpp 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/parser/CSSParserFastPaths.cpp 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -43,7 +43,6 @@
</span><span class="cx"> #include "StyleColor.h"
</span><span class="cx"> #include "StylePropertyShorthand.h"
</span><span class="cx"> #include "StyleSheetContents.h"
</span><del>-#include "WebKitCSSTransformValue.h"
</del><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><span class="lines">@@ -1064,7 +1063,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> template <typename CharType>
</span><del>-static bool parseTransformTranslateArguments(CharType*& pos, CharType* end, unsigned expectedCount, WebKitCSSTransformValue* transformValue)
</del><ins>+static bool parseTransformTranslateArguments(CharType*& pos, CharType* end, unsigned expectedCount, CSSFunctionValue* transformValue)
</ins><span class="cx"> {
</span><span class="cx"> while (expectedCount) {
</span><span class="cx"> size_t delimiter = WTF::find(pos, end - pos, expectedCount == 1 ? ')' : ',');
</span><span class="lines">@@ -1077,7 +1076,7 @@
</span><span class="cx"> return false;
</span><span class="cx"> if (!number && unit == CSSPrimitiveValue::CSS_NUMBER)
</span><span class="cx"> unit = CSSPrimitiveValue::UnitType::CSS_PX;
</span><del>- if (unit == CSSPrimitiveValue::UnitType::CSS_NUMBER || (unit == CSSPrimitiveValue::UnitType::CSS_PERCENTAGE && (transformValue->operationType() == WebKitCSSTransformValue::TranslateZTransformOperation || (transformValue->operationType() == WebKitCSSTransformValue::Translate3DTransformOperation && expectedCount == 1))))
</del><ins>+ if (unit == CSSPrimitiveValue::UnitType::CSS_NUMBER || (unit == CSSPrimitiveValue::UnitType::CSS_PERCENTAGE && (transformValue->name() == CSSValueTranslatez || (transformValue->name() == CSSValueTranslate3d && expectedCount == 1))))
</ins><span class="cx"> return false;
</span><span class="cx"> transformValue->append(CSSPrimitiveValue::create(number, unit));
</span><span class="cx"> pos += argumentLength + 1;
</span><span class="lines">@@ -1087,7 +1086,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> template <typename CharType>
</span><del>-static bool parseTransformNumberArguments(CharType*& pos, CharType* end, unsigned expectedCount, WebKitCSSTransformValue* transformValue)
</del><ins>+static bool parseTransformNumberArguments(CharType*& pos, CharType* end, unsigned expectedCount, CSSFunctionValue* transformValue)
</ins><span class="cx"> {
</span><span class="cx"> while (expectedCount) {
</span><span class="cx"> size_t delimiter = WTF::find(pos, end - pos, expectedCount == 1 ? ')' : ',');
</span><span class="lines">@@ -1107,9 +1106,8 @@
</span><span class="cx">
</span><span class="cx"> static const int kShortestValidTransformStringLength = 12;
</span><span class="cx">
</span><del>-// FIXME-NEWPARSER: Should use CSSFunctionValue and ditch WebkitCSSTransformValue.
</del><span class="cx"> template <typename CharType>
</span><del>-static RefPtr<WebKitCSSTransformValue> parseSimpleTransformValue(CharType*& pos, CharType* end)
</del><ins>+static RefPtr<CSSFunctionValue> parseSimpleTransformValue(CharType*& pos, CharType* end)
</ins><span class="cx"> {
</span><span class="cx"> if (end - pos < kShortestValidTransformStringLength)
</span><span class="cx"> return nullptr;
</span><span class="lines">@@ -1125,22 +1123,22 @@
</span><span class="cx"> && toASCIILower(pos[8]) == 'e';
</span><span class="cx">
</span><span class="cx"> if (isTranslate) {
</span><del>- WebKitCSSTransformValue::TransformOperationType transformType;
</del><ins>+ CSSValueID transformType;
</ins><span class="cx"> unsigned expectedArgumentCount = 1;
</span><span class="cx"> unsigned argumentStart = 11;
</span><span class="cx"> CharType c9 = toASCIILower(pos[9]);
</span><span class="cx"> if (c9 == 'x' && pos[10] == '(') {
</span><del>- transformType = WebKitCSSTransformValue::TranslateXTransformOperation;
</del><ins>+ transformType = CSSValueTranslatex;
</ins><span class="cx"> } else if (c9 == 'y' && pos[10] == '(') {
</span><del>- transformType = WebKitCSSTransformValue::TranslateYTransformOperation;
</del><ins>+ transformType = CSSValueTranslatey;
</ins><span class="cx"> } else if (c9 == 'z' && pos[10] == '(') {
</span><del>- transformType = WebKitCSSTransformValue::TranslateZTransformOperation;
</del><ins>+ transformType = CSSValueTranslatez;
</ins><span class="cx"> } else if (c9 == '(') {
</span><del>- transformType = WebKitCSSTransformValue::TranslateTransformOperation;
</del><ins>+ transformType = CSSValueTranslate;
</ins><span class="cx"> expectedArgumentCount = 2;
</span><span class="cx"> argumentStart = 10;
</span><span class="cx"> } else if (c9 == '3' && toASCIILower(pos[10]) == 'd' && pos[11] == '(') {
</span><del>- transformType = WebKitCSSTransformValue::Translate3DTransformOperation;
</del><ins>+ transformType = CSSValueTranslate3d;
</ins><span class="cx"> expectedArgumentCount = 3;
</span><span class="cx"> argumentStart = 12;
</span><span class="cx"> } else {
</span><span class="lines">@@ -1147,7 +1145,7 @@
</span><span class="cx"> return nullptr;
</span><span class="cx"> }
</span><span class="cx"> pos += argumentStart;
</span><del>- RefPtr<WebKitCSSTransformValue> transformValue = WebKitCSSTransformValue::create(transformType);
</del><ins>+ RefPtr<CSSFunctionValue> transformValue = CSSFunctionValue::create(transformType);
</ins><span class="cx"> if (!parseTransformTranslateArguments(pos, end, expectedArgumentCount, transformValue.get()))
</span><span class="cx"> return nullptr;
</span><span class="cx"> return transformValue;
</span><span class="lines">@@ -1165,7 +1163,7 @@
</span><span class="cx">
</span><span class="cx"> if (isMatrix3d) {
</span><span class="cx"> pos += 9;
</span><del>- RefPtr<WebKitCSSTransformValue> transformValue = WebKitCSSTransformValue::create(WebKitCSSTransformValue::Matrix3DTransformOperation);
</del><ins>+ RefPtr<CSSFunctionValue> transformValue = CSSFunctionValue::create(CSSValueMatrix3d);
</ins><span class="cx"> if (!parseTransformNumberArguments(pos, end, 16, transformValue.get()))
</span><span class="cx"> return nullptr;
</span><span class="cx"> return transformValue;
</span><span class="lines">@@ -1182,7 +1180,7 @@
</span><span class="cx">
</span><span class="cx"> if (isScale3d) {
</span><span class="cx"> pos += 8;
</span><del>- RefPtr<WebKitCSSTransformValue> transformValue = WebKitCSSTransformValue::create(WebKitCSSTransformValue::Scale3DTransformOperation);
</del><ins>+ RefPtr<CSSFunctionValue> transformValue = CSSFunctionValue::create(CSSValueScale3d);
</ins><span class="cx"> if (!parseTransformNumberArguments(pos, end, 3, transformValue.get()))
</span><span class="cx"> return nullptr;
</span><span class="cx"> return transformValue;
</span><span class="lines">@@ -1251,7 +1249,7 @@
</span><span class="cx"> ++pos;
</span><span class="cx"> if (pos >= end)
</span><span class="cx"> break;
</span><del>- RefPtr<WebKitCSSTransformValue> transformValue = parseSimpleTransformValue(pos, end);
</del><ins>+ RefPtr<CSSFunctionValue> transformValue = parseSimpleTransformValue(pos, end);
</ins><span class="cx"> if (!transformValue)
</span><span class="cx"> return nullptr;
</span><span class="cx"> if (!transformList)
</span></span></pre></div>
<a id="trunkSourceWebCorecssparserCSSPropertyParsercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/parser/CSSPropertyParser.cpp (209804 => 209805)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/parser/CSSPropertyParser.cpp 2016-12-14 16:19:05 UTC (rev 209804)
+++ trunk/Source/WebCore/css/parser/CSSPropertyParser.cpp 2016-12-14 16:34:48 UTC (rev 209805)
</span><span class="lines">@@ -80,7 +80,6 @@
</span><span class="cx"> #include "SVGPathUtilities.h"
</span><span class="cx"> #include "StylePropertyShorthand.h"
</span><span class="cx"> #include "StylePropertyShorthandFunctions.h"
</span><del>-#include "WebKitCSSTransformValue.h"
</del><span class="cx"> #include <bitset>
</span><span class="cx"> #include <memory>
</span><span class="cx"> #include <wtf/text/StringBuilder.h>
</span><span class="lines">@@ -1712,7 +1711,7 @@
</span><span class="cx"> return consumeLineWidth(range, cssParserMode, UnitlessQuirk::Forbid);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-static bool consumeTranslate3d(CSSParserTokenRange& args, CSSParserMode cssParserMode, RefPtr<WebKitCSSTransformValue>& transformValue)
</del><ins>+static bool consumeTranslate3d(CSSParserTokenRange& args, CSSParserMode cssParserMode, RefPtr<CSSFunctionValue>& transformValue)
</ins><span class="cx"> {
</span><span class="cx"> unsigned numberOfArguments = 2;
</span><span class="cx"> RefPtr<CSSValue> parsedValue;
</span><span class="lines">@@ -1731,7 +1730,7 @@
</span><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-static bool consumeNumbers(CSSParserTokenRange& args, RefPtr<WebKitCSSTransformValue>& transformValue, unsigned numberOfArguments)
</del><ins>+static bool consumeNumbers(CSSParserTokenRange& args, RefPtr<CSSFunctionValue>& transformValue, unsigned numberOfArguments)
</ins><span class="cx"> {
</span><span class="cx"> do {
</span><span class="cx"> RefPtr<CSSPrimitiveValue> parsedValue = consumeNumber(args, ValueRangeAll);
</span><span class="lines">@@ -1744,7 +1743,7 @@
</span><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-static bool consumePerspective(CSSParserTokenRange& args, CSSParserMode cssParserMode, RefPtr<WebKitCSSTransformValue>& transformValue)
</del><ins>+static bool consumePerspective(CSSParserTokenRange& args, CSSParserMode cssParserMode, RefPtr<CSSFunctionValue>& transformValue)
</ins><span class="cx"> {
</span><span class="cx"> RefPtr<CSSPrimitiveValue> parsedValue = consumeLength(args, cssParserMode, ValueRangeNonNegative);
</span><span class="cx"> if (!parsedValue) {
</span><span class="lines">@@ -1759,58 +1758,6 @@
</span><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-// FIXME-NEWPARSER: This has no reason to exist once we eliminate WebkitCSSTransformValue in favor
-// of CSSFunctionValue.
-static WebKitCSSTransformValue::TransformOperationType transformOperationForCSSValueID(CSSValueID functionId)
-{
- switch (functionId) {
- case CSSValueRotate:
- return WebKitCSSTransformValue::RotateTransformOperation;
- case CSSValueRotatex:
- return WebKitCSSTransformValue::RotateXTransformOperation;
- case CSSValueRotatey:
- return WebKitCSSTransformValue::RotateYTransformOperation;
- case CSSValueRotatez:
- return WebKitCSSTransformValue::RotateZTransformOperation;
- case CSSValueSkewx:
- return WebKitCSSTransformValue::SkewXTransformOperation;
- case CSSValueSkewy:
- return WebKitCSSTransformValue::SkewYTransformOperation;
- case CSSValueSkew:
- return WebKitCSSTransformValue::SkewTransformOperation;
- case CSSValueScalex:
- return WebKitCSSTransformValue::ScaleXTransformOperation;
- case CSSValueScaley:
- return WebKitCSSTransformValue::ScaleYTransformOperation;
- case CSSValueScalez:
- return WebKitCSSTransformValue::ScaleZTransformOperation;
- case CSSValueScale:
- return WebKitCSSTransformValue::ScaleTransformOperation;
- case CSSValuePerspective:
- return WebKitCSSTransformValue::PerspectiveTransformOperation;
- case CSSValueTranslatex:
- return WebKitCSSTransformValue::TranslateXTransformOperation;
- case CSSValueTranslatey:
- return WebKitCSSTransformValue::TranslateYTransformOperation;
- case CSSValueTranslate:
- return WebKitCSSTransformValue::TranslateTransformOperation;
- case CSSValueTranslatez:
- return WebKitCSSTransformValue::TranslateZTransformOperation;
- case CSSValueMatrix:
- return WebKitCSSTransformValue::MatrixTransformOperation;
- case CSSValueMatrix3d:
- return WebKitCSSTransformValue::Matrix3DTransformOperation;
- case CSSValueScale3d:
- return WebKitCSSTransformValue::Scale3DTransformOperation;
- case CSSValueRotate3d:
- return WebKitCSSTransformValue::Rotate3DTransformOperation;
- case CSSValueTranslate3d:
- return WebKitCSSTransformValue::Translate3DTransformOperation;
- default:
- return WebKitCSSTransformValue::UnknownTransformOperation;
- }
-}
-
</del><span class="cx"> static RefPtr<CSSValue> consumeTransformValue(CSSParserTokenRange& range, CSSParserMode cssParserMode)
</span><span class="cx"> {
</span><span class="cx"> CSSValueID functionId = range.peek().functionId();
</span><span class="lines">@@ -1820,9 +1767,7 @@
</span><span class="cx"> if (args.atEnd())
</span><span class="cx"> return nullptr;
</span><span class="cx">
</span><del>- // FIXME-NEWPARSER: Do we really need WebkitCSSTransformValue? A CSSFunctionValue is good
- // enough and has the CSSValueID as the operation type. Blink has eliminated it.
- RefPtr<WebKitCSSTransformValue> transformValue = WebKitCSSTransformValue::create(transformOperationForCSSValueID(functionId));
</del><ins>+ RefPtr<CSSFunctionValue> transformValue = CSSFunctionValue::create(functionId);
</ins><span class="cx"> RefPtr<CSSValue> parsedValue;
</span><span class="cx"> switch (functionId) {
</span><span class="cx"> case CSSValueRotate:
</span></span></pre>
</div>
</div>
</body>
</html>