firefox-ios icon indicating copy to clipboard operation
firefox-ios copied to clipboard

Homepage is displayed in the same tab as "github.com" after +4 hours

Open SimonBasca opened this issue 2 years ago • 8 comments

Steps to reproduce

  1. Open a page (e.g "github.com")
  2. Close the app from app switcher
  3. Change time to +4 hours

Expected behavior

  • When the app is opened for the first time in 4 hours after visiting a page, it opens it should create a new tab to the homepage. If last tab was the homepage, it should not create a new tab.

Actual behavior

  • Homepage is displayed in the same tab as "github.com"

Device & build information

  • Device: iPad Pro (15.4)
  • Build version: 102.0 (11255)

Notes

  • This issue is also reproducible on iPhone
  • See : https://mozilla-hub.atlassian.net/browse/FXIOS-1739 Attachments:

https://user-images.githubusercontent.com/32130829/174771555-60d56e77-5398-430f-a074-8610a5e79bfe.mp4

┆Issue is synchronized with this Jira Task

SimonBasca avatar Jun 21 '22 09:06 SimonBasca

➤ Roux Buciu commented:

After some investigation, it’s become clear that issue has been happening (in prod) since the StartAtHome feature was released. The decision was taken for it to not be a release blocker for v102.

data-sync-user avatar Jun 22 '22 13:06 data-sync-user

@SimonBasca can you try in v103 / v104 and lmk if this is still happening on iPhone

If you can reproduce this on Simulator then that would be really helpful as well

nbhasin2 avatar Aug 03 '22 21:08 nbhasin2

This issue is still reproducible on v103.0 (13910) and v104 (14880). On simulator the issue is not reproducible

SimonBasca avatar Aug 04 '22 13:08 SimonBasca

@SimonBasca I followed the same steps on iPhone but couldn't reproduce it. Is it on iPad only?

Do you have a video for iPhone as well?

nbhasin2 avatar Aug 04 '22 14:08 nbhasin2

I couldn't reproduce either following the STR, I tried on iPhone and iPad on main and v104 (14880). If you can reproduce still @SimonBasca we might need to sync because it seems we are missing some configuration or step

yoanarios avatar Aug 04 '22 19:08 yoanarios

The issue is 100% reproducible on my side. Just note that if Firefox is left in background (without closing it from app switcher) before changing the time, the issue will not be reproducible.

https://user-images.githubusercontent.com/32130829/183071431-c97c34bc-241d-496f-a8b0-21206699e36f.mp4

Just ping me on slack if you still have issues reproducing and we can sync.

SimonBasca avatar Aug 05 '22 11:08 SimonBasca

I’m able to reproduce this issue constantly when following this steps:

  1. Have only one Homepage tab open.
  2. Visit github.com in that tab.
  3. Force close the app.
  4. Go to iOS Settings and move the time fwd 4 hrs +
  5. Open Firefox.

Expected: Two tabs are open. One with Github and one with the Homepage.

Result: The Homepage opens in the same tab where github.com was instead of a new tab. The back button in the tab is active but tapping on it doesn't do anything. Long pressing on the back button shows the tab history as shown bellow.

Skype_Picture_2022_08_23T10_15_39_856Z

csuciu avatar Aug 23 '22 10:08 csuciu

➤ Nishant Bhasin commented:

Winnie Teichmann try it out and let me know if you have better luck reproducing this and we can even pair to fix this bug.

data-sync-user avatar Sep 14 '22 15:09 data-sync-user

➤ Nishant Bhasin commented:

Here are the log values of isFxAHomeTab where because we are only comparing it against internal:// there is a an issue where some urls always hit home even though they are not.

TODO FIX: The fix is to account for the urls which are actually homepage urls and not just a prefix for internal://

default 11:54:05.298363-0400 Client NB -- url 1 -- internal://local/sessionrestore?history=%7B%0A%20%20%22history%22%20:%20%5B%0A%20%20%20%20%22internal:%5C/%5C/local%5C/about%5C/home%23panel%3D0%22,%0A%20%20%20%20%22https:%5C/%5C/github.com%5C/%22%0A%20%20%5D,%0A%20%20%22currentPage%22%20:%200%0A%7D

default 11:54:05.493810-0400 Client NB -- url 1 -- internal://local/sessionrestore?url=https://github.com/

default 11:54:05.591276-0400 Client NB -- url 1 -- internal://local/about/home!Screen Shot 2022-10-17 at 2.55.38 AM.png|width=976,height=333!

data-sync-user avatar Oct 17 '22 06:10 data-sync-user

➤ Catalin Suciu commented:

This issue is no longer reproducible. Tested on v9000 (20761).

data-sync-user avatar Oct 25 '22 10:10 data-sync-user