From 00183756b972979631c23fe32d37992d1b0f76bb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Emilio=20Cobos=20=C3=81lvarez?= Date: Wed, 24 Jul 2019 00:42:21 +0200 Subject: [PATCH] Bug 1567237 - followup: Fix ESLint complaints in test. CLOSED TREE --- ...scroll_space_no_range_overflow_scroll.html | 21 ++++++++----------- 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/layout/base/tests/test_scroll_space_no_range_overflow_scroll.html b/layout/base/tests/test_scroll_space_no_range_overflow_scroll.html index ed5d911d43d1..a72ea7462968 100644 --- a/layout/base/tests/test_scroll_space_no_range_overflow_scroll.html +++ b/layout/base/tests/test_scroll_space_no_range_overflow_scroll.html @@ -56,17 +56,14 @@ promise_test(async function() { assert_true(scrollable.scrollTopMax > 0, "Should be able to scroll #scrollable"); assert_equals(unscrollable.scrollTopMax, 0, "#unscrollable should not be able to scroll at all (checking scrollTopMax)"); - { - scrollable.focus(); - await doPageDown(scrollable); - assert_not_equals(scrollable.scrollTop, 0, "Should have scrolled when pressing space"); - } - { - unscrollable.focus(); - let rootScrollTop = rootScroller.scrollTop; // Could've scrolled to scroll `scrollable` into view before. - await doPageDown(window); - assert_equals(unscrollable.scrollTop, 0, "Should not be able to scroll the unscrollable div"); - assert_not_equals(rootScroller.scrollTop, rootScrollTop, "Root should be able to scroll"); - } + scrollable.focus(); + await doPageDown(scrollable); + assert_not_equals(scrollable.scrollTop, 0, "Should have scrolled when pressing space"); + + unscrollable.focus(); + let rootScrollTop = rootScroller.scrollTop; // Could've scrolled to scroll `scrollable` into view before. + await doPageDown(window); + assert_equals(unscrollable.scrollTop, 0, "Should not be able to scroll the unscrollable div"); + assert_not_equals(rootScroller.scrollTop, rootScrollTop, "Root should be able to scroll"); }, "Overflow scroll without range doesn't block scrolling of the main document");