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

Unified Diff: third_party/WebKit/LayoutTests/compositing/change-preferCompositingToLCDText-setting.html

Issue 2196363002: Modify LayoutTests to prep for compositing solid background scollers (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Revert change to fixed-position-in-fixed-overflow.html Created 4 years, 4 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/change-preferCompositingToLCDText-setting.html
diff --git a/third_party/WebKit/LayoutTests/compositing/change-compositing-settings.html b/third_party/WebKit/LayoutTests/compositing/change-preferCompositingToLCDText-setting.html
similarity index 58%
rename from third_party/WebKit/LayoutTests/compositing/change-compositing-settings.html
rename to third_party/WebKit/LayoutTests/compositing/change-preferCompositingToLCDText-setting.html
index 48c0a63dbb2d411f841ff40056478a0b9022bf74..816565142a5b959f5fde83b39484285fcb0efbbc 100644
--- a/third_party/WebKit/LayoutTests/compositing/change-compositing-settings.html
+++ b/third_party/WebKit/LayoutTests/compositing/change-preferCompositingToLCDText-setting.html
@@ -1,24 +1,24 @@
<!DOCTYPE HTML>
-<script src="../resources/js-test.js"></script>
+<script src="../resources/testharness.js"></script>
+<script src="../resources/testharnessreport.js"></script>
<style>
#scroller {
overflow: scroll;
height: 300px;
width: 300px;
- background-color: red;
+ background-color: rgba(255,0,0,0.5); /* Force non-promotion due to background opacity */;
}
#scrolled {
height: 1000px;
width: 250px;
- background-color: green;
+ background-color: rgba(0,128,0,0.5); /* Force non-promotion due to background opacity */;
}
</style>
<script>
-jsTestIsAsync = true;
-description('This test checks that changes to the compositor\'s settings are applied correctly.');
+var t = async_test("Check that the preferCompositingToLCDText flag results in compositing");
function isUsingCompositedScrolling(layer) {
layer = layer || JSON.parse(window.internals.layerTreeAsText(document));
@@ -34,21 +34,18 @@ function isUsingCompositedScrolling(layer) {
}
window.onload = function() {
- if (!window.internals) {
- testFailed('This test requires window.internals');
- return;
- }
+ t.step(function() {
+ assert_not_equals(window.internals, null, 'This test requires window.internals');
+
window.internals.settings.setPreferCompositingToLCDTextEnabled(false);
var stateBefore = isUsingCompositedScrolling();
window.internals.settings.setPreferCompositingToLCDTextEnabled(true);
var stateAfter = isUsingCompositedScrolling();
- if (!stateBefore && stateAfter) {
- testPassed('Composited scrolling became enabled.');
- } else {
- testFailed('Composited scrolling did not become enabled: ' + stateBefore + ", " + stateAfter);
- }
- finishJSTest();
+ assert_false(stateBefore, 'Composited scrolling began disabled');
+ assert_true(stateAfter, 'Composited scrolling became enabled');
+ t.done();
+ });
};
</script>

Powered by Google App Engine
This is Rietveld 408576698