Ref #4221: Set correct preference value for BraveSearch default.

brave · Oct 13, 2021 · af03ba8 · af03ba8
1 parent 40f1a37
commit af03ba8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/Client/Frontend/Browser/Search/InitialSearchEngines.swift b/Client/Frontend/Browser/Search/InitialSearchEngines.swift
@@ -26,6 +26,15 @@ class InitialSearchEngines {
             }
         }
 
+        /// Due to legacy reasons, the search engine name may need to use an old value.
+        /// This is caused because we use 'display name' as a preference key.
+        var legacyName: String? {
+            switch self {
+            case .braveSearch: return "Brave Search beta"
+            default: return nil
+            }
+        }
+
         func excludedFromOnboarding(for locale: Locale) -> Bool {
             switch self {
             case .braveSearch: return true

diff --git a/Client/Frontend/Browser/Search/SearchEngines.swift b/Client/Frontend/Browser/Search/SearchEngines.swift
@@ -70,7 +70,8 @@ class SearchEngines {
     }
 
     func searchEngineSetup() {
-        setInitialDefaultEngine(initialSearchEngines.defaultSearchEngine.rawValue)
+        let engine = initialSearchEngines.defaultSearchEngine
+        setInitialDefaultEngine(engine.legacyName ?? engine.rawValue)
     }
 
     /// If no engine type is specified this method returns search engine for regular browsing.