Bug 146415 - [WK2][iOS] Swipe gesture snapshot stays up for 3 seconds when swiping back after top-hit navigation
Summary: [WK2][iOS] Swipe gesture snapshot stays up for 3 seconds when swiping back af...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P1 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-06-29 11:59 PDT by Chris Dumez
Modified: 2015-06-29 12:57 PDT (History)
3 users (show)

See Also:


Attachments
Patch (9.38 KB, patch)
2015-06-29 12:06 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (9.38 KB, patch)
2015-06-29 12:28 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2015-06-29 11:59:17 PDT
Swipe gesture snapshot stays up for 3 seconds when swiping back after top-hit navigation. This is because we fail to restore the scroll position in this case and the UI side is waiting for the scroll position to be restored before taking down the swipe snapshot.
Comment 1 Chris Dumez 2015-06-29 11:59:32 PDT
rdar://problem/21595801
Comment 2 Chris Dumez 2015-06-29 12:06:14 PDT
Created attachment 255766 [details]
Patch
Comment 3 Tim Horton 2015-06-29 12:08:06 PDT
Comment on attachment 255766 [details]
Patch

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

> Source/WebKit2/ChangeLog:3
> +        [WK2[[iOS] Swipe gesture snapshot stays up for 3 seconds when swiping back after top-hit navigation

Too many [][][][[
Comment 4 Chris Dumez 2015-06-29 12:28:43 PDT
Created attachment 255768 [details]
Patch
Comment 5 WebKit Commit Bot 2015-06-29 12:56:58 PDT
Comment on attachment 255768 [details]
Patch

Clearing flags on attachment: 255768

Committed r186082: <http://trac.webkit.org/changeset/186082>
Comment 6 WebKit Commit Bot 2015-06-29 12:57:02 PDT
All reviewed patches have been landed.  Closing bug.