Bugfix FXIOS-14055 ⁃ [S2S] [iOS26] - Incorrect display of the CFR after a specific scenario

[dicarobinho]

📜 Tickets

Jira ticket
Github issue

💡 Description

Reset summarize timer

🎥 Demos

ScreenRecording_11-10-2025.14-58-14_1.mp4

Before	After

Demo

📝 Checklist

• I filled in the ticket numbers and a description of my work
• I updated the PR name to follow our PR naming guidelines
• I ensured unit tests pass and wrote tests for new code
• If working on UI, I checked and implemented accessibility (Dynamic Text and VoiceOver)
• If adding telemetry, I read the data stewardship requirements and will request a data review
• If adding or modifying strings, I read the guidelines and will request a string review from l10n
• If needed, I updated documentation and added comments to complex code

[mobiletest-ci-bot]

	Messages
📖	Project coverage: 37.31%

🧹 Tidy commit

Just 2 file(s) touched. Thanks for keeping it clean and review-friendly!

🙌 Friday high-five

Thanks for pushing us across the finish line this week! 🙌

❌ Per-file test coverage gate

The following changed file(s) are below 35.0% coverage:

File	Coverage	Required
firefox-ios/Client/Frontend/Browser/BrowserViewController/Extensions/BrowserViewController+TabToolbarDelegate.swift	5.4%	35.0%

Client.app: Coverage: 36.67

File	Coverage
BrowserViewController+TabToolbarDelegate.swift	5.39%	⚠️
BrowserViewController.swift	30.29%	⚠️

Generated by 🚫 Danger Swift against 2c9c97c

[cyndichin]

should we do this for translations as well? Can we make a method like stopAndDismissCFRs so that it encapsulates the CFR related code with a comment on context?

[dicarobinho]

Since PR with dismiss has been reverted, we can't do that refactor. Also, translations CFR don't have this issue, so is not necessary to invalidate it. We can keep the code as it is for now and maybe if there will be more calls for the timer, we can encapsulate all in one method.

[cyndichin]

if there will be more calls for the timer, we can encapsulate all in one method.

We have two here if you want to encapsulate.

resetDataClearanceCFRTimer()
resetSummarizeToolbarCFRTimer()

Have we tested no issues on iPad as well with this change?

[dicarobinho]

I don't see issues on iPad. @cyndichin

[mergify]

This pull request has conflicts when rebasing. Could you fix it @dicarobinho? 🙏

[github-actions]

🚀 PR merged to main, targeting version: 145.3