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

Unified Diff: third_party/WebKit/LayoutTests/animations/filter-responsive-neutral-keyframe.html

Issue 2309413002: Deflake virtual/threaded/animations/filter-responsive-neutral-keyframe.html (Closed)
Patch Set: Remove moar flakiness sources Created 4 years, 3 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/animations/filter-responsive-neutral-keyframe.html
diff --git a/third_party/WebKit/LayoutTests/animations/filter-responsive-neutral-keyframe.html b/third_party/WebKit/LayoutTests/animations/filter-responsive-neutral-keyframe.html
index 849866b9a6e3dd101ac2c2f1e5aec87d43c5787a..782192b0f1c81aea0b87d19c3954592b67750fd4 100644
--- a/third_party/WebKit/LayoutTests/animations/filter-responsive-neutral-keyframe.html
+++ b/third_party/WebKit/LayoutTests/animations/filter-responsive-neutral-keyframe.html
@@ -14,7 +14,7 @@ div {
background: green;
}
@keyframes test {
- to { -webkit-filter: opacity(0.5); }
+ to { -webkit-filter: opacity(0); }
}
</style>
<div id="green"></div>
@@ -23,13 +23,14 @@ div {
if (window.testRunner)
testRunner.waitUntilDone();
-requestAnimationFrame(() => {
- requestAnimationFrame(() => {
- target.style.webkitFilter = 'opacity(0)';
- requestAnimationFrame(() => {
- if (window.testRunner)
- testRunner.notifyDone();
- });
- });
+function waitForCompositor() {
+ return target.animate({transform: ['none', 'none']}, 1).ready;
+}
+
+waitForCompositor().then(() => {
+ target.style.webkitFilter = 'opacity(0)';
+}).then(() => waitForCompositor()).then(() => {
+ if (window.testRunner)
+ testRunner.notifyDone();
});
</script>
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698