Avoid sending null when dismissing FindInPage.

cmonfortep · cmonfortep · commit 90c48d9e3f1b · 2020-02-08T17:00:39.000+01:00
Latest chromium release 80.0.3987.87 Feb 4 2020 introduced a null check inside AwContents.findAllAsync
diff --git a/app/src/main/java/com/duckduckgo/app/browser/BrowserTabFragment.kt b/app/src/main/java/com/duckduckgo/app/browser/BrowserTabFragment.kt
@@ -512,7 +512,7 @@ class BrowserTabFragment : Fragment(), FindListener, CoroutineScope {
             }
             is Command.DownloadImage -> requestImageDownload(it.url)
             is Command.FindInPageCommand -> webView?.findAllAsync(it.searchTerm)
-            is Command.DismissFindInPage -> webView?.findAllAsync(null)
+            is Command.DismissFindInPage -> webView?.findAllAsync("")
             is Command.ShareLink -> launchSharePageChooser(it.url)
             is Command.CopyLink -> {
                 clipboardManager.primaryClip = ClipData.newPlainText(null, it.url)

Original file line number	Diff line number	Diff line change
`@@ -512,7 +512,7 @@ class BrowserTabFragment : Fragment(), FindListener, CoroutineScope {`
`512`	`512`	`}`
`513`	`513`	`is Command.DownloadImage -> requestImageDownload(it.url)`
`514`	`514`	`is Command.FindInPageCommand -> webView?.findAllAsync(it.searchTerm)`
`515`		`- is Command.DismissFindInPage -> webView?.findAllAsync(null)`
	`515`	`+ is Command.DismissFindInPage -> webView?.findAllAsync("")`
`516`	`516`	`is Command.ShareLink -> launchSharePageChooser(it.url)`
`517`	`517`	`is Command.CopyLink -> {`
`518`	`518`	`clipboardManager.primaryClip = ClipData.newPlainText(null, it.url)`