Add showLogo param to NTP (#6539)

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

### Description
- Adds an optional `showLogo` param to hide the logo from NTP.

### Steps to test this PR
- [ ] Verify that NTP works as expected
- [ ] Verify that the Search Tab on the Input Screen works as expected

Author: joshliebe

app/src/main/java/com/duckduckgo/app/browser/newtab/NewTabLegacyPageView.kt

@@ -73,6 +73,7 @@ class NewTabLegacyPageView @JvmOverloads constructor(
     context: Context,
     attrs: AttributeSet? = null,
     defStyle: Int = 0,
+    private val showLogo: Boolean = true,
 ) : LinearLayout(context, attrs, defStyle) {
 
     @Inject
@@ -136,10 +137,15 @@ class NewTabLegacyPageView @JvmOverloads constructor(
         val isHomeBackgroundLogoVisible = (!viewState.onboardingComplete || viewState.message == null) &&
             viewState.favourites.isEmpty()
 
-        if (isHomeBackgroundLogoVisible) {
-            homeBackgroundLogo.showLogo()
+        if (!showLogo && isHomeBackgroundLogoVisible) {
+            this.gone()
         } else {
-            homeBackgroundLogo.hideLogo()
+            this.show()
+            if (isHomeBackgroundLogoVisible) {
+                homeBackgroundLogo.showLogo()
+            } else {
+                homeBackgroundLogo.hideLogo()
+            }
         }
         if (viewState.message != null && viewState.onboardingComplete) {
             showRemoteMessage(viewState.message, viewState.newMessage)

app/src/main/java/com/duckduckgo/app/browser/newtab/NewTabPageProvider.kt

@@ -51,8 +51,8 @@ class RealNewTabPageProvider @Inject constructor(
 )
 class NewTabLegacyPage @Inject constructor() : NewTabPagePlugin {
 
-    override fun getView(context: Context): View {
-        return NewTabLegacyPageView(context)
+    override fun getView(context: Context, showLogo: Boolean): View {
+        return NewTabLegacyPageView(context, showLogo = showLogo)
     }
 }
 

app/src/test/java/com/duckduckgo/app/browser/newtab/NewTabPageProviderTest.kt

@@ -125,13 +125,13 @@ class NewTabPageProviderTest {
     }
 
     class LegacyNewTabPlugin : NewTabPagePlugin {
-        override fun getView(context: Context): View {
+        override fun getView(context: Context, showLogo: Boolean): View {
             return View(context)
         }
     }
 
-    class NewNewTabPlugin() : NewTabPagePlugin {
-        override fun getView(context: Context): View {
+    class NewNewTabPlugin : NewTabPagePlugin {
+        override fun getView(context: Context, showLogo: Boolean): View {
             return View(context)
         }
     }

new-tab-page/new-tab-page-api/src/main/java/com/duckduckgo/newtabpage/api/NewTabPagePlugin.kt

@@ -29,9 +29,13 @@ interface NewTabPagePlugin : ActivePlugin {
 
     /**
      * This method returns a [View] that will be used as the NewTabPage content
+     * @param context The context to create the view with
+     * @param showLogo Whether to show the logo in the new tab page
      * @return [View]
      */
-    fun getView(context: Context): View
+    fun getView(context: Context, showLogo: Boolean = true): View {
+        return getView(context)
+    }
 
     companion object {
         const val PRIORITY_LEGACY_NTP = 0

new-tab-page/new-tab-page-impl/src/main/java/com/duckduckgo/newtabpage/impl/NewTabPage.kt

@@ -35,7 +35,7 @@ import javax.inject.Inject
 )
 class NewTabPage @Inject constructor() : NewTabPagePlugin {
 
-    override fun getView(context: Context): View {
-        return NewTabPageView(context)
+    override fun getView(context: Context, showLogo: Boolean): View {
+        return NewTabPageView(context, showLogo = showLogo)
     }
 }

new-tab-page/new-tab-page-impl/src/main/java/com/duckduckgo/newtabpage/impl/view/NewTabPageView.kt

@@ -43,7 +43,6 @@ import com.duckduckgo.newtabpage.impl.databinding.ViewNewTabPageBinding
 import com.duckduckgo.newtabpage.impl.view.NewTabPageViewModel.ViewState
 import dagger.android.support.AndroidSupportInjection
 import javax.inject.Inject
-import kotlinx.coroutines.cancel
 import kotlinx.coroutines.flow.launchIn
 import kotlinx.coroutines.flow.onEach
 import logcat.logcat
@@ -53,6 +52,7 @@ class NewTabPageView @JvmOverloads constructor(
     context: Context,
     attrs: AttributeSet? = null,
     defStyle: Int = 0,
+    private val showLogo: Boolean = true,
 ) : LinearLayout(context, attrs, defStyle) {
 
     @Inject
@@ -96,10 +96,15 @@ class NewTabPageView @JvmOverloads constructor(
         if (viewState.loading) {
             binding.newTabContentShimmer.startShimmer()
         } else {
-            if (viewState.showDax) {
-                binding.ddgLogo.show()
+            if (!showLogo && viewState.showDax) {
+                this.gone()
             } else {
-                binding.ddgLogo.gone()
+                this.show()
+                if (viewState.showDax) {
+                    binding.ddgLogo.show()
+                } else {
+                    binding.ddgLogo.gone()
+                }
             }
 
             if (viewState.sections.isEmpty()) {