Refactor FXIOS-10205 [Swiftlint] Resolve 1 implicitly_unwrapped_optio…

…nal violations in TopTabsViewController (#23835) Resolve warnings in TopTabsViewController using a lazy var instead of implicitly unwrapping
mozilla-mobile · Dec 18, 2024 · 956a5a5 · 956a5a5
1 parent 46718b8
commit 956a5a5
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/firefox-ios/Client/Frontend/Browser/TopTabsViewController.swift b/firefox-ios/Client/Frontend/Browser/TopTabsViewController.swift
@@ -36,7 +36,15 @@ class TopTabsViewController: UIViewController, Themeable, Notifiable, FeatureFla
     // MARK: - Properties
     let tabManager: TabManager
     weak var delegate: TopTabsDelegate?
-    private var topTabDisplayManager: TopTabDisplayManager!
+
+    private lazy var topTabDisplayManager = TopTabDisplayManager(
+        collectionView: collectionView,
+        tabManager: tabManager,
+        tabDisplayer: self,
+        reuseID: TopTabCell.cellIdentifier,
+        profile: profile
+    )
+
     var tabCellIdentifier: TabDisplayerDelegate.TabCellIdentifier = TopTabCell.cellIdentifier
     var profile: Profile
     var themeManager: ThemeManager
@@ -122,11 +130,6 @@ class TopTabsViewController: UIViewController, Themeable, Notifiable, FeatureFla
         self.themeManager = themeManager
         self.notificationCenter = notificationCenter
         super.init(nibName: nil, bundle: nil)
-        topTabDisplayManager = TopTabDisplayManager(collectionView: self.collectionView,
-                                                    tabManager: self.tabManager,
-                                                    tabDisplayer: self,
-                                                    reuseID: TopTabCell.cellIdentifier,
-                                                    profile: profile)
         collectionView.dataSource = topTabDisplayManager
         collectionView.delegate = tabLayoutDelegate
     }