Closed Bug 1741314 Opened 3 years ago Closed 3 years ago

Text in google docs gradually gets darker and darker with experimental WebGL canvas

Categories

(Core :: Graphics, defect)

defect

Tracking

()

VERIFIED FIXED
96 Branch
Tracking Status
firefox96 --- fixed

People

(Reporter: jrmuizel, Assigned: lsalzman)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

Just noticed this today.

Attached image Before

I can reproduce this by switching to the tab and then repeatedly selecting and unselecting the text underneath it.

This only seems to happen if you end up with the canvas renderer.

Ah turns out this was caused by gfx.canvas.accelerated=true

Severity: -- → S4
Summary: Text in google docs gradually gets darker and darker → Text in google docs gradually gets darker and darker with experimental WebGL canvas

I was accidentally using OP_OVER when trying to flush the Skia target to the WebGL context.
This meant any alpha values caused blending with the previous WebGL context contents, rather
than overwriting them entirely which was the actually desired effect.

Incidental finding while debugging this is that we also need to handle empty rects better by
just ignoring them.

Assignee: nobody → lsalzman
Status: NEW → ASSIGNED
Pushed by lsalzman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/84ef54ebd02c
Use OP_SOURCE in DrawTargetWebgl::FlushFromSkia. r=jrmuizel
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
Blocks: gpu-canvas
Flags: qe-verify+

I tried to reproduce the issue on Win10x64 using build 96.0a1 (20211115215316) and setting 'gfx.canvas.accelerated=true', but text in my google document did not get any darker even after switching tabs.
Can you please add OS used?
Is the issue still reproducing for you on latest RC? (Fx96.0) build(https://archive.mozilla.org/pub/firefox/candidates/96.0b9-candidates/build1/win64/en-US/) ?
Thank you.

Flags: needinfo?(jmuizelaar)

This was fixed.

Flags: needinfo?(jmuizelaar)

Closing based on comment#9.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: