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

Unified Diff: LayoutTests/compositing/overflow/ignore-main-thread-scroll-reasons-when-main-frame-not-scrollable.html

Issue 135853004: Partially revert "should-scroll-on-main-thread reasons if the main thread is not scrollable" (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 11 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 | LayoutTests/compositing/overflow/ignore-main-thread-scroll-reasons-when-main-frame-not-scrollable-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/compositing/overflow/ignore-main-thread-scroll-reasons-when-main-frame-not-scrollable.html
diff --git a/LayoutTests/compositing/overflow/ignore-main-thread-scroll-reasons-when-main-frame-not-scrollable.html b/LayoutTests/compositing/overflow/ignore-main-thread-scroll-reasons-when-main-frame-not-scrollable.html
deleted file mode 100644
index 8e85e587cbc31ad5d0625c9cd141d59695eb44aa..0000000000000000000000000000000000000000
--- a/LayoutTests/compositing/overflow/ignore-main-thread-scroll-reasons-when-main-frame-not-scrollable.html
+++ /dev/null
@@ -1,117 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<style>
-.non-layer-viewport-constrained {
- position: fixed;
-}
-
-.box {
- width: 50px;
- height: 50px;
-}
-
-.container {
- overflow: scroll;
- border: 1px solid black;
-}
-
-.container-element {
- position: relative;
- background-color: blue;
- margin: 10px;
-}
-
-#sibling {
- top: 220px;
- left: 10px;
- background-color: green;
-}
-
-#parent {
- position: absolute;
- left: 10px;
- top: 10px;
- width: 200px;
- height: 200px;
-}
-
-#description {
- position: absolute;
- top: 300px;
- left: 10px;
-}
-
-#console {
- position: absolute;
- top: 290px;
- left: 10px;
-}
-
-</style>
-<script src="../../resources/js-test.js"></script>
-<script>
-if (window.internals) {
- // Our fixed position element must not be composited; this is a trigger for
- // forcing main thread compositing.
- window.internals.settings.setForceCompositingMode(true);
- window.internals.settings.setAcceleratedCompositingForFixedPositionEnabled(false);
- window.internals.settings.setAcceleratedCompositingForOverflowScrollEnabled(true);
-}
-
-window.jsTestIsAsync = true;
-
-function shouldIgnore() {
- if (window.internals) {
- var reasons = internals.mainThreadScrollingReasons(document);
- if (reasons === '')
- testPassed("No main thread scrolling reasons.");
- else
- testFailed("Should have found no main thread scrolling reasons. " +
- "Instead, they were '" + reasons + "'.");
- } else {
- debug("This test requires window.internals.");
- description("This test ensures that when the main frame cannot be " +
- "scrolled, we ignore reasons for scrolling on the main " +
- "thread that only matter when the main frame scrolls");
- }
-}
-
-function shouldNotIgnore() {
- document.body.style.height = "3000px";
- document.body.offsetTop;
-
- if (window.internals) {
- var reasons = internals.mainThreadScrollingReasons(document);
- if (reasons === '')
- testFailed("Should have found main thread scrolling reasons, but didn't.");
- else
- testPassed("Found main thread scrolling reasons: " + reasons + ".");
- } else {
- debug("This test requires window.internals.");
- description("This test ensures that when the main frame cannot be " +
- "scrolled, we ignore reasons for scrolling on the main " +
- "thread that only matter when the main frame scrolls");
- }
-}
-
-function runTests() {
- shouldIgnore();
- shouldNotIgnore();
- finishJSTest();
-}
-
-window.onload = runTests;
-</script>
-</head>
-<body>
- <div class="non-layer-viewport-constrained box" id="sibling"></div>
- <div class="container" id="parent">
- <div class="box container-element"></div>
- <div class="box container-element"></div>
- <div class="box container-element"></div>
- <div class="box container-element"></div>
- <div class="box container-element"></div>
- </div>
-</body>
-</html>
« no previous file with comments | « no previous file | LayoutTests/compositing/overflow/ignore-main-thread-scroll-reasons-when-main-frame-not-scrollable-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698