Remove GeckoInterface.createShortcut

RESOLVED FIXED in Firefox 55

Status

()

RESOLVED FIXED
a year ago
a year ago

People

(Reporter: jchen, Assigned: jchen)

Tracking

50 Branch
Firefox 55
Unspecified
Android
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Updated

a year ago
Blocks: 1362162
(Assignee)

Comment 1

a year ago
Created attachment 8868170 [details] [diff] [review]
Move shortcut creation to static methods in GeckoApplication (v1)

Move shortcut creation code in GeckoApp to GeckoApplication, and make
the methods static so that we can call them without a GeckoInterface
instance. This lets us remove GeckoInterface.createShortcut.
Attachment #8868170 - Flags: review?(droeh)
Comment on attachment 8868170 [details] [diff] [review]
Move shortcut creation to static methods in GeckoApplication (v1)

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

Looks good with a couple of nits.

::: mobile/android/base/java/org/mozilla/gecko/GeckoApplication.java
@@ +445,5 @@
> +                  }
> +              });
> +    }
> +
> +    /* protected */ static void createShortcutWithIcon(final String aTitle, final String aURI,

Did you mean /* package */ here?

::: mobile/android/geckoview/src/main/java/org/mozilla/gecko/util/ActivityUtils.java
@@ +61,5 @@
>  
>      /**
>       * Finish this activity and launch the default home screen activity.
>       */
> +    public static void goToHomeScreen(Context activity) {

Rename activity to context.
Attachment #8868170 - Flags: review?(droeh) → review+

Comment 3

a year ago
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6f5485d306e1
Move shortcut creation to static methods in GeckoApplication; r=droeh

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6f5485d306e1
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in before you can comment on or make changes to this bug.