Bug 196011 - [iOS WK2] Turn on async overflow scrolling by default
Summary: [iOS WK2] Turn on async overflow scrolling by default
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-03-20 10:02 PDT by Simon Fraser (smfr)
Modified: 2019-03-21 18:06 PDT (History)
10 users (show)

See Also:


Attachments
Patch (49.96 KB, patch)
2019-03-20 10:29 PDT, Simon Fraser (smfr)
koivisto: review+
Details | Formatted Diff | Diff
Patch (49.88 KB, patch)
2019-03-20 10:45 PDT, Simon Fraser (smfr)
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2019-03-20 10:02:53 PDT
[iOS WK2] Turn on async overflow scrolling by default
Comment 1 Simon Fraser (smfr) 2019-03-20 10:29:37 PDT
Created attachment 365355 [details]
Patch
Comment 2 EWS Watchlist 2019-03-20 10:32:28 PDT
Attachment 365355 [details] did not pass style-queue:


ERROR: LayoutTests/platform/ios-wk2/TestExpectations:1316:  Path does not exist.  [test/expectations] [5]
ERROR: /Volumes/Data/StyleQueue/WebKit/LayoutTests/platform/ios-wk2/TestExpectations:1316:  Path does not exist.  [test/expectations] [5]
Total errors found: 2 in 22 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Simon Fraser (smfr) 2019-03-20 10:45:31 PDT
Created attachment 365357 [details]
Patch
Comment 4 WebKit Commit Bot 2019-03-20 14:44:51 PDT
Comment on attachment 365357 [details]
Patch

Rejecting attachment 365357 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'apply-attachment', '--no-update', '--non-interactive', 365357, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Logging in as commit-queue@webkit.org...
Fetching: https://bugs.webkit.org/attachment.cgi?id=365357&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=196011&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Processing patch 365357 from bug 196011.
Fetching: https://bugs.webkit.org/attachment.cgi?id=365357
Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Parsed 22 diffs from patch file(s).
patching file Source/WebKit/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Source/WebKit/Shared/WebPreferences.yaml
patching file Source/WebKit/Shared/WebPreferencesDefaultValues.h
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file LayoutTests/compositing/layer-creation/absolute-in-async-overflow-scroll-expected.txt
Hunk #1 FAILED at 16.
1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/compositing/layer-creation/absolute-in-async-overflow-scroll-expected.txt.rej
patching file LayoutTests/platform/ios-wk2/TestExpectations
patching file LayoutTests/platform/ios-wk2/compositing/geometry/limit-layer-bounds-transformed-overflow-expected.txt
patching file LayoutTests/platform/ios-wk2/compositing/layer-creation/absolute-in-async-overflow-scroll-expected.txt
patching file LayoutTests/platform/ios-wk2/compositing/layer-creation/overflow-scroll-overlap-expected.txt
patching file LayoutTests/platform/ios-wk2/compositing/layer-creation/scroll-partial-update-expected.txt
patching file LayoutTests/platform/ios-wk2/compositing/overflow/content-gains-scrollbars-expected.txt
patching file LayoutTests/platform/ios-wk2/compositing/overflow/overflow-scroll-expected.txt
patching file LayoutTests/platform/ios-wk2/compositing/overflow/overflow-scrollbar-layer-positions-expected.txt
patching file LayoutTests/platform/ios-wk2/compositing/overflow/overflow-scrollbar-layers-expected.txt
patching file LayoutTests/platform/ios-wk2/compositing/overflow/remove-overflow-crash2-expected.txt
patching file LayoutTests/platform/ios-wk2/compositing/overflow/scrollbar-painting-expected.txt
patching file LayoutTests/platform/ios/compositing/geometry/limit-layer-bounds-clipping-ancestor-expected.txt
patching file LayoutTests/platform/ios/compositing/layers-inside-overflow-scroll-expected.txt
patching file LayoutTests/platform/ios/compositing/overflow/scroll-ancestor-update-expected.txt
patching file LayoutTests/platform/ios/compositing/reflections/nested-reflection-on-overflow-expected.txt
patching file LayoutTests/platform/ios/compositing/self-painting-layers-expected.txt
patching file LayoutTests/scrollingcoordinator/scrolling-tree/positioned-nodes-complex-expected.txt

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: https://webkit-queues.webkit.org/results/11585099
Comment 5 Simon Fraser (smfr) 2019-03-20 16:00:07 PDT
https://trac.webkit.org/r243250
Comment 6 Jon Lee 2019-03-20 16:06:48 PDT
<rdar://problem/48453859> Turn async scrolling for overflow:scroll on by default
Comment 7 Lucas Forschler 2019-03-21 08:37:01 PDT
rdar://problem/48453859
Comment 8 Shawn Roberts 2019-03-21 09:24:29 PDT
Had to roll out in https://trac.webkit.org/changeset/243292/webkit

Was causing 109 layout test failures and 2 crashes.

All 111 failures happen in 243250, all pass in 243249.

Link to results page :

https://build.webkit.org/results/Apple%20iOS%2012%20Simulator%20Debug%20WK2%20(Tests)/r243288%20(2909)/results.html# 

Assertion Crashes : 

ASSERTION FAILED: !m_scrollDimensionsDirty

Will attach test list file on Radar.
Comment 9 Simon Fraser (smfr) 2019-03-21 17:09:01 PDT
Re-landed in https://trac.webkit.org/changeset/243347/webkit.
Comment 10 Simon Fraser (smfr) 2019-03-21 17:34:46 PDT
Test cleanup in https://trac.webkit.org/changeset/243349/webkit
Comment 11 Simon Fraser (smfr) 2019-03-21 17:49:49 PDT
And https://trac.webkit.org/r243350
Comment 12 Simon Fraser (smfr) 2019-03-21 18:06:48 PDT
The asserting tests were:
imported/blink/fast/text/output-isolate-at-end-of-line-crash.html
legacy-animation-engine/imported/blink/fast/text/output-isolate-at-end-of-line-crash.html

These are the same test in 2 places. The test sets overflow: -webkit-paged-y on a MathML element, which is pretty obscure. I filed bug 196123 about it.