Fix delegate scoping isse and move it to messaging module

Author: CrisBarreiro

js-messaging/js-messaging-impl/build.gradle

@@ -35,6 +35,7 @@ dependencies {
     implementation KotlinX.coroutines.core
     implementation Google.dagger
     implementation Square.retrofit2.converter.moshi
+    implementation AndroidX.webkit
 
     // Testing dependencies
     testImplementation "org.mockito.kotlin:mockito-kotlin:_"

content-scope-scripts/content-scope-scripts-impl/src/main/java/com/duckduckgo/js/messaging/impl/RealAddDocumentStartScriptDelegate.kt renamed to js-messaging/js-messaging-impl/src/main/java/com/duckduckgo/js/messaging/impl/RealAddDocumentStartScriptDelegate.kt

@@ -23,7 +23,7 @@ import com.duckduckgo.app.browser.api.WebViewCapabilityChecker
 import com.duckduckgo.app.di.AppCoroutineScope
 import com.duckduckgo.browser.api.webviewcompat.WebViewCompatWrapper
 import com.duckduckgo.common.utils.DispatcherProvider
-import com.duckduckgo.di.scopes.FragmentScope
+import com.duckduckgo.di.scopes.AppScope
 import com.duckduckgo.js.messaging.api.AddDocumentStartJavaScriptPlugin
 import com.duckduckgo.js.messaging.api.AddDocumentStartJavaScriptScriptStrategy
 import com.duckduckgo.js.messaging.api.AddDocumentStartScriptDelegate
@@ -33,7 +33,7 @@ import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
 
-@ContributesBinding(FragmentScope::class)
+@ContributesBinding(AppScope::class)
 /**
  * Delegate class that implements AddDocumentStartJavaScriptPlugin and handles
  * the common script injection logic