use ivar for tld

duckduckgo · Jan 17, 2025 · 74db5ab · 74db5ab
1 parent 7a3b779
commit 74db5ab
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/DuckDuckGo/TextZoomCoordinator.swift b/DuckDuckGo/TextZoomCoordinator.swift
@@ -51,7 +51,7 @@ protocol TextZoomCoordinating {
     func onTextZoomChange(applyToWebView webView: WKWebView)
 
     /// Shows a text zoom editor for the current webview. Does nothing if the feature is disabled.
-    func showTextZoomEditor(inController controller: UIViewController, forWebView webView: WKWebView)
+    func showTextZoomEditor(inController controller: UIViewController, forWebView webView: WKWebView) async
 
     /// Creates a browsing menu entry for the given link.  Returns nil if the feature is disabled.
     func makeBrowsingMenuEntry(forLink: Link, inController controller: UIViewController, forWebView webView: WKWebView) -> BrowsingMenuEntry?
@@ -63,27 +63,29 @@ final class TextZoomCoordinator: TextZoomCoordinating {
     let appSettings: AppSettings
     let storage: TextZoomStoring
     let featureFlagger: FeatureFlagger
+    let tld: TLD
 
     var isEnabled: Bool {
         featureFlagger.isFeatureOn(.textZoom)
     }
 
-    init(appSettings: AppSettings, storage: TextZoomStoring, featureFlagger: FeatureFlagger) {
+    init(appSettings: AppSettings, storage: TextZoomStoring, featureFlagger: FeatureFlagger, tld: TLD = TLD()) {
         self.appSettings = appSettings
         self.storage = storage
         self.featureFlagger = featureFlagger
+        self.tld = tld
     }
 
     func textZoomLevel(forHost host: String?) -> TextZoomLevel {
-        let domain = TLD().eTLDplus1(host) ?? ""
+        let domain = tld.eTLDplus1(host) ?? ""
         // If the webview returns no host then there won't be a setting for a blank string anyway.
         return storage.textZoomLevelForDomain(domain)
             // And if there's no setting for whatever domain is passed in, use the app default
             ?? appSettings.defaultTextZoomLevel
     }
 
     func set(textZoomLevel level: TextZoomLevel, forHost host: String?) {
-        guard let domain = TLD().eTLDplus1(host) else { return }
+        guard let domain = tld.eTLDplus1(host) else { return }
         if level == appSettings.defaultTextZoomLevel {
             storage.removeTextZoomLevel(forDomain: domain)
         } else {
@@ -118,7 +120,7 @@ final class TextZoomCoordinator: TextZoomCoordinating {
     func showTextZoomEditor(inController controller: UIViewController, forWebView webView: WKWebView) {
         guard isEnabled else { return }
 
-        guard let domain = TLD().eTLDplus1(webView.url?.host) else { return }
+        guard let domain = tld.eTLDplus1(webView.url?.host) else { return }
         let zoomController = TextZoomController(
             domain: domain,
             coordinator: self,
@@ -147,7 +149,7 @@ final class TextZoomCoordinator: TextZoomCoordinating {
         guard isEnabled else { return nil }
 
         let label: String
-        if let domain = TLD().eTLDplus1(link.url.host),
+        if let domain = tld.eTLDplus1(link.url.host),
            let level = storage.textZoomLevelForDomain(domain) {
             label = UserText.textZoomWithPercentForMenuItem(level.rawValue)
         } else {