[fenix] For https://github.com/mozilla-mobile/fenix/issues/16790 - Allow top sites to have an empty name when renamed (https://github.com/mozilla-mobile/fenix/pull/17290)

4 years ago · 9a60af122f
parent 4e1f40b41b
commit 9a60af122f
1 changed files with 3 additions and 6 deletions
--- a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/SessionControlController.kt
+++ b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/SessionControlController.kt
@ -321,12 +321,9 @@ class DefaultSessionControlController(
                setTitle(R.string.rename_top_site)
                setView(customLayout)
                setPositiveButton(R.string.top_sites_rename_dialog_ok) { dialog, _ ->
-                    val newTitle = topSiteLabelEditText.text.toString()
-                    if (newTitle.isNotBlank()) {
-                        viewLifecycleScope.launch(Dispatchers.IO) {
-                            with(activity.components.useCases.topSitesUseCase) {
-                                renameTopSites(topSite, newTitle)
-                            }
+                    viewLifecycleScope.launch(Dispatchers.IO) {
+                        with(activity.components.useCases.topSitesUseCase) {
+                            renameTopSites(topSite, topSiteLabelEditText.text.toString())
                        }
                    }
                    dialog.dismiss()