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

Unified Diff: LayoutTests/svg/css/baseline-shift-inherit.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/svg/css/baseline-shift-inherit.html
diff --git a/LayoutTests/svg/css/baseline-shift-inherit.html b/LayoutTests/svg/css/baseline-shift-inherit.html
index 94e6a85cc2494e6f79ca051d33f50c4734bb9fea..021aee6d39ec3c78f846843dc952ca7f719d67c8 100644
--- a/LayoutTests/svg/css/baseline-shift-inherit.html
+++ b/LayoutTests/svg/css/baseline-shift-inherit.html
@@ -12,10 +12,16 @@ setup(function() {
window.svg = document.querySelector('svg');
window.textElement = document.querySelector('text');
});
-['baseline', 'sub', 'super', '10px', '100%'].forEach(function(item) {
+[
+ {value: 'baseline', expected: '0px'},
+ {value: 'sub'},
+ {value: 'super'},
+ {value: '10px'},
+ {value: '100%'}
+].forEach(function(item) {
test(function() {
- svg.style.setProperty('baseline-shift', item);
- assert_equals(getComputedStyle(textElement).baselineShift, item);
+ svg.style.setProperty('baseline-shift', item.value);
+ assert_equals(getComputedStyle(textElement).baselineShift, item.expected || item.value);
}, 'Inheritance of the baseline-shift property - ' + item + '.');
});
</script>

Powered by Google App Engine
This is Rietveld 408576698