Closed Bug 1973324 Opened 4 months ago Closed 4 months ago

Switched to home when reopening through launcher

Categories

(Firefox for Android :: Homepage, defect, P1)

All
Android
defect

Tracking

()

VERIFIED FIXED
142 Branch
Tracking Status
firefox140 --- unaffected
firefox141 --- verified
firefox142 --- verified

People

(Reporter: royang, Assigned: gl)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression, Whiteboard: [fxdroid][group2])

Attachments

(4 files)

Steps to reproduce

With PBM lock feature disabled:

  1. Open a private tab
  2. Switch to Android OS home screen
  3. Tap on the Firefox Nightly icon in launcher to go back into Firefox Nightly.

Expected behavior

  1. See the last opened private tab.

Actual behavior

  1. Flashes and see private tab home.
  2. Doing the same thing one more time then see normal home.

Device information

  • Firefox version: v141
  • Android device model:
  • Android OS version:

Any additional information?

:Gela This is found on v141.0a1-20250617143356. Not sure if this is already fixed by in progress reviews but thought I'll open the bug just in case.

Flags: needinfo?(gmalekpour)
Flags: needinfo?(gmalekpour)
Summary: PBM lock move to home when reopening through launcher → PBM lock moves to home when reopening through launcher
Blocks: 1971378
Whiteboard: [fxdroid][group2]
Severity: -- → S3

Thanks for filing this. I gave it a go on different builds and here are the results:

  • Can reproduce on the Nightly from Play Store
  • Can reproduce on the most recent Main if I set the build variant to Nightly
  • Cannot reproduce on Beta

After further discussions, we don't think it's related to the PBM changes for the following reasons:

  1. This happens regardless of whether or not PBM lock is enabled
  2. This does not reproduce on Beta where we have all the PBM related changes uplifted to, only happens on Nightly.
  3. There are other similar bugs related to this already filed for Homepage/Toolbar here and here.

So I'm going to move this to the appropriate component and move out of the PBM lock meta.

No longer blocks: 1971378
Component: Privacy → Homepage

Confirmed reverting the change in https://bugzilla.mozilla.org/show_bug.cgi?id=1968048 the issue is resolved. Marking this as the regressor.

Keywords: regression
Regressed by: 1968048
Summary: PBM lock moves to home when reopening through launcher → Switched to home when reopening through launcher
Severity: S3 → S2

:gl, since you are the author of the regressor, bug 1968048, could you take a look?

For more information, please visit BugBot documentation.

Flags: needinfo?(gl)
Assignee: nobody → gl
Status: NEW → ASSIGNED
Flags: needinfo?(gl)
  • Refactors HomeActivity.getModeFromIntentOrLastKnown into BrowsingModeManger. This moves more of the logic related to computing the browsing mode from the provided intent, last known mode settings and number of private tabs opened into BrowsingModeManger and out of the HomeActivity.
  • Introduces a public BrowsingModeManager.updateMode function for updating the browsing mode with a new Intent.
Attachment #9497543 - Attachment description: Bug 1973324 - Part 1: Persist private mode when the user is still on private mode and has private tabs opened on warm start → Bug 1973324 - Part 1: Persists private mode when the user is still in private mode and has private tabs opened on warm start
Flags: qe-verify+
Pushed by gluong@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/6a8d3ef03e14 https://hg.mozilla.org/integration/autoland/rev/974534c78831 Part 1: Persists private mode when the user is still in private mode and has private tabs opened on warm start r=android-reviewers,Roger https://github.com/mozilla-firefox/firefox/commit/2af7989dd9da https://hg.mozilla.org/integration/autoland/rev/370ef56195ba Part 2: Refactor setting the appropriate browsing mode into BrowsingModeManger r=Roger,android-reviewers
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 142 Branch

The patch landed in nightly and beta is affected.
:gl, is this bug important enough to require an uplift?

For more information, please visit BugBot documentation.

Flags: needinfo?(gl)

Verified as fixed in the latest Nightly build (142.0a1 from 2025-07-01);

The last opened tab is displayed when going back to the app with a page open in PB.

Device used: Samsung Galaxy S23 Ultra (Android 15).

Marking as verified on 142.

Status: RESOLVED → VERIFIED
Flags: qe-verify+

Comment on attachment 9497543 [details]
Bug 1973324 - Part 1: Persists private mode when the user is still in private mode and has private tabs opened on warm start

Beta/Release Uplift Approval Request

  • User impact if declined/Reason for urgency: This will persist private mode when the user reopens Fenix when they in a warm start, which would otherwise restart user back in normal browsing mode.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Low risk because it is covered by tests and QA has verified in Nightly.
  • String changes made/needed:
  • Is Android affected?: Yes
Flags: needinfo?(gl)
Attachment #9497543 - Flags: approval-mozilla-beta?
Priority: -- → P1

Gabriel, you requested uplift in the bug only for part 1 of the fix, does part 2 also needs to be uplifted? Thanks

Flags: needinfo?(gl)

Part 2 is not needed for the fix in the uplift. Thanks!

Flags: needinfo?(gl)
Attachment #9497543 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Verified as fixed in the latest Beta build as well (141.0b6);

The last opened tab is displayed when going back to the app with a page open in PB.

Device used: Samsung Galaxy S23 Ultra (Android 15).

Marking as verified on 141.

Verified as fixed in Firefox 141.0b7 with Asus Zenfone 9 (Android 13).

Blocks: 1975089
Regressions: 1977790
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: