Closed Bug 866567 Opened 7 years ago Closed 7 years ago

mask layers sometimes broken with OMTC?

Categories

(Core :: Graphics: Layers, defect)

21 Branch
defect
Not set

Tracking

()

VERIFIED FIXED
mozilla24
Tracking Status
firefox22 --- unaffected
firefox23 + verified
firefox24 + verified

People

(Reporter: nrc, Assigned: nrc)

References

Details

(Keywords: regression)

Attachments

(2 files)

Bug 862236 wallpapered over an issue where mask layers were causing an assertion. I could not reproduce at the time. If we ever notice dodgy rounded corners under OMTC we could undo the patch there and try and track down the cause.
Attached file test case
I have no idea if this is due to the bug I mentioned in the above comment, but this case is definitely broken. I think this might be a more recent regression because it is part of my mask layers test suite which I run every now and then, but it does trip the added warning.
Go on then, I suppose I'll try and fix this.
Assignee: nobody → ncameron
Attached patch fixSplinter Review
Attachment #762378 - Flags: review?(matt.woodrow)
Attachment #762378 - Flags: review?(matt.woodrow) → review+
Comment on attachment 762378 [details] [diff] [review]
fix

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 867474
User impact if declined: incorrect rendering of some elements with border-radius
Testing completed (on m-c, etc.): I'll let it marinate for a couple of days on m-c
Risk to taking this patch (and alternatives if risky): low - tiny amount of code that was present before 867474 and is present on main thread layers backends.
String or IDL/UUID changes made by this patch: none
Attachment #762378 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/630b073105d0
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
Comment on attachment 762378 [details] [diff] [review]
fix

Low risk Fx23 regression.Approving for uplift.
Attachment #762378 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Mozilla/5.0 (X11; Linux i686; rv:23.0) Gecko/20100101 Firefox/23.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:23.0) Gecko/20100101 Firefox/23.0
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20100101 Firefox/23.0

I don`t know what exactly would be the correct behavior but I loaded the test case on 23 RC (buildID: 20130730113002) and latest Nightly (buildID: 20130731030203) and after moving the mouse pointer on the right side of the browser the rectangle moves slowly on the right but does not change form, it does change color. 
In Chrome, Opera, Safari the rectangle moves to the right changes the color and the rounded corners change into straight corners.

What is the correct behavior here?
Flags: needinfo?
(In reply to Bogdan Maris [QA] [:bogdan_maris] from comment #9)
> Mozilla/5.0 (X11; Linux i686; rv:23.0) Gecko/20100101 Firefox/23.0
> Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:23.0) Gecko/20100101
> Firefox/23.0
> Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20100101 Firefox/23.0
> 
> I don`t know what exactly would be the correct behavior but I loaded the
> test case on 23 RC (buildID: 20130730113002) and latest Nightly (buildID:
> 20130731030203) and after moving the mouse pointer on the right side of the
> browser the rectangle moves slowly on the right but does not change form, it
> does change color. 
> In Chrome, Opera, Safari the rectangle moves to the right changes the color
> and the rounded corners change into straight corners.
> 
> What is the correct behavior here?

Move the cursor over the rectangle, and keep it there as it moves. It should move to the right and turn a paler colour.
Yes it does that so if that is the correct behavior then this is verified as fixed.
Flags: needinfo?
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Firefox/24.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:24.0) Gecko/20100101 Firefox/24.0
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Firefox/24.0

Verified as fixed on Firefox 24 beta 4 (buildID: 20130819170952) and latest Nightly (buildID: 20130820030206).
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.