Closed Bug 1778937 Opened 3 years ago Closed 2 years ago

Page design is broken on coolmod.com

Categories

(GeckoView :: General, defect, P2)

All
Android
defect

Tracking

(firefox103 affected, firefox104 affected, firefox105 affected, firefox107 unaffected, firefox108 unaffected, firefox109 unaffected)

RESOLVED WORKSFORME
Tracking Status
firefox103 --- affected
firefox104 --- affected
firefox105 --- affected
firefox107 --- unaffected
firefox108 --- unaffected
firefox109 --- unaffected

People

(Reporter: ibedregeanu, Unassigned)

Details

(Whiteboard: [geckoview:2022q3] [geckoview:2022q4])

Attachments

(1 file)

From github: https://github.com/mozilla-mobile/focus-android/issues/6280.

Steps to reproduce

  1. Navigate to https://www.coolmod.com/
  2. Observe the page.

Expected behavior

Page design is displayed correctly.

Actual behavior

Page design is broken (CSS, images).

Device information

  • Android device: Google Pixel 5 (Android 12) - 1080 x 2340 pixels, 19.5:9 ratio (~432 ppi density), Samsung Galaxy S8 (Android 9) - 1440 x 2960 pixels, 18.5:9 ratio (~570 ppi density)
  • Focus version: Firefox Focus Nightly 98.0 (🦎 98.0a1-20220124093541)

Notes

  1. The issue is not reproducible on Firefox Nightly 98.0a1 (🦎 98.0a1-20220124093541) and Firefox Focus Release 96.2.0 (🦎 96.0-20220117150835).
  2. The issue is reproducible on Firefox Focus Nightly even if I disable ETP.
  3. Screenshot attached.

Change performed by the Move to Bugzilla add-on.

Oana, can you reproduce this bug in the current versions of Focus Release and Nightly?

Component: GeckoViewExample → General
Flags: needinfo?(oana.arbuzov)
Summary: Page design is broken on coolmod.com → [Focus] Page design is broken on coolmod.com

Since the status is marked as ? for nightly and release, is it ? for beta too?
For more information, please visit auto_nag documentation.

Chris I can still reproduce it on my side.
https://prnt.sc/lumsO9LlhaTT

Tested with:
Browser / Version: Firefox Focus Release 103.1.0 (🦎 102.0-20220718155818), Firefox Focus Nightly 105.0a1 (🦎 105.0a1-20220726094428)
Operating System: Google Pixel 5 (Android 12) - 1080 x 2340 pixels, 19.5:9 ratio (~432 ppi density), Samsung Galaxy S8 (Android 9) - 1440 x 2960 pixels, 18.5:9 ratio (~570 ppi density)

Flags: needinfo?(oana.arbuzov) → needinfo?(cpeterson)

(In reply to Oana Arbuzov [:oanaarbuzov] from comment #3)

Chris I can still reproduce it on my side.

Thanks for confirming. Like you, I can reproduce in Focus (Release 103 and Nightly 105, with or without ETP), but not in Fenix (Release 103 or Nightly 105).

I don't know what the difference is between Focus and Fenix. I thought this might be caused by TCP being enabled in Focus and not Fenix, but the page was still broken in Focus after changing the network.cookie.cookieBehavior pref from 5 (TCP enabled) to 4 (TCP disabled).

I'll share this bug in Android bug triage tomorrow.

Flags: needinfo?(cpeterson)
Summary: [Focus] Page design is broken on coolmod.com → Page design is broken on coolmod.com in Focus, but not Fenix

Even though we could reproduce this bug after disabling ETP, Arturo still suspects this bug is related to Tracking Protection. Maybe there is a Focus bug related to how or when it disables ETP?

Arturo is landing some ETP fixes in GV this week or next. Let's retest in a week after his changes land.

Do we see any content getting blocked in the developer console log?

Component: General → Core

Arturo, did you ETP fixes landed already? Kevin says he can still reproduce this bug in today's Focus Nightly build.

Flags: needinfo?(amejiamarmol)

Yes, it landed. This may require further investigation.

Flags: needinfo?(amejiamarmol)

Iorga or Ionut, since this bug affects Focus but not Fenix, can one of you please investigate? If this bug looks like a problem in GeckoView or Gecko (such as ETP or TCP), feel free to send the bug back to me. I can then find a GeckoView engineer to continue the investigation.

Severity: -- → S3
Flags: needinfo?(ibedregeanu)
Flags: needinfo?(giorga)
Priority: -- → P2
Whiteboard: [geckoview:2022q3]

I succeed to reproduce it also on Firefox Fenix Nightly 105 and GeckoView Example without doing something different, so I don't think is something specific to Focus

Flags: needinfo?(ibedregeanu)

I tried again to reproduce it on Fenix and initially I didn't succeed(coolmod.com was displayed as expected), but after that I cleared the storage, cache to simulate a fresh install and coolmod.com was broken again

I reproduced the bug on GeckoView Example.apk .

Flags: needinfo?(giorga)
Attached video 22-08-10-09-32-33.mp4

(In reply to Ionut Bedregeanu from comment #9)

I succeed to reproduce it also on Firefox Fenix Nightly 105 and GeckoView Example without doing something different, so I don't think is something specific to Focus

Interesting. Thanks for finding that. I'll add this to the GV team's P2 backlog.

Removing "Focus" from the bug summary since Ionut reproduced the bug in Fenix and GeckoView Example.

Summary: Page design is broken on coolmod.com in Focus, but not Fenix → Page design is broken on coolmod.com
Whiteboard: [geckoview:2022q3] → [geckoview:2022q3] [geckoview:2022q4]

This seems to be fixed, similar to https://github.com/mozilla-mobile/focus-android/issues/7812. It would be good to see what change fixed this.

(In reply to Mihai Adrian Carare from comment #15)

This seems to be fixed, similar to https://github.com/mozilla-mobile/focus-android/issues/7812. It would be good to see what change fixed this.

The bug was last reproduced in Focus 105. The page works for me in Focus 106+, so either the website changed or this bug was fixed in 106. Someone could use mozregression to bisect between 105 and 106 to find the fix commit. (I don't know how to use mozregression on Android.)

Since the site is working now, I'll close this bug.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WORKSFORME
Component: Core → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: