Closed Bug 1859275 Opened 1 year ago Closed 1 year ago

Flickering with uBlock Origin & JavaScript disabled

Categories

(Core :: DOM: Core & HTML, defect)

Firefox 118
defect

Tracking

()

RESOLVED DUPLICATE of bug 1740499

People

(Reporter: jannayurevna1979, Unassigned, NeedInfo)

Details

Steps to reproduce:

  1. Add www.gov.uk###global-cookie-message filter or simply add AdGuard/uBO – Cookie Notices filter list.
  2. Disable JavaScript (about:config - javascript.enabled -> false).
  3. Go to https://www.gov.uk/
  4. See brief flickering.

Actual results:

A brief flicker occurs when JavaScript is disabled when using uBlock Origin. When JavaScript is enabled, there is no flicker.

Demo: https://user-images.githubusercontent.com/128177553/275344539-8f2ccef0-101c-488d-9154-6842927f107a.mp4

https://github.com/uBlockOrigin/uBlock-issues/issues/2875

Expected results:

No flickering.

The Bugbug bot thinks this bug should belong to the 'Core::DOM: Core & HTML' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: Untriaged → DOM: Core & HTML
Product: Firefox → Core

(In reply to User1 from comment #0)

Steps to reproduce:

  1. Add www.gov.uk###global-cookie-message filter or simply add AdGuard/uBO – Cookie Notices filter list.
  2. Disable JavaScript (about:config - javascript.enabled -> false).
  3. Go to https://www.gov.uk/
  4. See brief flickering.

Actual results:

A brief flicker occurs when JavaScript is disabled when using uBlock Origin. When JavaScript is enabled, there is no flicker.

Demo: https://user-images.githubusercontent.com/128177553/275344539-8f2ccef0-101c-488d-9154-6842927f107a.mp4

https://github.com/uBlockOrigin/uBlock-issues/issues/2875

Expected results:

No flickering.

Sean, can you help me to confirm if this is expected as https://github.com/uBlockOrigin/uBlock-issues/issues/2875#issuecomment-1764564864 suggests? Also if we think it's a valid issue on Gecko, I am not sure if DOM:Core&HTML is the best component.

Flags: needinfo?(sefeng)

I don't think there's bug, though I am not sure if there's no improvement that can be done.

Here's a profile with JS enabled: https://share.firefox.dev/3Q7sBCO

I think the flicking is caused by the initial blank paint. As far as I can tell, there's was a stylesheet loaded before that paint, and that tick was caused by an user font update. I believe we are working as expected though maybe we can avoid that paint. Maybe user font update doesn't need to schedule a paint if parsing has not done yet?

WDYT Emilio?

Severity: -- → S3
Flags: needinfo?(sefeng) → needinfo?(emilio)

So it seems we're trying to unsuppress painting too soon. I was going to try tweaking the nglayout.initialpaint.delay prefs etc, but it seems they no longer have such an effect on most pages due to the fast unsuppress that olli added.

Olli, any idea for this? I suspect it's not super-actionable. I suspect this is effectively just bug 1740499, but let me know if that doesn't seem right.

Status: UNCONFIRMED → RESOLVED
Closed: 1 year ago
Duplicate of bug: 1740499
Flags: needinfo?(emilio) → needinfo?(smaug)
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.