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

Don't try to unwrap the result of requestAnimationFrame callback

Browse source
This commit is contained in:
David Zbarsky 2015-07-31 16:27:35 -04:00
parent 5bab439ab6
commit b77698a649
6 changed files with 39 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
components/script/dom/window.rs
View file

@ -531,7 +531,8 @@ impl<'a> WindowMethods for &'a Window {
let callback = move |now: f64| {
// TODO: @jdm The spec says that any exceptions should be suppressed;
callback.Call__(Finite::wrap(now), ExceptionHandling::Report).unwrap();
// https://github.com/servo/servo/issues/6928
let _ = callback.Call__(Finite::wrap(now), ExceptionHandling::Report);
};
doc.r().request_animation_frame(Box::new(callback))

1
tests/wpt/metadata/html/dom/interfaces.html.ini
View file

@ -8978,3 +8978,4 @@
[HTMLOptionElement interface: new Option() must inherit property "index" with the proper type (7)]
expected: FAIL

1
tests/wpt/metadata/websockets/interfaces.html.ini
View file

@ -56,3 +56,4 @@
[CloseEvent interface: existence and properties of interface prototype object]
expected: FAIL

6
tests/wpt/mozilla/meta/MANIFEST.json
View file

@ -773,6 +773,12 @@
"url": "/_mozilla/mozilla/window_requestAnimationFrame.html"
}
],
"mozilla/window_requestAnimationFrame2.html": [
{
"path": "mozilla/window_requestAnimationFrame2.html",
"url": "/_mozilla/mozilla/window_requestAnimationFrame2.html"
}
],
"mozilla/window_setInterval.html": [
{
"path": "mozilla/window_setInterval.html",

6
tests/wpt/mozilla/meta/mozilla/window_requestAnimationFrame2.html.ini Normal file
View file

@ -0,0 +1,6 @@
[window_requestAnimationFrame2.html]
type: testharness
expected: TIMEOUT
[Test throwing an error inside requestAnimationFrame callback]
expected: NOTRUN

23
tests/wpt/mozilla/tests/mozilla/window_requestAnimationFrame2.html Normal file
View file

@ -0,0 +1,23 @@
<html>
<head>
<title>Test throwing an error inside requestAnimationFrame callback</title>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
</head>
<body>
<script>
var stepCalled = false;
function step() {
if (stepCalled) {
setTimeout(done, 0);
} else {
stepCalled = true;
window.requestAnimationFrame(step);
}
throw new Error();
}
window.requestAnimationFrame(step);
assert_equals(true, true, "rAF should not throw errors");
</script>
</body>
</html>