Status

()

Core
Layout
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: jrmuizel, Unassigned)

Tracking

({regression})

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

6 years ago
This shows blocky images with current trunk. It looks fine with FF 4. http://ie.microsoft.com/testdrive/Graphics/Transform2D/Default.html

This was not caused by the pixman update.
(Reporter)

Updated

6 years ago
Keywords: regressionwindow-wanted
How do we nominate bugs to block fx5 (in the "need to find what caused this and get rid of it" sense)?
Keywords: regression
It looks like the resolution clamping and automatic rerendering is broken...
The problem here is that we're creating an intermediate surface for a ContainerLayer and ContainerLayers don't know about resolutions. So the ThebesLayer's effective transform is just a translation, but then we scale the ContainerLayer's intermediate surface up a lot when we draw that.

I don't know how this worked in Firefox 4, I'm not aware of any relevant code that's changed.

Comment 4

6 years ago
Probably I may misread STR and the issue(I don't understand "all blocky").

However, I notice one problem.
The image is blurred while image zoom-in. And then the image is repainted with correct resolution.

Regression window ;
Works:
http://hg.mozilla.org/mozilla-central/rev/488ea306526b
Mozilla/5.0 (Windows; Windows NT 6.1; WOW64; rv:2.0b3pre) Gecko/20100801
Minefield/4.0b3pre ID:20100801201130
Fails:
http://hg.mozilla.org/mozilla-central/rev/99807f92a67c
Mozilla/5.0 (Windows; Windows NT 6.1; WOW64; rv:2.0b3pre) Gecko/20100801
Minefield/4.0b3pre ID:20100801221929
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=488ea306526b&tochange=99807f92a67c
In local build on ubuntu10.04(from m-c repository):
build from 9a45bd27ec75 : fails
build from ed9773300a85 : works
Triggered by : 9a45bd27ec75	Matt Woodrow — Bug 505115. Use layers to render active 2D transforms. r=roc,dbaron
And Bug 604899 may be related, 

I'm sorry if I misunderstand it.
(Reporter)

Comment 5

6 years ago
Turns out when I saw this, I was running basic layers in FF4. So there isn't a real regression here. However, basic layers does look much better than accelerated layers.
Thanks Alice, you're absolutely right as usual!

I think BasicLayers is looking better because we choose to use component alpha for the transformed stuff (not sure why actually) and BasicLayers doesn't retain component alpha, so everything just gets rerendered from scratch all the time.

I think the right way to fix this is to move the resolution functionality out of the layer system altogether, as we've discussed in the past to fix other bugs. I'll do that in bug 646607.
Depends on: 646607
No longer depends on: 646607
Depends on: 637852
Should have been fixed in bug 637852. Please retest on a nightly build.

Comment 8

5 years ago
I cannot reproduce on Firefox/10.0.2, 11Beta, Aurora12.0a2 and Nightly13.0a1 anymore.
Thanks Alice. Let's call it fixed then. Please reopen if that is not the case.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Keywords: regressionwindow-wanted
You need to log in before you can comment on or make changes to this bug.