WebView sunset dialog removal

Author: truefedex

app/src/main/java/com/phlox/tvwebbrowser/Config.kt

@@ -2,6 +2,7 @@ package com.phlox.tvwebbrowser
 
 import android.annotation.SuppressLint
 import android.content.SharedPreferences
+import android.os.Build
 import com.phlox.tvwebbrowser.utils.Utils
 import com.phlox.tvwebbrowser.utils.observable.ObservableValue
 import org.mozilla.geckoview.GeckoRuntimeSettings
@@ -34,15 +35,30 @@ class Config(val prefs: SharedPreferences) {
         const val NOTIFICATION_ABOUT_ENGINE_CHANGE_SHOWN_KEY = "notification_about_engine_change_shown"
         const val APP_VERSION_CODE_MARK_KEY = "app_version_code_mark"
 
+        const val ENGINE_GECKO_VIEW = "GeckoView"
+        const val ENGINE_WEB_VIEW = "WebView"
+
         const val DEFAULT_ADBLOCK_LIST_URL = "https://easylist.to/easylist/easylist.txt"
         val SearchEnginesTitles = arrayOf("Google", "Bing", "Yahoo!", "DuckDuckGo", "Yandex", "Startpage", "Custom")
         val SearchEnginesNames = arrayOf("google", "bing", "yahoo", "ddg", "yandex", "startpage", "custom")
         val SearchEnginesURLs = listOf("https://www.google.com/search?q=[query]", "https://www.bing.com/search?q=[query]",
             "https://search.yahoo.com/search?p=[query]", "https://duckduckgo.com/?q=[query]",
             "https://yandex.com/search/?text=[query]", "https://www.startpage.com/sp/search?query=[query]", "")
-        val SupportedWebEngines = arrayOf("GeckoView", "WebView")
+        val SupportedWebEngines = arrayOf(ENGINE_GECKO_VIEW, ENGINE_WEB_VIEW)
         const val HOME_PAGE_URL = "https://tvbro.phlox.dev/appcontent/home/"
         //const val HOME_PAGE_URL = "http://10.0.2.2:5000/appcontent/home/"
+
+        fun canRecommendGeckoView(): Boolean {
+            var recommendedGeckoView = false
+            val deviceRAM = Utils.memInfo(TVBro.instance).totalMem
+            val cpuHas64Bit = Build.SUPPORTED_64_BIT_ABIS.isNotEmpty()
+            val cpuCores = Runtime.getRuntime().availableProcessors()
+            val threeGB = 3000000000L//~3GB minus protected memory
+            if (deviceRAM >= threeGB && cpuHas64Bit && cpuCores >= 6) {
+                recommendedGeckoView = true
+            }
+            return recommendedGeckoView
+        }
     }
 
     enum class Theme {
@@ -125,7 +141,14 @@ class Config(val prefs: SharedPreferences) {
     var searchEngineURL = ObservableStringPreference(SearchEnginesURLs[0], SEARCH_ENGINE_URL_PREF_KEY)
 
     var webEngine: String
-        get() = prefs.getString(WEB_ENGINE, SupportedWebEngines[0])!!
+        get() {
+            if (!prefs.contains(WEB_ENGINE)) {
+                prefs.edit().putString(WEB_ENGINE, if (canRecommendGeckoView())
+                    SupportedWebEngines[0] else SupportedWebEngines[1]).apply()
+            }
+
+            return prefs.getString(WEB_ENGINE, SupportedWebEngines[0])!!
+        }
         set(value) {
             prefs.edit().putString(WEB_ENGINE, value).apply()
         }

app/src/main/java/com/phlox/tvwebbrowser/activity/main/MainActivity.kt

@@ -418,7 +418,6 @@ open class MainActivity : AppCompatActivity(), ActionBar.Callback {
     }
 
     private fun loadState() = lifecycleScope.launch(Dispatchers.Main) {
-        showWebViewSunsetDialogIfNeeded()
         WebEngineFactory.initialize(this@MainActivity, vb.flWebViewContainer)
 
         vb.progressBarGeneric.visibility = View.VISIBLE
@@ -473,31 +472,6 @@ open class MainActivity : AppCompatActivity(), ActionBar.Callback {
         }
     }
 
-    private suspend fun showWebViewSunsetDialogIfNeeded() {
-        suspendCoroutine{ coroutine ->
-            if ((config.isWebEngineNotSet() || !config.isWebEngineGecko()) &&
-                (config.notificationAboutEngineChangeShown == 0 || config.notificationAboutEngineChangeShown != BuildConfig.VERSION_CODE)) {
-                AlertDialog.Builder(this@MainActivity)
-                    .setTitle(R.string.webview_sunset_dialog_title)
-                    .setMessage(R.string.webview_sunset_dialog_message)
-                    .setPositiveButton(R.string.webview_sunset_dialog_positive_button) { _, _ ->
-                        config.webEngine = Config.SupportedWebEngines[0]
-                    }
-                    .setNegativeButton(R.string.webview_sunset_dialog_negative_button) { _, _ ->
-                        config.webEngine = Config.SupportedWebEngines[1]
-                    }
-                    .setCancelable(false)
-                    .setOnDismissListener {
-                        config.notificationAboutEngineChangeShown = BuildConfig.VERSION_CODE
-                        coroutine.resume(Unit)
-                    }
-                    .show().getButton(DialogInterface.BUTTON_POSITIVE).requestFocus()
-            } else {
-                coroutine.resume(Unit)
-            }
-        }
-    }
-
     private fun openInNewTab(url: String?, index: Int = 0, needToHideMenuOverlay: Boolean = true, navigateImmediately: Boolean): WebEngine? {
         if (url == null) {
             return null

app/src/main/java/com/phlox/tvwebbrowser/activity/main/dialogs/settings/MainSettingsView.kt

@@ -67,23 +67,54 @@ class MainSettingsView @JvmOverloads constructor(
         vb.spWebEngine.onItemSelectedListener = object : AdapterView.OnItemSelectedListener {
             override fun onItemSelected(parent: AdapterView<*>, view: View, position: Int, id: Long) {
                 if (config.webEngine == Config.SupportedWebEngines[position]) return
-                config.webEngine = Config.SupportedWebEngines[position]
-                AlertDialog.Builder(context)
-                        .setTitle(R.string.need_restart)
-                        .setMessage(R.string.need_restart_message)
-                        .setPositiveButton(R.string.exit) { _, _ ->
-                            TVBro.instance.needToExitProcessAfterMainActivityFinish = true
-                            TVBro.instance.needRestartMainActivityAfterExitingProcess = true
-                            activity!!.finish()
+                if (Config.SupportedWebEngines[position] == Config.ENGINE_GECKO_VIEW && !Config.canRecommendGeckoView()) {
+                    AlertDialog.Builder(context)
+                        .setTitle(R.string.warning)
+                        .setMessage(R.string.settings_engine_change_gecko_msg)
+                        .setPositiveButton(R.string.ok) { _, _ ->
+                            config.webEngine = Config.SupportedWebEngines[position]
+                            showRestartDialog()
+                        }
+                        .setNegativeButton(R.string.cancel) { _, _ ->
+                            vb.spWebEngine.setSelection(Config.SupportedWebEngines.indexOf(config.webEngine), false)
+                        }
+                        .show()
+                    return
+                } else if (Config.SupportedWebEngines[position] == Config.ENGINE_WEB_VIEW) {
+                    AlertDialog.Builder(context)
+                        .setTitle(R.string.warning)
+                        .setMessage(R.string.settings_engine_change_webview_msg)
+                        .setPositiveButton(R.string.ok) { _, _ ->
+                            config.webEngine = Config.SupportedWebEngines[position]
+                            showRestartDialog()
+                        }
+                        .setNegativeButton(R.string.cancel) { _, _ ->
+                            vb.spWebEngine.setSelection(Config.SupportedWebEngines.indexOf(config.webEngine), false)
                         }
-                        .setCancelable(false)
                         .show()
+                    return
+                }
+                config.webEngine = Config.SupportedWebEngines[position]
+                showRestartDialog()
             }
 
             override fun onNothingSelected(parent: AdapterView<*>) {}
         }
     }
 
+    private fun showRestartDialog() {
+        AlertDialog.Builder(context)
+            .setTitle(R.string.need_restart)
+            .setMessage(R.string.need_restart_message)
+            .setPositiveButton(R.string.exit) { _, _ ->
+                TVBro.instance.needToExitProcessAfterMainActivityFinish = true
+                TVBro.instance.needRestartMainActivityAfterExitingProcess = true
+                activity!!.finish()
+            }
+            .setCancelable(false)
+            .show()
+    }
+
     private fun initThemeSettingsUI() {
         val webViewSupportsDarkening = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
             WebViewFeature.isFeatureSupported(WebViewFeature.ALGORITHMIC_DARKENING)

app/src/main/java/com/phlox/tvwebbrowser/utils/Utils.kt

@@ -278,4 +278,11 @@ object Utils {
         return (appProcessInfo.importance == RunningAppProcessInfo.IMPORTANCE_FOREGROUND
                 || appProcessInfo.importance == RunningAppProcessInfo.IMPORTANCE_VISIBLE)
     }
+
+    fun memInfo(ctx: Context): ActivityManager.MemoryInfo {
+        val actManager = ctx.getSystemService(Context.ACTIVITY_SERVICE) as ActivityManager
+        val memInfo = ActivityManager.MemoryInfo()
+        actManager.getMemoryInfo(memInfo)
+        return memInfo
+    }
 }

app/src/main/res/values-de/strings.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="UTF-8"?>
+<?xml version='1.0' encoding='utf-8'?>
 <resources>
   <string name="app_name">TV Bro: Webbrowser für TVs</string>
   <string name="error">Fehler</string>
@@ -188,12 +188,8 @@
   <string name="no">NEIN</string>
   <string name="site_asks_to_open_unknown_url">Die Website fordert zum Öffnen einer unbekannten URL auf</string>
   <string name="site_asks_to_open_unknown_url_message">Möchten Sie diese URL in einer externen App öffnen?</string>
-  <string name="webview_sunset_dialog_title">TV Bro wechselt die Engine.</string>
-  <string name="webview_sunset_dialog_message">Hallo! TV Bro wechselt die Engine. Seit jeher verwendet TV Bro die Systemkomponente WebView, um Webinhalte anzuzeigen. Dies führte jedoch immer wieder zu diversen Problemen, da das WebView selbstständig aktualisiert wird und oft eigene Fehler aufweist. Jetzt ändert TV Bro die Engine auf GeckoView, das eingebaut werden kann, damit es solche Probleme nicht mehr gibt. Leider ist das interne Datenformat dieser Engines nicht kompatibel, sodass die Daten offener Sessions bei der Umstellung verloren gehen. Daher wird TV Bro noch einige Zeit beide Engines unterstützen (man kann in den Einstellungen umschalten).
-
-Sind Sie also bereit, die GeckoView-Version zu verwenden, oder möchten Sie vorerst auf der WebView bleiben?</string>
-  <string name="webview_sunset_dialog_positive_button">Ja, beginnen wir mit der Verwendung einer neuen Engine</string>
-  <string name="webview_sunset_dialog_negative_button">Nein, verwende vorerst die alte</string>
   <string name="err_no_app_to_handle_url">Kann keine geeignete Anwendung finden, um diese URL zu verarbeiten</string>
   <string name="home_page_links">Links auf der Homepage</string>
-</resources>
+  <string name="settings_engine_change_gecko_msg">Von GeckoView empfohlene Systemanforderungen: 3 GB RAM, 64-Bit-Octa-Core-CPU. Möchtest du fortfahren\?</string>
+  <string name="settings_engine_change_webview_msg">WebView wird in zukünftigen Updates weniger Funktionen haben. Möchtest du fortfahren\?</string>
+</resources>

app/src/main/res/values-fa/strings.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="UTF-8"?>
+<?xml version='1.0' encoding='utf-8'?>
 <resources>
   <string name="app_name">"TV Bro: مرورگر وب تلویزیون"</string>
   <string name="error">"خطا"</string>
@@ -192,12 +192,8 @@
   <string name="no">خیر</string>
   <string name="site_asks_to_open_unknown_url">سایت می خواهد URL ناشناخته را باز کند</string>
   <string name="site_asks_to_open_unknown_url_message">آیا می خواهید این URL را در برنامه خارجی باز کنید؟</string>
-  <string name="webview_sunset_dialog_title">تی وی برادر موتور را عوض می کند.</string>
-  <string name="webview_sunset_dialog_message">سلام! تی وی برادر موتور را عوض می کند. در تمام این مدت، TV Bro از مؤلفه سیستم WebView برای نمایش محتوای وب استفاده کرده است. اما این به دلیل اینکه WebView به طور مستقل به روز می شود و اغلب دارای اشکالات خاص خود است، همیشه به مشکلات مختلفی منجر می شود. اکنون TV Bro موتور را به GeckoView تغییر می دهد که می تواند در آن تعبیه شود بنابراین چنین مشکلاتی وجود نخواهد داشت. متأسفانه، فرمت داده های داخلی این موتورها سازگار نیست، بنابراین داده های جلسات باز در طول انتقال از بین می روند. بنابراین، برای مدتی TV Bro همچنان از هر دو موتور پشتیبانی می کند (می توانید در تنظیمات تغییر دهید).
-
-بنابراین آیا آماده استفاده از نسخه GeckoView هستید یا می خواهید در حال حاضر در WebView بمانید؟</string>
-  <string name="webview_sunset_dialog_positive_button">بله، بیایید استفاده از موتور جدید را شروع کنیم</string>
-  <string name="webview_sunset_dialog_negative_button">نه، فعلا از قدیمی استفاده کنید</string>
   <string name="err_no_app_to_handle_url">نمی توان برنامه مناسبی برای مدیریت این URL پیدا کرد</string>
   <string name="home_page_links">لینک ها در صفحه اصلی</string>
-</resources>
+  <string name="settings_engine_change_gecko_msg">سیستم مورد نیاز توصیه شده GeckoView: 3 گیگابایت رم، CPU هشت هسته ای 64 بیتی. می خواهید ادامه دهید؟</string>
+  <string name="settings_engine_change_webview_msg">WebView در به روز رسانی های بعدی ویژگی های کمتری خواهد داشت. می خواهید ادامه دهید؟</string>
+</resources>

app/src/main/res/values-it/strings.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="UTF-8"?>
+<?xml version='1.0' encoding='utf-8'?>
 <resources>
   <string name="app_name">TV Bro: TV Web Browser</string>
   <string name="error">Errore</string>
@@ -188,12 +188,8 @@
   <string name="no">NO</string>
   <string name="site_asks_to_open_unknown_url">Il sito chiede di aprire un URL sconosciuto</string>
   <string name="site_asks_to_open_unknown_url_message">Vuoi aprire questo URL in un&amp;#39;app esterna?</string>
-  <string name="webview_sunset_dialog_title">TV Bro cambia motore.</string>
-  <string name="webview_sunset_dialog_message">Ciao! TV Bro cambia motore. Per tutto questo tempo, TV Bro ha utilizzato il componente di sistema WebView per visualizzare i contenuti web. Ma questo ha sempre portato a vari problemi dovuti al fatto che WebView viene aggiornato in modo indipendente e spesso presenta i propri bug. Ora TV Bro sta cambiando il motore in GeckoView, che può essere integrato, quindi non ci saranno problemi di questo tipo. Sfortunatamente, il formato dei dati interni di questi motori non è compatibile, quindi i dati delle sessioni aperte andranno persi durante la transizione. Pertanto, per qualche tempo TV Bro supporterà ancora entrambi i motori (puoi cambiare le impostazioni).
-
-Quindi sei pronto per utilizzare la versione GeckoView o vuoi rimanere su WebView per ora?</string>
-  <string name="webview_sunset_dialog_positive_button">Sì, iniziamo a utilizzare il nuovo motore</string>
-  <string name="webview_sunset_dialog_negative_button">No, usa quello vecchio per ora</string>
   <string name="err_no_app_to_handle_url">Impossibile trovare un&amp;#39;applicazione adatta per gestire questo URL</string>
   <string name="home_page_links">Link in home page</string>
-</resources>
+  <string name="settings_engine_change_gecko_msg">Requisiti di sistema consigliati da GeckoView: 3 GB di RAM, CPU octa-core a 64 bit. Vuoi continuare\?</string>
+  <string name="settings_engine_change_webview_msg">WebView avrà meno funzionalità negli aggiornamenti futuri. Vuoi continuare\?</string>
+</resources>

app/src/main/res/values-iw/strings.xml

@@ -1,8 +1,7 @@
-<?xml version="1.0" encoding="UTF-8"?>
+<?xml version='1.0' encoding='utf-8'?>
 <resources>
   <string name="app_name">"TV Bro: דפדפן לטלויזיה"</string>
   <string name="error">"שגיאה"</string>
-  <!-- Fuzzy -->
   <string name="ok">"OK"</string>
   <string name="loading">"טוען..."</string>
   <string name="url_prompt">"הכנס טקסט לחיפוש או URL"</string>
@@ -192,12 +191,8 @@
   <string name="no">לא</string>
   <string name="site_asks_to_open_unknown_url">האתר מבקש לפתוח כתובת URL לא ידועה</string>
   <string name="site_asks_to_open_unknown_url_message">האם ברצונך לפתוח את כתובת האתר הזו באפליקציה חיצונית?</string>
-  <string name="webview_sunset_dialog_title">TV Bro מחליף מנוע.</string>
-  <string name="webview_sunset_dialog_message">שלום! TV Bro מחליף מנוע. כל הזמן הזה, TV Bro השתמש ברכיב מערכת WebView כדי להציג תוכן אינטרנט. אבל זה כל הזמן הוביל לבעיות שונות בגלל העובדה שה-WebView מתעדכן באופן עצמאי ולעתים קרובות יש לו באגים משלו. כעת TV Bro משנה את המנוע ל- GeckoView שניתן לבנות כך שלא יהיו בעיות כאלה. למרבה הצער, פורמט הנתונים הפנימי של המנועים הללו אינו תואם, כך שהנתונים של הפעלות פתוחות יאבדו במהלך המעבר. לכן, במשך זמן מה TV Bro עדיין יתמוך בשני המנועים (תוכל לעבור בהגדרות).
-
-אז האם אתה מוכן להשתמש בגרסת GeckoView או רוצה להישאר ב-WebView לעת עתה?</string>
-  <string name="webview_sunset_dialog_positive_button">כן, בואו נתחיל להשתמש במנוע חדש</string>
-  <string name="webview_sunset_dialog_negative_button">לא, השתמש בישן לעת עתה</string>
   <string name="err_no_app_to_handle_url">לא ניתן למצוא אפליקציה מתאימה לטיפול בכתובת האתר הזו</string>
   <string name="home_page_links">קישורים בעמוד הבית</string>
-</resources>
+  <string name="settings_engine_change_gecko_msg">דרישות מערכת מומלצות של GeckoView: 3GB RAM, מעבד מתומן ליבות של 64 סיביות. האם אתה רוצה להמשיך\?</string>
+  <string name="settings_engine_change_webview_msg">ל-WebView יהיו פחות תכונות בעדכונים עתידיים. האם אתה רוצה להמשיך\?</string>
+</resources>