Bug 200588

Summary: [iOS WK2] Remove context menu hints on navigation
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, simon.fraser, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
thorton: review+
Patch none

Description Simon Fraser (smfr) 2019-08-09 14:07:04 PDT
[iOS WK2] Remove context menu previews on navigation
Comment 1 Simon Fraser (smfr) 2019-08-09 14:08:36 PDT
Created attachment 375950 [details]
Patch
Comment 2 Simon Fraser (smfr) 2019-08-09 14:08:38 PDT
<rdar://problem/54061796>
Comment 3 Tim Horton 2019-08-09 14:10:38 PDT
Comment on attachment 375950 [details]
Patch

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

> Source/WebKit/ChangeLog:9
> +        Make sure the context menu preview doesn't linger across navigations by hosting it in its

hint! the preview is the platter that is presented above the context menu.

> Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:3954
> +    [self _hideContextMenu];

You don't hide the menu, just the hint

> Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:6397
> +    if (_contextMenuContainerView) {

Ditto
Comment 4 Simon Fraser (smfr) 2019-08-09 14:19:51 PDT
Created attachment 375956 [details]
Patch
Comment 5 WebKit Commit Bot 2019-08-09 15:01:46 PDT
Comment on attachment 375956 [details]
Patch

Clearing flags on attachment: 375956

Committed r248481: <https://trac.webkit.org/changeset/248481>
Comment 6 Alexey Proskuryakov 2019-08-09 15:30:09 PDT
Comment on attachment 375956 [details]
Patch

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

> Source/WebKit/ChangeLog:15
> +        (-[WKContentView _didCommitLoadForMainFrame]):

The issue will still be happening for subframe navigations, correct?
Comment 7 Simon Fraser (smfr) 2019-08-10 16:54:56 PDT
Yes, but we can fix that separately.