Post visual update: Send pixel when address bar focused (#6655)

Task/Issue URL:
https://app.asana.com/1/137249556945/project/488551667048375/task/1211021593099148?focus=true

### Description

This PR adds a pixel (`m_addressbar_focus_ntp`) that is sent when the
address bar is focused on NTP. It contains the following parameters:

- ~~"is_duck_ai_button_shown"~~"is_tab_switcher_button_shown"
- "is_fire_button_shown"
- "is_browser_menu_button_shown"

### Steps to test this PR

_NTP_
- [x] Open a NTP
- [x] Focus on the address bar
- [x] Verify the `m_addressbar_focus_ntp` pixel is sent with the correct
parameters
- [x] Start typing
- [x] Verify the pixel is sent once with all parameters set to false
(the buttons are not visible)

_Browser_
- [x] Load some website
- [x] Tap on the address bar to add focus
- [x] Verify that pixel (`m_addressbar_focus_ntp`) is not sent

Author: 0nko

app/src/main/java/com/duckduckgo/app/browser/omnibar/OmnibarLayoutViewModel.kt

@@ -74,8 +74,10 @@ import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.SharingStarted
 import kotlinx.coroutines.flow.combine
 import kotlinx.coroutines.flow.distinctUntilChanged
+import kotlinx.coroutines.flow.filter
 import kotlinx.coroutines.flow.flowOn
 import kotlinx.coroutines.flow.launchIn
+import kotlinx.coroutines.flow.map
 import kotlinx.coroutines.flow.onEach
 import kotlinx.coroutines.flow.receiveAsFlow
 import kotlinx.coroutines.flow.stateIn
@@ -216,6 +218,25 @@ class OmnibarLayoutViewModel @Inject constructor(
                 it.copy(showChatMenu = showDuckAiButton)
             }
         }.launchIn(viewModelScope)
+
+        viewState.map {
+            NewTabPixelParams(
+                isNtp = it.viewMode == NewTab,
+                isFocused = it.hasFocus,
+                isTabSwitcherButtonVisible = it.showTabsMenu,
+                isFireButtonVisible = it.showFireIcon,
+                isBrowserMenuButtonVisible = it.showBrowserMenu,
+            )
+        }.distinctUntilChanged()
+            .filter { it.isNtp && it.isFocused }
+            .onEach {
+                val params = mapOf(
+                    Pixel.PixelParameter.IS_TAB_SWITCHER_BUTTON_SHOWN to it.isTabSwitcherButtonVisible.toString(),
+                    Pixel.PixelParameter.IS_FIRE_BUTTON_SHOWN to it.isFireButtonVisible.toString(),
+                    Pixel.PixelParameter.IS_BROWSER_MENU_BUTTON_SHOWN to it.isBrowserMenuButtonVisible.toString(),
+                )
+                pixel.fire(pixel = AppPixelName.ADDRESS_BAR_NTP_FOCUSED, parameters = params)
+            }.launchIn(viewModelScope)
     }
 
     fun onFindInPageRequested() {
@@ -867,4 +888,12 @@ class OmnibarLayoutViewModel @Inject constructor(
             command.send(Command.LaunchInputScreen(query = textToPreFill))
         }
     }
+
+    private data class NewTabPixelParams(
+        val isNtp: Boolean,
+        val isFocused: Boolean,
+        val isTabSwitcherButtonVisible: Boolean,
+        val isFireButtonVisible: Boolean,
+        val isBrowserMenuButtonVisible: Boolean,
+    )
 }

app/src/main/java/com/duckduckgo/app/pixels/AppPixelName.kt

