Commit 41a9e47d authored by Matt Giuca's avatar Matt Giuca Committed by Commit Bot

Move "Open link in <app>" lower down on the link context menu.

Makes it not so prominent (after "open in incognito" rather than being
the topmost item on the menu).

Bug: 907298
Change-Id: I74997f18a7b6bb39db1210912ed1f40e69a4fffe
Reviewed-on: https://chromium-review.googlesource.com/c/1345754
Commit-Queue: Matt Giuca <mgiuca@chromium.org>
Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610509}
parent 0e6b8671
...@@ -1034,8 +1034,6 @@ void RenderViewContextMenu::AppendLinkItems() { ...@@ -1034,8 +1034,6 @@ void RenderViewContextMenu::AppendLinkItems() {
base::FeatureList::IsEnabled(features::kDesktopPWAWindowing) && base::FeatureList::IsEnabled(features::kDesktopPWAWindowing) &&
browser && browser->is_app(); browser && browser->is_app();
AppendOpenInBookmarkAppLinkItems();
menu_model_.AddItemWithStringId( menu_model_.AddItemWithStringId(
IDC_CONTENT_CONTEXT_OPENLINKNEWTAB, IDC_CONTENT_CONTEXT_OPENLINKNEWTAB,
in_app ? IDS_CONTENT_CONTEXT_OPENLINKNEWTAB_INAPP in_app ? IDS_CONTENT_CONTEXT_OPENLINKNEWTAB_INAPP
...@@ -1054,6 +1052,7 @@ void RenderViewContextMenu::AppendLinkItems() { ...@@ -1054,6 +1052,7 @@ void RenderViewContextMenu::AppendLinkItems() {
in_app ? IDS_CONTENT_CONTEXT_OPENLINKOFFTHERECORD_INAPP in_app ? IDS_CONTENT_CONTEXT_OPENLINKOFFTHERECORD_INAPP
: IDS_CONTENT_CONTEXT_OPENLINKOFFTHERECORD); : IDS_CONTENT_CONTEXT_OPENLINKOFFTHERECORD);
AppendOpenInBookmarkAppLinkItems();
AppendOpenWithLinkItems(); AppendOpenWithLinkItems();
// While ChromeOS supports multiple profiles, only one can be open at a // While ChromeOS supports multiple profiles, only one can be open at a
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment