Bug 40005: Modify Default toolbar menu (e4d552ec) · Commits · The Tor Project / Applications / android-components

components/browser/menu/src/main/java/mozilla/components/browser/menu/WebExtensionBrowserMenuBuilder.kt

+7 −22

Original line number	Diff line number	Diff line
		@@ -9,7 +9,6 @@ import androidx.annotation.ColorRes
		import androidx.annotation.DrawableRes
		import mozilla.components.browser.menu.item.BackPressMenuItem
		import mozilla.components.browser.menu.item.BrowserMenuDivider
		import mozilla.components.browser.menu.item.BrowserMenuImageText
		import mozilla.components.browser.menu.item.NO_ID
		import mozilla.components.browser.menu.item.ParentBrowserMenuItem
		import mozilla.components.browser.menu.item.WebExtensionBrowserMenuItem
		@@ -96,20 +95,10 @@ class WebExtensionBrowserMenuBuilder(
		iconTintColorResource = style.webExtIconTintColorResource
		)

		val addonsManagerMenuItem = BrowserMenuImageText(
		label = context.getString(R.string.mozac_browser_menu_addons_manager),
		imageResource = style.addonsManagerMenuItemDrawableRes,
		iconTintColorResource = style.webExtIconTintColorResource
		) {
		onAddonsManagerTapped.invoke()
		}

		val webExtSubMenuItems = if (appendExtensionSubMenuAtStart) {
		listOf(backPressMenuItem) + BrowserMenuDivider() +
		filteredExtensionMenuItems +
		BrowserMenuDivider() + addonsManagerMenuItem
		filteredExtensionMenuItems
		} else {
		listOf(addonsManagerMenuItem) + BrowserMenuDivider() +
		filteredExtensionMenuItems +
		BrowserMenuDivider() + backPressMenuItem
		}
		@@ -125,25 +114,21 @@ class WebExtensionBrowserMenuBuilder(
		endOfMenuAlwaysVisible = endOfMenuAlwaysVisible
		)
		} else {
		BrowserMenuImageText(
		label = context.getString(R.string.mozac_browser_menu_addons),
		imageResource = style.addonsManagerMenuItemDrawableRes,
		iconTintColorResource = style.webExtIconTintColorResource
		) {
		onAddonsManagerTapped.invoke()
		}
		null
		}
		val mainMenuIndex = items.indexOfFirst { browserMenuItem ->
		(browserMenuItem as? WebExtensionPlaceholderMenuItem)?.id ==
		WebExtensionPlaceholderMenuItem.MAIN_EXTENSIONS_MENU_ID
		}

		return if (mainMenuIndex != -1) {
		return if (mainMenuIndex != -1 && addonsMenuItem != null) {
		items[mainMenuIndex] = addonsMenuItem
		items
		// if we do not have a placeholder we should add the extension submenu at top or bottom
		} else {
		if (appendExtensionSubMenuAtStart) {
		if (addonsMenuItem == null) {
		items
		} else if (appendExtensionSubMenuAtStart) {
		listOf(addonsMenuItem) + items
		} else {
		items + addonsMenuItem