From 3d797576116f8b43e73aa7643c0673e572f34494 Mon Sep 17 00:00:00 2001 From: Craig Russell Date: Mon, 27 Aug 2018 13:45:01 +0100 Subject: [PATCH] Remove duplicate call to finish() when clearing data from app shortcut --- .../main/java/com/duckduckgo/app/browser/BrowserActivity.kt | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/com/duckduckgo/app/browser/BrowserActivity.kt b/app/src/main/java/com/duckduckgo/app/browser/BrowserActivity.kt index 47aede2e9e24..cd05c16962fa 100644 --- a/app/src/main/java/com/duckduckgo/app/browser/BrowserActivity.kt +++ b/app/src/main/java/com/duckduckgo/app/browser/BrowserActivity.kt @@ -125,8 +125,6 @@ class BrowserActivity : DuckDuckGoActivity() { if (intent.getBooleanExtra(PERFORM_FIRE_ON_ENTRY_EXTRA, false)) { viewModel.onClearRequested() clearPersonalDataAction.clear() - Toast.makeText(applicationContext, R.string.fireDataCleared, Toast.LENGTH_LONG).show() - finish() return } @@ -195,8 +193,8 @@ class BrowserActivity : DuckDuckGoActivity() { fun launchFire() { val dialog = FireDialog(context = this, pixel = pixel, clearPersonalDataAction = clearPersonalDataAction) - dialog.clearStarted = {viewModel.onClearRequested()} - dialog.clearComplete = { viewModel.onClearComplete()} + dialog.clearStarted = { viewModel.onClearRequested() } + dialog.clearComplete = { viewModel.onClearComplete() } dialog.show() }