report WebView support for MULTI_PROFILE and DELETE_BROWSING_DATA features (#7061)

Task/Issue URL: https://app.asana.com/1/137249556945/project/72649045549333/task/1211819474504985?focus=true ### Description Adds a daily pixel that report WebView support for `MULTI_PROFILE` and `DELETE_BROWSING_DATA` features. ### Steps to test this PR - [ ] Set your logcat to filter for: `message:webview_capabilities` - [ ] Launch the app. - [ ] Verify that `webview_capabilities` pixel is not sent. - [ ] Go to developer settings and enable `reportWebViewCapabilities`. - [ ] Force close and restart the app. - [ ] Verify that `webview_capabilities` pixel is sent with params: `version={your webview version}, multi_profile={true/false}, delete_browsing_data={true/false}, petal=true` - [ ] Background and reopen the app. - [ ] Verify that `webview_capabilities` pixel is ignored.

Author: LukasPaczos

app/src/main/java/com/duckduckgo/app/browser/RealWebViewCapabilityChecker.kt

@@ -16,30 +16,64 @@
 
 package com.duckduckgo.app.browser
 
+import androidx.lifecycle.LifecycleOwner
 import androidx.webkit.WebViewFeature
 import com.duckduckgo.app.browser.api.WebViewCapabilityChecker
 import com.duckduckgo.app.browser.api.WebViewCapabilityChecker.WebViewCapability
 import com.duckduckgo.app.browser.api.WebViewCapabilityChecker.WebViewCapability.DocumentStartJavaScript
 import com.duckduckgo.app.browser.api.WebViewCapabilityChecker.WebViewCapability.WebMessageListener
+import com.duckduckgo.app.di.AppCoroutineScope
+import com.duckduckgo.app.lifecycle.MainProcessLifecycleObserver
+import com.duckduckgo.app.pixels.remoteconfig.AndroidBrowserConfigFeature
+import com.duckduckgo.app.statistics.pixels.Pixel
 import com.duckduckgo.browser.api.WebViewVersionProvider
 import com.duckduckgo.common.utils.DispatcherProvider
 import com.duckduckgo.common.utils.extensions.compareSemanticVersion
+import com.duckduckgo.common.utils.plugins.pixel.PixelParamRemovalPlugin
+import com.duckduckgo.common.utils.plugins.pixel.PixelParamRemovalPlugin.PixelParameter
 import com.duckduckgo.di.scopes.AppScope
 import com.squareup.anvil.annotations.ContributesBinding
+import com.squareup.anvil.annotations.ContributesMultibinding
+import dagger.SingleInstanceIn
+import kotlinx.coroutines.CoroutineScope
+import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
 import javax.inject.Inject
 
-@ContributesBinding(AppScope::class)
+@ContributesMultibinding(AppScope::class, MainProcessLifecycleObserver::class)
+@ContributesBinding(AppScope::class, WebViewCapabilityChecker::class)
+@SingleInstanceIn(AppScope::class)
 class RealWebViewCapabilityChecker @Inject constructor(
+ @AppCoroutineScope private val coroutineScope: CoroutineScope,
  private val dispatchers: DispatcherProvider,
  private val webViewVersionProvider: WebViewVersionProvider,
-) : WebViewCapabilityChecker {
+ private val pixel: Pixel,
+ private val androidBrowserConfigFeature: AndroidBrowserConfigFeature,
+) : WebViewCapabilityChecker, MainProcessLifecycleObserver {
  override suspend fun isSupported(capability: WebViewCapability): Boolean =
  when (capability) {
  DocumentStartJavaScript -> isDocumentStartJavaScriptSupported()
  WebMessageListener -> isWebMessageListenerSupported()
  }
 
+ override fun onStart(owner: LifecycleOwner) {
+ reportWebViewCapabilities()
+ }
+
+ private fun reportWebViewCapabilities() {
+ coroutineScope.launch(dispatchers.io()) {
+ if (androidBrowserConfigFeature.reportWebViewCapabilities().isEnabled()) {
+ val params = mapOf(
+ "version" to webViewVersionProvider.getFullVersion(),
+ "multi_profile" to isMultiProfileSupported().toString(),
+ "delete_browsing_data" to isDeleteBrowsingDataSupported().toString(),
+ "petal" to "true",
+ )
+ pixel.fire(pixel = WebViewCapabilityPixelName.WEBVIEW_CAPABILITIES, parameters = params, type = Pixel.PixelType.Daily())
+ }
+ }
+ }
+
  private suspend fun isWebMessageListenerSupported(): Boolean =
  withContext(dispatchers.io()) {
  webViewVersionProvider
@@ -50,8 +84,25 @@ class RealWebViewCapabilityChecker @Inject constructor(
 
  private fun isDocumentStartJavaScriptSupported(): Boolean = WebViewFeature.isFeatureSupported(WebViewFeature.DOCUMENT_START_SCRIPT)
 
+ private fun isMultiProfileSupported(): Boolean = WebViewFeature.isFeatureSupported(WebViewFeature.MULTI_PROFILE)
+
+ private fun isDeleteBrowsingDataSupported(): Boolean = WebViewFeature.isFeatureSupported(WebViewFeature.DELETE_BROWSING_DATA)
+
  companion object {
  // critical fixes didn't exist until this WebView version. See https://issues.chromium.org/issues/338340758#comment42
  private const val WEB_MESSAGE_LISTENER_WEBVIEW_VERSION = "126.0.6478.40"
  }
 }
+
+enum class WebViewCapabilityPixelName(override val pixelName: String) : Pixel.PixelName {
+ WEBVIEW_CAPABILITIES("webview_capabilities"),
+}
+
+@ContributesMultibinding(AppScope::class)
+class WebViewCapabilityPixelParamRemovalPlugin @Inject constructor() : PixelParamRemovalPlugin {
+ override fun names(): List<Pair<String, Set<PixelParameter>>> {
+ return listOf(
+ WebViewCapabilityPixelName.WEBVIEW_CAPABILITIES.pixelName to PixelParameter.removeAll(),
+ )
+ }
+}

app/src/main/java/com/duckduckgo/app/pixels/remoteconfig/AndroidBrowserConfigFeature.kt

@@ -194,4 +194,12 @@ interface AndroidBrowserConfigFeature {
  */
  @Toggle.DefaultValue(DefaultFeatureValue.FALSE)
  fun splitOmnibar(): Toggle
+
+ /**
+ * @return `true` when the remote config has the global "repotWebViewCapabilities" androidBrowserConfig
+ * sub-feature flag enabled
+ * If the remote feature is not present defaults to `false`
+ */
+ @Toggle.DefaultValue(DefaultFeatureValue.FALSE)
+ fun reportWebViewCapabilities(): Toggle
 }