Skip to content

Commit

Permalink
Remove non essential UI + change to Task(priority:) for background te…
Browse files Browse the repository at this point in the history
…lemetry queue
  • Loading branch information
lmarceau committed Feb 4, 2025
1 parent b172a8d commit 1a33cf9
Showing 1 changed file with 2 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -751,7 +751,7 @@ class BrowserViewController: UIViewController,
setupEssentialUI()
subscribeToRedux()

DispatchQueue.global(qos: .background).async {
Task(priority: .background) {
// App startup telemetry accesses RustLogins to queryLogins, shouldn't be on the app startup critical path
self.trackStartupTelemetry()
}
Expand All @@ -762,12 +762,6 @@ class BrowserViewController: UIViewController,
setupConstraints()
setupNotifications()

DispatchQueue.main.async {
self.setupNonEssentialUI()
}
}

private func setupNonEssentialUI() {
overlayManager.setURLBar(urlBarView: urlBarView)

// Update theme of already existing views
Expand All @@ -784,7 +778,7 @@ class BrowserViewController: UIViewController,
setupAccessibleActions()

clipboardBarDisplayHandler = ClipboardBarDisplayHandler(prefs: profile.prefs,
tabManager: tabManager)
tabManager: tabManager)
clipboardBarDisplayHandler?.delegate = self

navigationToolbarContainer.toolbarDelegate = self
Expand Down

0 comments on commit 1a33cf9

Please sign in to comment.