Skip to content

Commit c6d746a

Browse files
committed
Merge branch 'release/5.40.3'
2 parents 98173a0 + 3a9a1dc commit c6d746a

File tree

3 files changed

+8
-9
lines changed

3 files changed

+8
-9
lines changed

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

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
package com.duckduckgo.app.browser
1818

1919
import android.Manifest
20-
import android.animation.AnimatorSet
2120
import android.animation.LayoutTransition.CHANGING
2221
import android.animation.LayoutTransition.DISAPPEARING
2322
import android.annotation.SuppressLint
@@ -116,8 +115,10 @@ import kotlin.coroutines.CoroutineContext
116115

117116
class BrowserTabFragment : Fragment(), FindListener, CoroutineScope {
118117

118+
private val supervisorJob = SupervisorJob()
119+
119120
override val coroutineContext: CoroutineContext
120-
get() = SupervisorJob() + Dispatchers.Main
121+
get() = supervisorJob + Dispatchers.Main
121122

122123
@Inject
123124
lateinit var webViewClient: BrowserWebViewClient
@@ -963,6 +964,7 @@ class BrowserTabFragment : Fragment(), FindListener, CoroutineScope {
963964
}
964965

965966
override fun onDestroy() {
967+
supervisorJob.cancel()
966968
popupMenu.dismiss()
967969
destroyWebView()
968970
super.onDestroy()
@@ -1189,18 +1191,15 @@ class BrowserTabFragment : Fragment(), FindListener, CoroutineScope {
11891191
val site = viewModel.siteLiveData.value
11901192
val events = site?.trackingEvents
11911193

1192-
val act = activity
1193-
if (act != null) {
1194+
activity?.let { activity ->
11941195
animatorHelper.createLoadedAnimation(
11951196
lastSeenCtaViewState?.cta,
1196-
act,
1197+
activity,
11971198
networksContainer,
11981199
loadingText,
11991200
omnibarViews(),
12001201
events
12011202
)
1202-
} else {
1203-
cancelAllAnimations()
12041203
}
12051204
}
12061205
}

app/src/main/java/com/duckduckgo/app/global/view/DaxDialog.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ class DaxDialog(
7777
}
7878

7979
override fun onDismiss(dialog: DialogInterface) {
80-
dialogText.cancelAnimation()
80+
dialogText?.cancelAnimation()
8181
dismissListener()
8282
super.onDismiss(dialog)
8383
}

app/version/version.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
VERSION=5.40.2
1+
VERSION=5.40.3

0 commit comments

Comments
 (0)