Closed Bug 623451 Opened 10 years ago Closed 10 years ago
"Assertion failed at gfx/cairo/cairo/src/cairo-surface
.c:588: CAIRO _REFERENCE _COUNT _HAS _REFERENCE (&surface->ref _count)" for shadow canvas and video OGL layers with
The problem is that we swap front/back on updates, but with GL we only use one surface and end up double-releasing. Pretty simple fix.
tracking-fennec: --- → ?
Not sure what the proper COM discipline is for this situation, but this patch doesn't make me vomit in my mouth /too/ much.
Assignee: nobody → jones.chris.g
Attachment #501574 - Flags: review?
10 years ago
Attachment #501574 - Flags: review? → review?(vladimir)
Oops, obviously ignore that hunk in cairo/src.
Comment on attachment 501574 [details] [diff] [review] Don't double-release surfaces you want jeff here.. and you also want someone to do the cairo patch dance (as well as getting rid of the extra whitespace :-)
Attachment #501574 - Flags: review?(vladimir) → review?(jmuizelaar)
That cairo hunk was just for my debugging purposes, I didn't mean to include it. Let me post a version with it removed.
Nix the debugging code.
10 years ago
Attachment #501578 - Flags: review?(jmuizelaar) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.