Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-07-23 15:23:42 +01:00
Code Issues Projects Releases Packages Wiki Activity

style: Add some more tests for disallowed rootMargin values.

Browse source
This commit is contained in:
Cameron McCormack 2017-10-17 15:08:52 +08:00 • committed by Emilio Cobos Álvarez
parent ac74cd57a2
commit 893415c85a
No known key found for this signature in database
GPG key ID: 056B727BB9C1027C
3 changed files with 19 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
components/style/values/specified/gecko.rs
View file

@ -81,6 +81,11 @@ impl ToNsCssValue for PixelOrPercentage {
} }
/// The value of an IntersectionObserver's rootMargin property. /// The value of an IntersectionObserver's rootMargin property.
///
/// Only bare px or percentage values are allowed. Other length units and
/// calc() values are not allowed.
///
/// https://w3c.github.io/IntersectionObserver/#parse-a-root-margin
pub struct IntersectionObserverRootMargin(pub Rect<PixelOrPercentage>); pub struct IntersectionObserverRootMargin(pub Rect<PixelOrPercentage>);
impl Parse for IntersectionObserverRootMargin { impl Parse for IntersectionObserverRootMargin {

2
tests/wpt/metadata/MANIFEST.json
View file

@ -557079,7 +557079,7 @@
"testharness" "testharness"
], ],
"intersection-observer/observer-exceptions.html": [ "intersection-observer/observer-exceptions.html": [
"28ccc6905713894b43033e30949170439215bf2e", "85d5416475091c1020e0ff92b15f841bf58c606e",
"testharness" "testharness"
], ],
"intersection-observer/observer-in-iframe.html": [ "intersection-observer/observer-in-iframe.html": [

14
tests/wpt/web-platform-tests/intersection-observer/observer-exceptions.html
View file

@ -31,7 +31,19 @@ test(function () {
assert_throws("SYNTAX_ERR", function() { assert_throws("SYNTAX_ERR", function() {
new IntersectionObserver(e => {}, {rootMargin: "auto"}) new IntersectionObserver(e => {}, {rootMargin: "auto"})
}) })
}, 'IntersectionObserver constructor width { rootMargin: "auto" }'); }, 'IntersectionObserver constructor with { rootMargin: "auto" }');
test(function () {
assert_throws("SYNTAX_ERR", function() {
new IntersectionObserver(e => {}, {rootMargin: "calc(1px + 2px)"})
})
}, 'IntersectionObserver constructor with { rootMargin: "calc(1px + 2px)" }');
test(function () {
assert_throws("SYNTAX_ERR", function() {
new IntersectionObserver(e => {}, {rootMargin: "1px !important"})
})
}, 'IntersectionObserver constructor with { rootMargin: "1px !important" }');
test(function () { test(function () {
assert_throws("SYNTAX_ERR", function() { assert_throws("SYNTAX_ERR", function() {