LayoutTests/fast/overflow/overflow-y-scroll.html - Issue 648913002: Clean up vestiges of ScrollView.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: LayoutTests/fast/overflow/overflow-y-scroll.html

Issue 648913002: Clean up vestiges of ScrollView. (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Created 6 years, 2 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:

Index: LayoutTests/fast/overflow/overflow-y-scroll.html

diff --git a/LayoutTests/fast/overflow/overflow-y-scroll.html b/LayoutTests/fast/overflow/overflow-y-scroll.html

index 179c23379cbb8f2a4c631e4a4e6f56293a39d9a6..73e66538e6b5a4a343a772826df3145eba102f00 100644

--- a/LayoutTests/fast/overflow/overflow-y-scroll.html

+++ b/LayoutTests/fast/overflow/overflow-y-scroll.html

@@ -7,7 +7,7 @@

function inject()

{

- var content = '<html><head><style>' +

+ var content = '<html><head><style>' +

'html,body { margin: 0; overflow: hidden; height: 100%; width: 100% } ' +

'html.scroll { overflow-x: auto; overflow-y: scroll; } ' +

'</style><body><div id="offender" style="height: 900px;"></div></body></html>';

@@ -18,7 +18,7 @@

doc.write(content);

doc.close();

doc.getElementsByTagName("html")[0].className = "scroll";

- // tickle ScrollView::updateScrollbars()

+ // tickle FrameView::updateScrollbars()

doc.getElementById("offender").scrollIntoView(false);

// trigger resize

injectee.style.width = "400px";

@@ -29,7 +29,7 @@

</head>

<body>

<p>

- Tests <kbd>ScrollView</kbd>'s behavior when only one of the scrollbars is made <kbd>ScrollbarAlwaysOn</kbd> dynamically.

+ Tests <kbd>FrameView</kbd>'s behavior when only one of the scrollbars is made <kbd>ScrollbarAlwaysOn</kbd> dynamically.

The correct behavior is not to remove this scrollbar, which is what this test checks for.

</p>