Skip to content

Commit 9a234fe

Browse files
committed
Make AdsjsJsMessageCallback an interface
1 parent 59f0b6a commit 9a234fe

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3044,7 +3044,7 @@ class BrowserTabFragment :
30443044
webViewClient.triggerJSInit(it)
30453045
adsJsContentScopeScripts.register(
30463046
it,
3047-
object : AdsjsJsMessageCallback() {
3047+
object : AdsjsJsMessageCallback {
30483048
override fun process(
30493049
featureName: String,
30503050
method: String,

content-scope-scripts/content-scope-scripts-impl/src/test/java/com/duckduckgo/contentscopescripts/impl/messaging/AdsjsContentScopeScriptsJsMessagingTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ class AdsjsContentScopeScriptsJsMessagingTest {
221221
verify(mockWebView).safeRemoveWebMessageListener(any())
222222
}
223223

224-
private val callback = object : AdsjsJsMessageCallback() {
224+
private val callback = object : AdsjsJsMessageCallback {
225225
var counter = 0
226226
override fun process(
227227
featureName: String,

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@ interface AdsjsMessaging {
4141
val allowedDomains: Set<String>
4242
}
4343

44-
abstract class AdsjsJsMessageCallback {
45-
abstract fun process(
44+
interface AdsjsJsMessageCallback {
45+
fun process(
4646
featureName: String,
4747
method: String,
4848
id: String?,

0 commit comments

Comments
 (0)