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

Unified Diff: third_party/WebKit/LayoutTests/css3/flexbox/overflow-keep-scrollpos.html

Issue 2130543003: Convert some flexbox tests to testharness.js (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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/css3/flexbox/overflow-keep-scrollpos.html
diff --git a/third_party/WebKit/LayoutTests/css3/flexbox/overflow-keep-scrollpos.html b/third_party/WebKit/LayoutTests/css3/flexbox/overflow-keep-scrollpos.html
index 117db779beac2d0e07c2d868ed701cd7c5e30636..19ab2df0e1e7e643ec2c37f2b47ee36df1450af0 100644
--- a/third_party/WebKit/LayoutTests/css3/flexbox/overflow-keep-scrollpos.html
+++ b/third_party/WebKit/LayoutTests/css3/flexbox/overflow-keep-scrollpos.html
@@ -1,7 +1,8 @@
<!DOCTYPE html>
<html>
<head>
-<script src="../../resources/js-test.js"></script>
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
<style>
div.flexbox {
display: flex;
@@ -21,12 +22,13 @@ div.flex {
function runTest() {
document.querySelector('.overflow-auto').scrollTop = 50;
document.querySelector('#sidebar').innerHTML = '';
- shouldBe("document.querySelector('.overflow-auto').scrollTop", "50");
+ assert_equals(document.querySelector('.overflow-auto').scrollTop, 50);
}
</script>
</head>
-<body onload="runTest()">
+<body onload="test(runTest, 'Scroll position is correct')">
+<div id=log></div>
<div class="container flexbox">
<div class="flex overflow-auto flexbox">
<div style="height: 400px"></div>

Powered by Google App Engine
This is Rietveld 408576698