third_party/WebKit/LayoutTests/scrollingcoordinator/non-fast-scrollable-region-transformed-iframe.html - Issue 2657183002: Use full mapLocalToAncestor machinery in ScrollingCoordinator.

Unified Diff: third_party/WebKit/LayoutTests/scrollingcoordinator/non-fast-scrollable-region-transformed-iframe.html

Issue 2657183002: Use full mapLocalToAncestor machinery in ScrollingCoordinator. (Closed)

Patch Set: test expectations Created 3 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

« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/scrollingcoordinator/non-fast-scrollable-region-transformed-iframe-expected.txt » ('j') | third_party/WebKit/Source/core/frame/FrameView.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/LayoutTests/scrollingcoordinator/non-fast-scrollable-region-transformed-iframe.html

diff --git a/third_party/WebKit/LayoutTests/scrollingcoordinator/non-fast-scrollable-region-transformed-iframe.html b/third_party/WebKit/LayoutTests/scrollingcoordinator/non-fast-scrollable-region-transformed-iframe.html

index 3db9d602095d9f7f23878feb900b2a1171385f14..6239e285ac4eb99e8bc1a81735d0127c2268402a 100644

--- a/third_party/WebKit/LayoutTests/scrollingcoordinator/non-fast-scrollable-region-transformed-iframe.html

+++ b/third_party/WebKit/LayoutTests/scrollingcoordinator/non-fast-scrollable-region-transformed-iframe.html

@@ -4,16 +4,20 @@

position: absolute;

left: 200px;

top: 200px;

- height: 120px;

- width: 120px;

+ height: 200px;

+ width: 200px;

padding: 10px;

border: none;

transform: scale(2);

}

+ .spacer {

+ height: 2000px;

+ }

</style>

-<iframe src="data:text/html;charset=utf-8,<html><body style='width:1000px; height:1000px;'><p style='font-size: 6px'>Should be covered by a green overlay.</p></body></html>"></iframe>

+<iframe src="resources/non-fast-scrollable-region-transformed-iframe-subframe.html"></iframe>

skobes 2017/01/27 18:37:25 Generally I find srcdoc to be nicer than adding se

szager1 2017/01/30 18:25:41 I converted it to srcdoc, but I don't really feel

skobes 2017/01/30 18:59:53 Yes the " is a bit gross but I find it easier

+<div class="spacer"></div>

@@ -27,7 +31,7 @@

runAfterLayoutAndPaint(function(){

nonFastScrollableRects = window.internals.nonFastScrollableRects(document);

shouldBe('nonFastScrollableRects.length', '1');

- shouldBeEqualToString('rectToString(nonFastScrollableRects[0])', '[150, 150, 240, 240]');

+ shouldBeEqualToString('rectToString(nonFastScrollableRects[0])', '[146, 146, 240, 240]');

drawNonFastScrollableRegionOverlays();