Closed
Bug 634784
Opened 14 years ago
Closed 14 years ago
Black squares appear then disappear with hardware acceleration
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
blocking2.0 | --- | betaN+ |
People
(Reporter: zpao, Assigned: mattwoodrow)
References
()
Details
(Whiteboard: [hardblocker])
Attachments
(2 files)
(deleted),
video/webm
|
Details | |
(deleted),
patch
|
joe
:
review+
|
Details | Diff | Splinter Review |
No idea if this is the right place to put this, so please move as necessary.
While using the demo at http://shakenandstirredweb.com/playground/blur/ I noticed that there was a black box appearing as a transition was ending and then it went away.
This goes away when not using hardware acceleration (tested on same machine, and also on a non-accelerated Linux).
From about:support:
Direct2D Enabled: false
DirectWrite Enabled: false
WebGL Renderer: NVIDIA Corporation -- NVIDIA GeForce 9400M OpenGL Engine -- 2.1 NVIDIA-1.6.26GPU
Accelerated Windows: 3/3 OpenGL
Reporter | ||
Comment 1•14 years ago
|
||
Updated•14 years ago
|
blocking2.0: --- → ?
Updated•14 years ago
|
Component: Layout → Graphics
QA Contact: layout → thebes
Comment 2•14 years ago
|
||
Wasn't there a similar bug about this sort of thing appearing at the tail end of transitions?
Comment 3•14 years ago
|
||
Probably another container layer regression. These are fraught with peril, and so the fixes to them need to block beta.
I'm going to look into this asap, but I've got a day of many meetings, so Matt, if you get a chance, please steal it from me.
Assignee: nobody → joe
blocking2.0: ? → betaN+
Keywords: regressionwindow-wanted
Whiteboard: [hardblocker]
Comment 4•14 years ago
|
||
Regression window: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=3470891975c7&tochange=199cb6282554
I suspect bug 586683.
Keywords: regressionwindow-wanted
Updated•14 years ago
|
Whiteboard: [hardblocker] → [hardblocker][eta feb 18]
Assignee | ||
Comment 5•14 years ago
|
||
(In reply to comment #2)
> Wasn't there a similar bug about this sort of thing appearing at the tail end
> of transitions?
Yes, thats bug 630835, but it doesn't appear to be causing this.
Disabling the changes in 586683 fixes the issue for me, working on an actual fix now.
Assignee | ||
Comment 6•14 years ago
|
||
Attachment #513227 -
Flags: review?(joe)
Assignee | ||
Updated•14 years ago
|
Whiteboard: [hardblocker][eta feb 18] → [hardblocker][eta feb 18][has patch]
Assignee | ||
Updated•14 years ago
|
Assignee: joe → matt.woodrow+bugzilla
Comment 7•14 years ago
|
||
(In reply to comment #6)
> Created attachment 513227 [details] [diff] [review]
> Fix
This would be easier to review if it came with a bit more description of what the fix does.
Assignee | ||
Comment 8•14 years ago
|
||
Haha, sorry Jeff!
The region we are passing to FillSurface has already been scaled, so there's no need for FillSurface to scale the context a second time.
I also snuck in a quick fix to the texture size limit check where we were checking against the unscaled size, not scaled.
Comment 9•14 years ago
|
||
Comment on attachment 513227 [details] [diff] [review]
Fix
That helps - thanks, Matt!
This is another one of the things I should have caught - the region to draw scale is just above where we pass the resolution to FillSurface!
Attachment #513227 -
Flags: review?(joe) → review+
Comment 10•14 years ago
|
||
Can we please get a test for this?
Updated•14 years ago
|
Keywords: testcase-wanted
Assignee | ||
Comment 11•14 years ago
|
||
Assignee | ||
Comment 12•14 years ago
|
||
We still get black bars on the edge of transformed layers that will likely cause tests to fail.
I think these are due to scaling the texture and rounding causing sampling to read from the opposite side of the texture. We fixed this on container layers by disabling wrapping, but on thebes layers the texture boundary can be in the middle of a physical texture. I'm not sure how we can fix this other than disabling the texture rotation code, which probably shouldn't be done for this release.
Comment 13•14 years ago
|
||
Should a new bug be filed for the black edges and this bug closed or is there still work here?
Comment 14•14 years ago
|
||
I don't think so, since this bug just got a partial fix.
Comment 15•14 years ago
|
||
Based on comment 12 it sounds like this bug should be marked FIXED and a follow-up bug targeted at the next release filed.
Whiteboard: [hardblocker][eta feb 18][has patch] → [hardblocker][eta feb 18][has patch][landed][fixed?]
Comment 16•14 years ago
|
||
This bug does not just have a partial fix.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [hardblocker][eta feb 18][has patch][landed][fixed?] → [hardblocker]
Reporter | ||
Comment 17•14 years ago
|
||
I'm still seeing small artifacts with this fix. Not the big black square, but a couple lines of dotted black (almost like a border: 1px dotted black). You can see them in the original recording I attached.
Should I get a new bug on file or is there something further that can be done here?
Comment 18•14 years ago
|
||
We see that bug too. Please file a new bug; we'll investigate it.
Comment 19•11 years ago
|
||
A user seems to still be having issues on Windows with an AMD graphics card. ( https://twitter.com/UKFX/status/390120790408323072 ) I think it's time to reopen the bug.
That sounds like bug 837489.
Updated•9 years ago
|
Keywords: testcase-wanted
You need to log in
before you can comment on or make changes to this bug.
Description
•