Rename isEnabled to canHandleMessaging

Author: CrisBarreiro

content-scope-scripts/content-scope-scripts-impl/src/main/java/com/duckduckgo/contentscopescripts/impl/messaging/ContentScopeScriptsWebMessagingPlugin.kt

@@ -47,7 +47,7 @@ class ContentScopeScriptsWebMessagingPlugin @Inject constructor(
         override val objectName: String
             get() = "contentScopeAdsjs"
 
-        override suspend fun isEnabled(): Boolean {
+        override suspend fun canHandleMessaging(): Boolean {
             return webViewCompatContentScopeScripts.isEnabled()
         }
 

js-messaging/js-messaging-api/src/main/java/com/duckduckgo/js/messaging/api/WebMessagingPlugin.kt

@@ -53,10 +53,10 @@ interface WebMessagingPluginStrategy {
     val objectName: String
 
     /**
-     * Determines whether messaging actions should proceed (i.e. by checking feature flags).
-     * @return true if messaging is allowed, false otherwise
+     * Determines whether messaging can be handled (i.e. by checking feature flags).
+     * @return true if messaging can be handled for this plugin, false otherwise
      */
-    suspend fun isEnabled(): Boolean
+    suspend fun canHandleMessaging(): Boolean
 
     /**
      * Provides the list of message handlers to process incoming messages.

js-messaging/js-messaging-impl/src/main/java/com/duckduckgo/js/messaging/impl/RealWebMessagingDelegate.kt

@@ -154,7 +154,7 @@ class RealWebMessagingDelegate @Inject constructor(
                 webView: WebView,
             ) {
                 appCoroutineScope.launch {
-                    if (!strategy.isEnabled()) {
+                    if (!strategy.canHandleMessaging()) {
                         return@launch
                     }
 
@@ -180,7 +180,7 @@ class RealWebMessagingDelegate @Inject constructor(
                 webView: WebView,
             ) {
                 appCoroutineScope.launch {
-                    if (!strategy.isEnabled()) return@launch
+                    if (!strategy.canHandleMessaging()) return@launch
                     runCatching {
                         return@runCatching webViewCompatWrapper.removeWebMessageListener(webView, strategy.objectName)
                     }.getOrElse { exception ->
@@ -210,7 +210,7 @@ class RealWebMessagingDelegate @Inject constructor(
             override fun postMessage(subscriptionEventData: SubscriptionEventData) {
                 runCatching {
                     appCoroutineScope.launch {
-                        if (!strategy.isEnabled()) {
+                        if (!strategy.canHandleMessaging()) {
                             return@launch
                         }