commit fe64b1f2d549c58036c18fde759c79eba9baa367
parent 3ed3ad9158e3990ffa74c2ed8357e1b4da33fd52
Author: Gabriel Luong <gabriel.luong@gmail.com>
Date: Wed, 1 Oct 2025 17:47:45 +0000
Bug 1973593 - Remove unused NewTabMenu r=android-reviewers,Roger
Differential Revision: https://phabricator.services.mozilla.com/D267041
Diffstat:
2 files changed, 0 insertions(+), 81 deletions(-)
diff --git a/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/components/toolbar/NewTabMenu.kt b/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/components/toolbar/NewTabMenu.kt
@@ -1,30 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-package org.mozilla.fenix.components.toolbar
-
-import android.content.Context
-import androidx.annotation.VisibleForTesting
-import mozilla.components.concept.menu.candidate.MenuCandidate
-import mozilla.components.ui.tabcounter.TabCounterMenu
-
-/**
- * An implementation of [TabCounterMenu] used to display a menu for the new tab button.
- */
-class NewTabMenu(
- context: Context,
- onItemTapped: (Item) -> Unit,
- iconColor: Int? = null,
-) : TabCounterMenu(context, onItemTapped, iconColor) {
-
- init {
- menuController.submitList(menuItems())
- }
-
- @VisibleForTesting
- internal fun menuItems(): List<MenuCandidate> = listOf(
- newTabItem,
- newPrivateTabItem,
- )
-}
diff --git a/mobile/android/fenix/app/src/test/java/org/mozilla/fenix/components/toolbar/NewTabMenuTest.kt b/mobile/android/fenix/app/src/test/java/org/mozilla/fenix/components/toolbar/NewTabMenuTest.kt
@@ -1,51 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-package org.mozilla.fenix.components.toolbar
-
-import android.content.Context
-import androidx.appcompat.view.ContextThemeWrapper
-import mozilla.components.concept.menu.candidate.TextMenuCandidate
-import mozilla.components.support.test.mock
-import mozilla.components.support.test.robolectric.testContext
-import mozilla.components.ui.tabcounter.TabCounterMenu
-import org.junit.Assert.assertEquals
-import org.junit.Before
-import org.junit.Test
-import org.junit.runner.RunWith
-import org.mockito.Mockito.verify
-import org.mozilla.fenix.R
-import org.robolectric.RobolectricTestRunner
-
-@RunWith(RobolectricTestRunner::class)
-class NewTabMenuTest {
-
- private lateinit var context: Context
- private lateinit var onItemTapped: (TabCounterMenu.Item) -> Unit
- private lateinit var menu: NewTabMenu
-
- @Before
- fun setup() {
- context = ContextThemeWrapper(testContext, R.style.NormalTheme)
- onItemTapped = mock()
- menu = NewTabMenu(context, onItemTapped)
- }
-
- @Test
- fun `WHEN menu items getter is called THEN return the new tab and private tab menu item`() {
- val items = menu.menuItems()
- val newTab = items[0] as TextMenuCandidate
- val newPrivateTab = items[1] as TextMenuCandidate
-
- assertEquals(2, items.size)
- assertEquals("New tab", newTab.text)
- assertEquals("New private tab", newPrivateTab.text)
-
- newTab.onClick()
- verify(onItemTapped).invoke(TabCounterMenu.Item.NewTab)
-
- newPrivateTab.onClick()
- verify(onItemTapped).invoke(TabCounterMenu.Item.NewPrivateTab)
- }
-}