From 80598e291df5cdabed5a0beba057983860c90306 Mon Sep 17 00:00:00 2001 From: Laurie Marceau Date: Thu, 30 Jan 2025 18:00:35 -0500 Subject: [PATCH] Remove redundant call to updateToolbarStateForTraitCollection Move telemetry in the telemetry tracking class --- .../Views/BrowserViewController.swift | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/firefox-ios/Client/Frontend/Browser/BrowserViewController/Views/BrowserViewController.swift b/firefox-ios/Client/Frontend/Browser/BrowserViewController/Views/BrowserViewController.swift index 670c3b771a71..2bde2a3ffea4 100644 --- a/firefox-ios/Client/Frontend/Browser/BrowserViewController/Views/BrowserViewController.swift +++ b/firefox-ios/Client/Frontend/Browser/BrowserViewController/Views/BrowserViewController.swift @@ -764,10 +764,6 @@ class BrowserViewController: UIViewController, // in getting the value. When the delay happens the credit cards might not sync // as the default value is false self.profile.syncManager?.updateCreditCardAutofillStatus(value: autofillCreditCardStatus) - - self.creditCardInitialSetupTelemetry() - - FakespotUtils().addSettingTelemetry() } } @@ -777,7 +773,7 @@ class BrowserViewController: UIViewController, DispatchQueue.main.async { self.overlayManager.setURLBar(urlBarView: self.urlBarView) - self.updateToolbarStateForTraitCollection(self.traitCollection) +// self.updateToolbarStateForTraitCollection(self.traitCollection) // Update theme of already existing views let theme = self.currentTheme() @@ -4359,6 +4355,8 @@ extension BrowserViewController { trackAccessibility() trackNotificationPermission() appStartupTelemetry.sendStartupTelemetry() + creditCardInitialSetupTelemetry() + FakespotUtils().addSettingTelemetry() } func trackAccessibility() {