-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
View-transitions should only be allowed when page is visible
- Skip the transition if document becomes hidden - Disallow oubound cross-document view transitions when document is hidden - Skip inbound cross-document view transition if document becomes hidden - When calling startViewTransition() in a hidden document, start is as skipped. Based on following CSS resolution: w3c/csswg-drafts#9543 (comment) Bug: 329302331 Change-Id: Ieef3238240b35c5d7f7f819793f65f71d7d75037
- Loading branch information
1 parent
525ebf0
commit 4c350ec
Showing
1 changed file
with
73 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,73 @@ | ||
<!DOCTYPE html> | ||
<html> | ||
|
||
<head> | ||
<title>View transitions: Transition in a hidden page</title> | ||
<link rel="help" href="https://drafts.csswg.org/css-view-transitions-1/"> | ||
<meta name="timeout" content="long"> | ||
<script src="/resources/testharness.js"></script> | ||
<script src="/resources/testharnessreport.js"></script> | ||
<script src="/resources/testdriver.js"></script> | ||
<script src="/resources/testdriver-vendor.js"></script> | ||
<script src="/page-visibility/resources/window_state_context.js"></script> | ||
<style> | ||
::view-transition-group(*) { | ||
animation-duration: 5s; | ||
} | ||
</style> | ||
<body> | ||
<script> | ||
promise_test(async t => { | ||
assert_implements(document.startViewTransition, "Missing document.startViewTransition"); | ||
|
||
const wsc = window_state_context(t); | ||
await wsc.minimize(); | ||
assert_true(document.hidden); | ||
const transition = document.startViewTransition(); | ||
await wsc.restore(); | ||
const [readyResult] = await Promise.allSettled([transition.ready]); | ||
assert_equals(readyResult.status, "rejected"); | ||
}, "A view transition should be immediately skipped if started when document is hidden"); | ||
|
||
promise_test(async t => { | ||
assert_implements(document.startViewTransition, "Missing document.startViewTransition"); | ||
|
||
const wsc = window_state_context(t); | ||
const transition = document.startViewTransition(async () => { | ||
await wsc.minimize(); | ||
}); | ||
|
||
let event_fired = false; | ||
|
||
window.addEventListener("visibilitychange", () => { | ||
if (document.hidden) | ||
event_fired = true; | ||
}); | ||
|
||
await wsc.restore(); | ||
|
||
const [readyResult] = await Promise.allSettled([transition.ready]); | ||
await transition.finished; | ||
assert_equals(readyResult.status, "rejected"); | ||
assert_true(event_fired, "visibilitychange event should fire before skipping the transition"); | ||
}, "A view transition should be skipped when a document becomes hidden while processing update callback"); | ||
|
||
promise_test(async t => { | ||
assert_implements(document.startViewTransition, "Missing document.startViewTransition"); | ||
assert_false(document.hidden); | ||
const wsc = window_state_context(t); | ||
const transition = document.startViewTransition(() => { }); | ||
await transition.ready; | ||
await new Promise(resolve => requestAnimationFrame(resolve)); | ||
await wsc.minimize(); | ||
const result = await new Promise(resolve => { | ||
transition.finished.then(() => resolve("finished")); | ||
t.step_timeout(() => resolve("timeout"), 1000); | ||
}); | ||
|
||
assert_equals(result, "finished"); | ||
}, "A view transition should be skipped when a document becomes hidden while animating"); | ||
</script> | ||
</body> | ||
|
||
</html> |