aboutsummaryrefslogtreecommitdiffstats
path: root/components/script/dom/webglframebuffer.rs
diff options
context:
space:
mode:
authorbors-servo <lbergstrom+bors@mozilla.com>2017-08-16 05:46:56 -0500
committerGitHub <noreply@github.com>2017-08-16 05:46:56 -0500
commit75cae287281ef553270194d7214a6f3ae8cb4930 (patch)
treece144624a94459ef02a8f42424cac05f48a08845 /components/script/dom/webglframebuffer.rs
parent467316e35bb494b9e9b2b4c480b3eda17c3a91b9 (diff)
parent9ecb0aa7fa890bf872c0310d213efb0587721c7e (diff)
downloadservo-75cae287281ef553270194d7214a6f3ae8cb4930.tar.gz
servo-75cae287281ef553270194d7214a6f3ae8cb4930.zip
Auto merge of #18103 - birtles:drop-fill-none-interpolation, r=hiro
Don't allow interpolating 'fill:none' with 'fill:none' From [Gecko bug 1390364](https://bugzilla.mozilla.org/show_bug.cgi?id=1390364). In SMIL we don't expect the 'none' value of the 'fill' property to be additive and hence the following animation should have no effect: ```svg <rect width="100" height="100" y="100" fill="blue"> <animate attributeName="fill" dur="3s" from="red" by="none"/> </rect> ``` Although SMIL doesn't make this entirely clear, [it says](https://www.w3.org/TR/smil-animation/#AnimFuncValues) that "by animation" and "from-by animation" may only be used "with attributes that support addition (e.g. most numeric attributes)" and [SVG says](https://www.w3.org/TR/SVG11/animate.html#AnimationAttributesAndProperties) that `<paint>`s are "only additive if each value can be converted to an RGB color". As a result, the animation above should have no effect. By extrapolation, animating from 'none' by 'none' should also have no effect: ```svg <rect width="100" height="100" y="100" fill="blue"> <animate attributeName="fill" dur="3s" from="none" by="none"/> </rect> ``` However, in Servo's interpolation of `<paint>`s we special case the interpolation and addition of 'none' such that if both values are 'none' it is allowed. We should disallow this in order to produce the expected behavior and in order to match Gecko's behavior. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18103) <!-- Reviewable:end -->
Diffstat (limited to 'components/script/dom/webglframebuffer.rs')
0 files changed, 0 insertions, 0 deletions