Unified Diff: mobile/android/base/java/org/mozilla/gecko/BrowserApp.java
Issue 29977568:
Issue 7198 - Hide ABP extension from dropdown menu (Closed)
Patch Set:
Created Jan. 9, 2019, 9:08 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: mobile/android/base/java/org/mozilla/gecko/BrowserApp.java |
| =================================================================== |
| --- a/mobile/android/base/java/org/mozilla/gecko/BrowserApp.java |
| +++ b/mobile/android/base/java/org/mozilla/gecko/BrowserApp.java |
| @@ -3467,16 +3467,20 @@ |
| /** |
| * Adds a WebExtension browser action to the menu. |
| */ |
| private void addBrowserActionMenuItem(final BrowserActionItemInfo info) { |
| if (mBrowserActionItemsCache == null) { |
| mBrowserActionItemsCache = new ArrayList<BrowserActionItemInfo>(); |
| } |
| + // We don't want to show extensions as items in the menu |
| + // See https://issues.adblockplus.org/ticket/7198 |
| + if (true) return; |
| + |
| // Mark it as added if the menu was ready. |
| info.added = (mMenu != null); |
| // Always cache so we can rebuild after a locale switch. |
| mBrowserActionItemsCache.add(info); |
| if (mMenu == null) { |
| return; |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
