Amend as per PR comments

Author: marcosholgado

app/src/androidTest/java/com/duckduckgo/app/email/EmailJavascriptInterfaceTest.kt

@@ -40,7 +40,7 @@ class EmailJavascriptInterfaceTest {
 
     @Before
     fun setup() {
-        testee = EmailJavascriptInterface(mockEmailManager, { counter++ }, mockWebView, DuckDuckGoUrlDetector(), coroutineRule.testDispatcherProvider)
+        testee = EmailJavascriptInterface(mockEmailManager, mockWebView, DuckDuckGoUrlDetector(), coroutineRule.testDispatcherProvider) { counter++ }
     }
 
     @Test

app/src/main/java/com/duckduckgo/app/email/EmailInjector.kt

@@ -35,7 +35,7 @@ class EmailInjectorJs(private val emailManager: EmailManager, private val urlDet
     private val javaScriptInjector = JavaScriptInjector()
 
     override fun addJsInterface(webView: WebView, onTooltipShown: () -> Unit) {
-        webView.addJavascriptInterface(EmailJavascriptInterface(emailManager, onTooltipShown, webView, urlDetector, dispatcherProvider), JAVASCRIPT_INTERFACE_NAME)
+        webView.addJavascriptInterface(EmailJavascriptInterface(emailManager, webView, urlDetector, dispatcherProvider, onTooltipShown), JAVASCRIPT_INTERFACE_NAME)
     }
 
     @UiThread

app/src/main/java/com/duckduckgo/app/email/EmailJavascriptInterface.kt

@@ -24,10 +24,10 @@ import kotlinx.coroutines.runBlocking
 
 class EmailJavascriptInterface(
     private val emailManager: EmailManager,
-    private val showNativeTooltip: () -> Unit,
     private val webView: WebView,
     private val urlDetector: DuckDuckGoUrlDetector,
-    private val dispatcherProvider: DispatcherProvider
+    private val dispatcherProvider: DispatcherProvider,
+    private val showNativeTooltip: () -> Unit
 ) {
 
     private fun isUrlFromDuckDuckGoEmail(): Boolean {