commit 59e2d6aa313559cb8e9cacdebbb4a0be23bd069e
parent c0fc2725e436c3af48d9cb0e9cb1acff0cfe0699
Author: Ryan VanderMeulen <rvandermeulen@mozilla.com>
Date: Tue, 16 Dec 2025 16:53:46 +0000
Bug 2006362 - Remove unnecessary null-safety checks. r=android-reviewers,mcarare
Differential Revision: https://phabricator.services.mozilla.com/D276677
Diffstat:
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarMiddleware.kt b/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarMiddleware.kt
@@ -429,7 +429,7 @@ class BrowserToolbarMiddleware(
searchTermOrURL = it,
newTab = false,
searchEngine = searchEngine,
- private = browsingModeManager?.mode == Private,
+ private = browsingModeManager.mode == Private,
)
} ?: run {
Logger("BrowserOriginContextMenu").error("Clipboard contains URL but unable to read text")
@@ -587,9 +587,9 @@ class BrowserToolbarMiddleware(
useCases.fenixBrowserUseCases.navigateToHomepage()
} else {
val directions = BrowserFragmentDirections.actionGlobalHome()
- browserAnimator?.captureEngineViewAndDrawStatically {
+ browserAnimator.captureEngineViewAndDrawStatically {
navController.navigate(directions)
- } ?: navController.navigate(directions)
+ }
}
next(action)
}
@@ -1104,12 +1104,12 @@ class BrowserToolbarMiddleware(
): Action = when (toolbarAction) {
ToolbarAction.NewTab -> ActionButtonRes(
drawableResId = iconsR.drawable.mozac_ic_plus_24,
- contentDescription = if (browsingModeManager?.mode == Private) {
+ contentDescription = if (browsingModeManager.mode == Private) {
R.string.home_screen_shortcut_open_new_private_tab_2
} else {
R.string.home_screen_shortcut_open_new_tab_2
},
- onClick = if (browsingModeManager?.mode == Private) {
+ onClick = if (browsingModeManager.mode == Private) {
AddNewPrivateTab(source)
} else {
AddNewTab(source)