@@ -340,6 +340,7 @@ enum class AppPixelName(override val pixelName: String) : Pixel.PixelName {
     ADDRESS_BAR_NEW_TAB_PAGE_CLOSED("m_addressbar_focus_close_ntp"),
     ADDRESS_BAR_WEBSITE_CLOSED("m_addressbar_focus_close_website"),
     ADDRESS_BAR_SERP_CLOSED("m_addressbar_focus_close_serp"),
+    ADDRESS_BAR_NTP_FOCUSED("m_addressbar_focus_ntp"),
 
     KEYBOARD_GO_NEW_TAB_CLICKED("m_keyboard_go_click_ntp"),
     KEYBOARD_GO_WEBSITE_CLICKED("m_keyboard_go_click_website"),

app/src/test/java/com/duckduckgo/app/browser/omnibar/OmnibarLayoutViewModelTest.kt

@@ -47,14 +47,18 @@ import kotlin.reflect.KClass
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.flowOf
 import kotlinx.coroutines.test.runTest
-import org.junit.Assert.*
+import org.junit.Assert.assertEquals
+import org.junit.Assert.assertFalse
+import org.junit.Assert.assertTrue
 import org.junit.Before
 import org.junit.Rule
 import org.junit.Test
 import org.junit.runner.RunWith
 import org.mockito.kotlin.any
 import org.mockito.kotlin.doAnswer
+import org.mockito.kotlin.eq
 import org.mockito.kotlin.mock
+import org.mockito.kotlin.never
 import org.mockito.kotlin.verify
 import org.mockito.kotlin.verifyNoInteractions
 import org.mockito.kotlin.whenever
@@ -1706,4 +1710,66 @@ class OmnibarLayoutViewModelTest {
     private fun Command.assertCommand(expectedType: KClass<out Command>) {
         assertTrue(String.format("Unexpected command type: %s", this::class.simpleName), this::class == expectedType)
     }
+
+    @Test
+    fun whenOmnibarFocusedOnNewTabThenAddressBarNtpFocusedPixelSent() = runTest {
+        givenSiteLoaded("")
+        testee.onViewModeChanged(ViewMode.NewTab)
+        testee.onOmnibarFocusChanged(true, "")
+
+        verify(pixel).fire(
+            AppPixelName.ADDRESS_BAR_NTP_FOCUSED,
+            mapOf(
+                Pixel.PixelParameter.IS_TAB_SWITCHER_BUTTON_SHOWN to "true",
+                Pixel.PixelParameter.IS_FIRE_BUTTON_SHOWN to "true",
+                Pixel.PixelParameter.IS_BROWSER_MENU_BUTTON_SHOWN to "true",
+            ),
+        )
+    }
+
+    @Test
+    fun whenOmnibarFocusedOnWebsiteThenAddressBarNtpFocusedPixelNotSent() = runTest {
+        givenSiteLoaded(RANDOM_URL)
+        testee.onViewModeChanged(ViewMode.Browser(RANDOM_URL))
+        testee.onOmnibarFocusChanged(true, "")
+
+        verify(pixel, never()).fire(
+            eq(AppPixelName.ADDRESS_BAR_NTP_FOCUSED),
+            any(),
+            any(),
+            any(),
+        )
+    }
+
+    @Test
+    fun whenOmnibarFocusedOnSerpThenAddressBarNtpFocusedPixelNotSent() = runTest {
+        givenSiteLoaded(SERP_URL)
+        testee.onViewModeChanged(ViewMode.Browser(SERP_URL))
+        testee.onOmnibarFocusChanged(true, "")
+
+        verify(pixel, never()).fire(
+            eq(AppPixelName.ADDRESS_BAR_NTP_FOCUSED),
+            any(),
+            any(),
+            any(),
+        )
+    }
+
+    @Test
+    fun whenOmnibarFocusedOnNewTabWithAllButtonsDisabledThenCorrectParametersSent() = runTest {
+        givenSiteLoaded("")
+        testee.onViewModeChanged(ViewMode.NewTab)
+
+        // Simulate focusing the omnibar with text, which hides the fire button
+        testee.onOmnibarFocusChanged(true, "some query")
+
+        verify(pixel).fire(
+            AppPixelName.ADDRESS_BAR_NTP_FOCUSED,
+            mapOf(
+                Pixel.PixelParameter.IS_TAB_SWITCHER_BUTTON_SHOWN to "false",
+                Pixel.PixelParameter.IS_FIRE_BUTTON_SHOWN to "false",
+                Pixel.PixelParameter.IS_BROWSER_MENU_BUTTON_SHOWN to "false",
+            ),
+        )
+    }
 }

statistics/statistics-api/src/main/java/com/duckduckgo/app/statistics/pixels/Pixel.kt

@@ -68,6 +68,9 @@ interface Pixel {
         const val TAB_INACTIVE_3W = "tab_inactive_3w"
         const val IS_ENABLED = "is_enabled"
         const val FROM_FOCUSED_NTP = "from_focused_ntp"
+        const val IS_TAB_SWITCHER_BUTTON_SHOWN = "is_tab_switcher_button_shown"
+        const val IS_FIRE_BUTTON_SHOWN = "is_fire_button_shown"
+        const val IS_BROWSER_MENU_BUTTON_SHOWN = "is_browser_menu_button_shown"
     }
 
     object PixelValues {