Fenix: [Bug] Favicon for 2 pages of the same site do not load at the same time

Created on 30 Aug 2019  路  8Comments  路  Source: mozilla-mobile/fenix

Steps to reproduce

https://www.franceculture.fr/religion-et-spiritualite/mes-quatre-annees-aux-cotes-du-pape-francois?utm_medium=Social&utm_source=Facebook#Echobox=1566724293

  • go back to the Tab list

Expected behavior

Both tabs should show the same favicon

Actual behavior

Only one of the 2 tabs show the correct favicon, the other shows the default one.
Depending on the order you have opened the 2 urls, it will be either one of the other.

image

Device information

  • Android device: Redmin Note 7 Android 9
  • Fenix version: #12420615
E3 P3 engverified ac 馃悶 bug

All 8 comments

I was unable to reproduce this on the latest nightly on Pixel 3 XL (Android 10). I could use a second pair of eyes on this :)

tab icons work

@codrut-topliceanu can you please check with the following 2 urls on your end? Thanks

While https://www.lemonde.fr is ok for me,
https://www.lemonde.fr/culture/article/2020/04/20/de-killing-eve-a-derapages-quatre-bonnes-raisons-de-rester-devant-la-tele_6037241_3246.html is not
Actually, several pages in lemonde.fr, apart from the home page, are failing.
This being said, all other tests I made so far are successful.

@klint Those pages load for me, and their icons look okay, I think. I'm afraid I don't have a Xiaomi Redmi to test it on :(

Actually any page in lemonde.fr apart from the home page will fail.

Do you mean that the pages don't load at all?

Screenshot from Pixel 3xl:

Oops, I think I commented in the wrong issue, sorry for wasting your time :)
And yes, I can confirm that I can't see this very issue happening anymore.
Thanks for your help anyway.

I was able to reproduce the issue on Firefox Preview Nightly 200229 (Build #21500608) using a OnePlus 6T by performing the following steps:

  1. Open https://www.franceculture.fr;
  2. Open any article in a new tab;
  3. Tap the tab counter and wait for both of the favicons to load;
  4. Restart Firefox Preview.

AC fix merged

Verified as fixed on Nightly 7/17 with Samsung Galaxy Note 10(Android 10), and LG G7 FIT (Android 8), following the steps from the description and additional str added.

Was this page helpful?
0 / 5 - 0 ratings