Add bookmark through context menu

[vesta0]

User Story

• As a user, I want to be able to long press on a link on a website, and have the option to bookmark that link, so I can quickly bookmark a page and go back to it later without having to load the page

Requirements

• Add "add a bookmark" to the link context menu (@jpfaller to provide exact strings)
• Add a telemetry probe to measure taps on "add a bookmark" through the context menu

Acceptance Criteria

• I can add a bookmark through the context menu.

Please use Fennec UI as reference:

┆Issue is synchronized with this Jira Task

[vesta0]

@lime124 I propose that we skip UX for this issue and refer to the Fennec UI instead.

[vesta0]

@liuche this needs to be re-sized as I added telemetry to the requirements.

[lime124]

@jpfaller can we reuse bookmark link from fennec for our context menu? thanks!

[jpfaller]

Yep, I think it can be re-used.

[liuche]

adding triage label to re-size, and this now has additional telemetry requirements.

[vesta0]

We resized this from E2 to E5 during backlog grooming today.

[vesta0]

@lime124 should we also add a "add to collection" option to this menu? I am thinking maybe that is in fact more useful/relevant. I am thinking the only reason a user would bookmark a link they haven't opened is to have the option to come back to it later without interrupting their browsing activity.

[lime124]

Adding group triage - does this have any dependencies? can a contributor pick it up?

@vesta0 Can we open that in a separate issue? I'd like to keep the scope on this one small/contained since it's ready for eng - there may be some additional design decisions on the flow that need to be made.

[sblatz]

This is AC work we can do as Fenix engineers if we'd like.

Let's modify the ContextMenuCandidate.kt file

@lime124. @vesta0 said she'd open a separate ticket for the other request :)

[vesta0]

@lime124 sure I can create a separate issue for "add to a collection". That aside, do you think "add bookmark" is a useful feature on its own? I just want to make sure that we look at all the context menu options holistically before adding new things.

[jackyzy823]

The android part is not so hard to complete , see https://github.com/jackyzy823/android-components/tree/copy-link-text and
https://github.com/jackyzy823/fenix/tree/context-menu-bookmark

The question is same as #16582 (comment)

You can not get bookmark's link text (aka title) from geckoview

Ref: fennec's implment:
https://hg.mozilla.org/releases/mozilla-esr68/file/tip/mobile/android/chrome/content/browser.js#l1165

[boek]

Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1812724

Change performed by the Move to Bugzilla add-on.