950785 - Remove unused selectWord and selectWordContext

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Text Selection, defect)

Description

[:Margaret Leibovic]

Looks like we got rid of the "Select Word" context menu item, so let's get rid of these unused functions.

Comment 1

[:Margaret Leibovic]

In fact, it looks like there are other unused function in ClipboardHelper, like selectAll and searchWith:

http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/browser.js#6236

Comment 2

[raymond [:retornam] (inactive)]

Attachment: bug-950785.patch

Comment 3

[:Margaret Leibovic]

Comment on attachment 8348905 [details] [diff] [review]
bug-950785.patch

Review of attachment 8348905 [details] [diff] [review]:
-----------------------------------------------------------------

This is looking good, but I found even more things we can remove!

(I don't know if you intended for me to review this patch, but in the future, you should be sure to set the review? or feedback? flag when you want to get review/feedback on a patch.)

::: mobile/android/chrome/content/browser.js
@@ -6243,3 @@
>    share: function() {
>      SelectionHandler.shareSelection();
>    },

I know you were just following what I posted in the description, but it looks like this share method also isn't used.

Doing a search through the other ClipboardHelper methods, it also looks like copy isn't used.

I filed bug 951374 about some more ways we can clean up ClipboardHelper, so you may be interested in that as well :)

Comment 4

[raymond [:retornam] (inactive)]

Attachment: bug-950785.patch

updated patch to remove share as well. I've cc'ed  myself on the other bug.

Comment 5

[:Margaret Leibovic]

Comment on attachment 8349061 [details] [diff] [review]
bug-950785.patch

Looks good, thanks!

Comment 6

[:Margaret Leibovic]

This will conflict with wesj's patch in bug 950464, but maybe he can land this for you and resolve the conflict while he does that :)

Comment 7

[raymond [:retornam] (inactive)]

(In reply to :Margaret Leibovic from comment #6)
> This will conflict with wesj's patch in bug 950464, but maybe he can land
> this for you and resolve the conflict while he does that :)

That will be great. Thanks.

Comment 8

[Wesley Johnston (:wesj)]

https://hg.mozilla.org/integration/fx-team/rev/b329b704e89a

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b329b704e89a