Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(210)

Unified Diff: LayoutTests/animations/interpolation/svg-baseline-shift-interpolation.html

Issue 956553004: Use Length for baselineShiftValue in SVGLayoutStyle (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: A few more TEs. Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/animations/interpolation/svg-baseline-shift-interpolation.html
diff --git a/LayoutTests/animations/interpolation/svg-baseline-shift-interpolation.html b/LayoutTests/animations/interpolation/svg-baseline-shift-interpolation.html
index 17bd8e911a65f32c7156dcf62cd531d92cb6888a..98e916bc8b72543da3baf22e3e00eee42ff0a3d1 100644
--- a/LayoutTests/animations/interpolation/svg-baseline-shift-interpolation.html
+++ b/LayoutTests/animations/interpolation/svg-baseline-shift-interpolation.html
@@ -26,7 +26,7 @@ svg {
We have to specify the target class on the text element because numerical
values of baseline-shift are not inherited properly. See crbug.com/304006.
-->
- <text x="0" y="25" font-size="20" class="target">
+ <text x="0" y="25" font-size="10" class="target">
<tspan>T</tspan>T
</text>
</svg>
@@ -112,26 +112,26 @@ assertInterpolation({
assertInterpolation({
property: 'baseline-shift',
from: '10px',
- to: '20em'
+ to: '21em'
}, [
- {at: -0.25, is: '10px'},
+ {at: -0.25, is: '-40px'},
{at: 0, is: '10px'},
- {at: 0.25, is: '10px'},
- {at: 0.75, is: '20em'},
- {at: 1, is: '20em'},
- {at: 1.25, is: '20em'},
+ {at: 0.25, is: '60px'},
+ {at: 0.75, is: '160px'},
+ {at: 1, is: '210px'},
+ {at: 1.25, is: '260px'},
]);
assertInterpolation({
property: 'baseline-shift',
from: 'inherit',
- to: '40em'
+ to: '33em'
}, [
- {at: -0.25, is: '30px'},
+ {at: -0.25, is: '-45px'},
{at: 0, is: '30px'},
- {at: 0.25, is: '30px'},
- {at: 0.75, is: '40em'},
- {at: 1, is: '40em'},
- {at: 1.25, is: '40em'},
+ {at: 0.25, is: '105px'},
+ {at: 0.75, is: '255px'},
+ {at: 1, is: '330px'},
+ {at: 1.25, is: '405px'},
]);
</script>

Powered by Google App Engine
This is Rietveld 408576698