Summary: | [iOS] Context Menu - Add to Reading List does nothing | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dean Jackson <dino> | ||||
Component: | New Bugs | Assignee: | Dean Jackson <dino> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | andersca, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Dean Jackson
2019-07-04 13:50:49 PDT
Created attachment 373471 [details]
Patch
Comment on attachment 373471 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=373471&action=review > Source/WebKit/UIProcess/API/Cocoa/_WKElementAction.mm:290 > + RELEASE_LOG(ContextMenu, "Executing action for type: %s", elementActionTypeToUIActionIdentifier([elementAction type]).UTF8String); Do you really need to log here? Committed r247185: <https://trac.webkit.org/changeset/247185> Comment on attachment 373471 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=373471&action=review >> Source/WebKit/UIProcess/API/Cocoa/_WKElementAction.mm:290 >> + RELEASE_LOG(ContextMenu, "Executing action for type: %s", elementActionTypeToUIActionIdentifier([elementAction type]).UTF8String); > > Do you really need to log here? No. But it seems we're more happy to release log these days. |