1355847 - Web apps - Share selected text doesn't work on Android 6+

Status: RESOLVED DUPLICATE of bug 1360658

(Firefox for Android Graveyard :: Web Apps (PWAs), defect, P1)

Description

[Oana Horvath [:ohorvath]]

Device: Android 6+ devices.
Build: Nightly 55.0a1 (2017-04-12).

Steps to reproduce:
1. Install a web app (e.g. Twiiter) and open it.
2. Long press on a word to select it.
3. Tap Share in the dialog box opened.

Expected result:
The Share action should open a list of apps to share with.

Actual result:
The Share button doesn't open the sharing apps list.

Comment 1

[Nevin Chen(Not active on Bugzilla)]

Attachment: Bug 1355847 - Share selected text should work.

Review commit: https://reviewboard.mozilla.org/r/134682/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/134682/

Comment 2

[Nevin Chen(Not active on Bugzilla)]

Comment on attachment 8862791 [details]
Bug 1355847 - Share selected text should work.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/134682/diff/1-2/

Comment 3

[Nevin Chen(Not active on Bugzilla)]

Comment on attachment 8862791 [details]
Bug 1355847 - Share selected text should work.

Just need to call super method so things in GeckoApp handleMessage can be called.

Comment 4

[Max Liu [:maliu]]

Comment on attachment 8862791 [details]
Bug 1355847 - Share selected text should work.

https://reviewboard.mozilla.org/r/134682/#review138102

LGTM

Comment 5

[Julian Chu [:walkingice]]

Comment on attachment 8862791 [details]
Bug 1355847 - Share selected text should work.

https://reviewboard.mozilla.org/r/134682/#review138264

Comment 6

[Nevin Chen(Not active on Bugzilla)]

Comment on attachment 8862791 [details]
Bug 1355847 - Share selected text should work.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/134682/diff/2-3/

Comment 7

[Dale Harvey (:daleharvey)]

Comment on attachment 8862791 [details]
Bug 1355847 - Share selected text should work.

https://reviewboard.mozilla.org/r/134682/#review138354

So r+, howevet JanH already landed this in https://bugzilla.mozilla.org/show_bug.cgi?id=1360658 so I think you can close as dupe

Comment 8

[Mozilla Autoland]

We're sorry - something has gone wrong while rewriting or rebasing your commits. The commits being pushed no longer match what was requested. Please file a bug.