Chromium Code Reviews Chromium Code Reviews
Issue 2262373003:
Clip PaintLayerScrollableArea::scrollIntoView return to layer bounds (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: third_party/WebKit/LayoutTests/fast/scrolling/scroll-into-view-small-size-ancestor.html |
| diff --git a/third_party/WebKit/LayoutTests/fast/scrolling/scroll-into-view-small-size-ancestor.html b/third_party/WebKit/LayoutTests/fast/scrolling/scroll-into-view-small-size-ancestor.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..93cdfefe95db282fec7059d1b825ae315ca3676b |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/fast/scrolling/scroll-into-view-small-size-ancestor.html |
| @@ -0,0 +1,32 @@ |
| +<!DOCTYPE html> |
| +<script src='../../resources/testharness.js'></script> |
| +<script src='../../resources/testharnessreport.js'></script> |
| +<body> |
|
bokan
2016/08/26 16:12:14
Nit: remove <body>
|
| +<div id = 'space1' style = 'height: 200px'> </div> |
|
bokan
2016/08/26 16:12:14
If you're just using this for space, there's bette
|
| +<div id = 'container' style = 'height: 300px; overflow: auto;' > |
|
bokan
2016/08/26 16:12:14
Nit: no spaces between attribute, =, and value.
i
|
| +<select id="content" multiple="multiple" style = 'height: 600px'> |
|
bokan
2016/08/26 16:12:14
Nit: Please indent the <select>
|
| + <option>I</option> |
| + <option>am</option> |
| + <option>already</option> |
| + <option>in</option> |
| + <option>the</option> |
| + <option>view</option> |
| + <option>.</option> |
| + <option>Do</option> |
| + <option>not</option> |
| + <option>scroll</option> |
| + <option>me</option> |
| + <option>up</option> |
| + <option>.</option> |
| +</select> |
| +</div> |
| +<div id = 'space2' style = 'height: 600px'></div> |
| +</body> |
| + |
| +<script> |
| +test(function(t) { |
| + var content = document.getElementById('content'); |
| + content.focus(); |
| + assert_equals(window.scrollY, 0); |
| +}, 'Tests that when the parent is already in the view of page, the page will not scroll the parent even when the child is not completely in the view.'); |
| +</script> |
