fix: avoid injecting javascript if webview not active (duckduckgo#922)

cmonfortep · web-flow · commit 6f7c7651e6a6 · 2020-09-01T10:48:41.000+02:00
diff --git a/app/src/main/java/com/duckduckgo/app/browser/BrowserTabFragment.kt b/app/src/main/java/com/duckduckgo/app/browser/BrowserTabFragment.kt
@@ -1583,7 +1583,7 @@ class BrowserTabFragment : Fragment(), FindListener, CoroutineScope, DaxDialogLi
                     }
                 }
 
-                if (!viewState.isLoading) {
+                if (!viewState.isLoading && lastSeenBrowserViewState?.browserShowing == true) {
                     swipeRefreshContainer.isRefreshing = false
                     webView?.detectOverscrollBehavior()
                 }

Original file line number	Diff line number	Diff line change
`@@ -1583,7 +1583,7 @@ class BrowserTabFragment : Fragment(), FindListener, CoroutineScope, DaxDialogLi`
`1583`	`1583`	`}`
`1584`	`1584`	`}`
`1585`	`1585`
`1586`		`- if (!viewState.isLoading) {`
	`1586`	`+ if (!viewState.isLoading && lastSeenBrowserViewState?.browserShowing == true) {`
`1587`	`1587`	`swipeRefreshContainer.isRefreshing = false`
`1588`	`1588`	`webView?.detectOverscrollBehavior()`
`1589`	`1589`	`}`