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

Unified Diff: third_party/WebKit/LayoutTests/compositing/overflow/scroller-with-border-radius.html

Issue 2859483006: cc: Enable composited border-radius scrolling.
Patch Set: test expectation Created 3 years, 6 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: third_party/WebKit/LayoutTests/compositing/overflow/scroller-with-border-radius.html
diff --git a/third_party/WebKit/LayoutTests/compositing/overflow/scroller-with-border-radius.html b/third_party/WebKit/LayoutTests/compositing/overflow/scroller-with-border-radius.html
deleted file mode 100644
index 5729ca0893d89d70569c11ef67ee3c06fdc89bd5..0000000000000000000000000000000000000000
--- a/third_party/WebKit/LayoutTests/compositing/overflow/scroller-with-border-radius.html
+++ /dev/null
@@ -1,65 +0,0 @@
-<!DOCTYPE HTML>
-<script src="resources/composited-scroll.js"></script>
-<style>
-#scroller {
- overflow: scroll;
- height: 300px;
- width: 300px;
- background-color: red;
-}
-
-#scrolled {
- height: 1000px;
- width: 250px;
- background-color: green;
-}
-
-#fixed {
- position: fixed;
- height: 100px;
- width: 100px;
- background-color: green;
- top: 400px;
- left: 100px;
-}
-</style>
-<div id="scroller">
- <div id="scrolled"></div>
- <div id="fixed"></div>
-</div>
-<script>
-if (window.internals)
- window.internals.settings.setPreferCompositingToLCDTextEnabled(true);
-
-if (window.testRunner) {
- window.testRunner.dumpAsText();
- window.testRunner.waitUntilDone();
-}
-
-var result = "";
-
-onload = function() {
- if (window.internals) {
- result += "No border radius (should be using composited scrolling): ";
- if (isUsingCompositedScrolling(JSON.parse(window.internals.layerTreeAsText(document))))
- result += "Pass.\n";
- else
- result += "Fail.\n"
- }
- document.getElementById("scroller").style.borderRadius = '5px';
- requestAnimationFrame(function() {
- if (window.internals) {
- result += "Has border radius (should not be using composited scrolling): ";
chrishtr 2017/06/16 22:34:33 Instead of deleting, modify the test to expect com
sunxd 2017/06/20 14:36:10 Done.
- if (!isUsingCompositedScrolling(JSON.parse(window.internals.layerTreeAsText(document))))
- result += "Pass.\n";
- else
- result += "Fail.\n"
- }
-
- if (window.testRunner) {
- window.testRunner.setCustomTextOutput(result);
- window.testRunner.notifyDone();
- }
- });
-};
-</script>

Powered by Google App Engine
This is Rietveld 408576698