Skip to content

Commit

Permalink
Keep valid reference to WidgetManagerDelegate in ContextMenuWidget la…
Browse files Browse the repository at this point in the history
…mbdas (#2704)

Fixes #2703
  • Loading branch information
bluemarvin committed Jan 27, 2020
1 parent 7e03c94 commit cb804e2
Showing 1 changed file with 3 additions and 2 deletions.
Expand Up @@ -77,17 +77,18 @@ public void setDismissCallback(Runnable aCallback) {
public void setContextElement(GeckoSession.ContentDelegate.ContextElement aContextElement) {
mItems = new ArrayList<>();
mItems.add(new MenuWidget.MenuItem(aContextElement.linkUri, 0, null));
final WidgetManagerDelegate widgetManager = mWidgetManager;
if (mWidgetManager.canOpenNewWindow()) {
mItems.add(new MenuWidget.MenuItem(getContext().getString(R.string.context_menu_open_new_window_1), 0, () -> {
if (!StringUtils.isEmpty(aContextElement.linkUri)) {
mWidgetManager.openNewWindow(aContextElement.linkUri);
widgetManager.openNewWindow(aContextElement.linkUri);
}
onDismiss();
}));
}
mItems.add(new MenuWidget.MenuItem(getContext().getString(R.string.context_menu_open_new_tab_1), 0, () -> {
if (!StringUtils.isEmpty(aContextElement.linkUri)) {
mWidgetManager.openNewTab(aContextElement.linkUri);
widgetManager.openNewTab(aContextElement.linkUri);
GleanMetricsService.Tabs.openedCounter(GleanMetricsService.Tabs.TabSource.CONTEXT_MENU);
}
onDismiss();
Expand Down

0 comments on commit cb804e2

Please sign in to comment.