WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
203713
Take down the gesture snapshot early when the back/forward cache is not leveraged
https://bugs.webkit.org/show_bug.cgi?id=203713
Summary
Take down the gesture snapshot early when the back/forward cache is not lever...
Chris Dumez
Reported
2019-10-31 18:46:11 PDT
Take down the gesture snapshot as soon as the gesture is done when the back/forward cache is not leveraged. Otherwise, the snapshot may stay up for a long time (while we load, parse and restore scroll position) and it looks to the user as if the view is unresponsive. Showing the back loading in such cases is less confusing.
Attachments
Patch
(7.44 KB, patch)
2019-10-31 18:53 PDT
,
Chris Dumez
no flags
Details
Formatted Diff
Diff
Patch
(7.44 KB, patch)
2019-10-31 18:55 PDT
,
Chris Dumez
no flags
Details
Formatted Diff
Diff
Patch
(7.98 KB, patch)
2019-10-31 19:14 PDT
,
Chris Dumez
no flags
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2019-10-31 18:49:51 PDT
<
rdar://problem/56803910
>
Chris Dumez
Comment 2
2019-10-31 18:53:49 PDT
Created
attachment 382534
[details]
Patch
Chris Dumez
Comment 3
2019-10-31 18:55:15 PDT
Created
attachment 382535
[details]
Patch
Chris Dumez
Comment 4
2019-10-31 19:14:38 PDT
Created
attachment 382538
[details]
Patch
WebKit Commit Bot
Comment 5
2019-10-31 21:07:44 PDT
Comment on
attachment 382538
[details]
Patch Clearing flags on attachment: 382538 Committed
r251907
: <
https://trac.webkit.org/changeset/251907
>
WebKit Commit Bot
Comment 6
2019-10-31 21:07:46 PDT
All reviewed patches have been landed. Closing bug.
Chris Dumez
Comment 7
2019-11-06 15:53:49 PST
Reverted
r251907
for reason: Causes white flashing in some cases <
rdar://problem/56957850
> Committed
r252155
: <
https://trac.webkit.org/changeset/252155
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug