Bug 237127 – Refactor logic for showing "Markup Image" and Quick Note items in the callout bar

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Wenson Hsieh]

.

[Radar WebKit Bug Importer]

<rdar://problem/89396617>

[Wenson Hsieh]

Created attachment 453068 [details]
For EWS

[Megan Gardner]

Comment on attachment 453068 [details]
For EWS

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

> Tools/TestWebKitAPI/Tests/WebKitCocoa/WKContentViewEditingActions.mm:96
> +    EXPECT_FALSE([menuBuilder containsActionWithTitle:WebCore::contextMenuItemTagAddHighlightToNewQuickNote()]);

You should still test for both App Highlight menu items. If things change around the logic for which item to show when, it's unlikely that this test would be updated as a part of that. So, best to just do it now.

> Tools/TestWebKitAPI/Tests/WebKitCocoa/WKContentViewEditingActions.mm:104
> +    EXPECT_TRUE([menuBuilder containsActionWithTitle:WebCore::contextMenuItemTagAddHighlightToNewQuickNote()]);

ditto.

[Wenson Hsieh]

Comment on attachment 453068 [details]
For EWS

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

Thanks for the review!

>> Tools/TestWebKitAPI/Tests/WebKitCocoa/WKContentViewEditingActions.mm:96
>> +    EXPECT_FALSE([menuBuilder containsActionWithTitle:WebCore::contextMenuItemTagAddHighlightToNewQuickNote()]);
> 
> You should still test for both App Highlight menu items. If things change around the logic for which item to show when, it's unlikely that this test would be updated as a part of that. So, best to just do it now.

Sounds good — I'll add checks to verify that the "Add to existing Quick Note" items are not also present in either case.

[Wenson Hsieh]

Created attachment 453109 [details]
For landing

[EWS]

Committed r290450 (247752@main): <https://commits.webkit.org/247752@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 453109 [details].