Bug 164855

Summary: [iOS WK2] When zoomed in and panning on pages with fixed bars, parts of the bars are sometimes missing
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Layout and RenderingAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: dino, simon.fraser, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 164905    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
sam: review+, buildbot: commit-queue-
Archive of layout-test-results from ews125 for ios-simulator-wk2 none

Description Simon Fraser (smfr) 2016-11-16 20:51:00 PST
On a page like http://smfr.org/misc/four-bars-shadows.html, if you zoom in and pan around, bits of the bars are sometimes missing.

This happens when they get into tiled mode, but we fail to update the tiles during scrolling.

This is related to https://trac.webkit.org/changeset/186155.
Comment 1 Simon Fraser (smfr) 2016-11-16 21:01:47 PST
Created attachment 295023 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2016-11-16 21:04:14 PST
<rdar://problem/29303891>
Comment 3 Radar WebKit Bug Importer 2016-11-16 21:04:14 PST
<rdar://problem/29303889>
Comment 4 Simon Fraser (smfr) 2016-11-16 21:15:20 PST
Created attachment 295024 [details]
Patch
Comment 5 Simon Fraser (smfr) 2016-11-16 22:30:11 PST
Created attachment 295030 [details]
Patch
Comment 6 Build Bot 2016-11-16 23:49:02 PST
Comment on attachment 295030 [details]
Patch

Attachment 295030 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/2529833

New failing tests:
scrollingcoordinator/ios/sync-layer-positions-after-scroll.html
Comment 7 Build Bot 2016-11-16 23:49:06 PST
Created attachment 295036 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6
Comment 8 Sam Weinig 2016-11-17 11:02:10 PST
Comment on attachment 295030 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=295030&action=review

> Source/WebCore/platform/graphics/GraphicsLayer.h:605
> +    // If set, overrides m_position. Only used for coverage computation.
> +    Optional<FloatPoint> m_approximatePosition;

This could use a slightly more in depth comment, as the term approximate is a bit scary.
Comment 9 Simon Fraser (smfr) 2016-11-17 21:04:52 PST
Some test infrastructure in bug 164905.
Comment 10 Simon Fraser (smfr) 2016-11-19 11:38:37 PST
https://trac.webkit.org/changeset/208927