Closed
Bug 768079
Opened 12 years ago
Closed 12 years ago
"ASSERTION: How did we end up with a 3D transform here?!"
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: jruderman, Assigned: nrc)
References
Details
(Keywords: assertion, testcase)
Attachments
(4 files, 1 obsolete file)
(deleted),
text/html
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
patch
|
roc
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
roc
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
###!!! ASSERTION: How did we end up with a 3D transform here?!: 'isMask2D', file gfx/layers/opengl/LayerManagerOGLProgram.cpp, line 349
This instance of this assertion was added in bug 716439.
Reporter | ||
Comment 1•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → ncameron
Assignee | ||
Comment 2•12 years ago
|
||
Having a transform and border radius on the body tag leads to a Thebes layer with a 3D transform and a mask layer. That is not an expected combination, so causes the assertion (indirectly because Thebes layers calculate an effective transformation differently from container layers, which is where 3D transforms are usually dealt with). I'll try to find a fix.
Assignee | ||
Comment 3•12 years ago
|
||
Attachment #636583 -
Flags: review?(roc)
Attachment #636583 -
Flags: review?(roc) → review+
Assignee | ||
Comment 4•12 years ago
|
||
Attachment #636585 -
Flags: review?(roc)
Assignee | ||
Comment 5•12 years ago
|
||
Attachment #636585 -
Attachment is obsolete: true
Attachment #636585 -
Flags: review?(roc)
Attachment #636586 -
Flags: review?(roc)
Attachment #636586 -
Flags: review?(roc) → review+
Assignee | ||
Comment 6•12 years ago
|
||
Assignee | ||
Comment 7•12 years ago
|
||
Comment on attachment 636583 [details] [diff] [review]
fix
[Approval Request Comment]
Bug caused by (feature/regressing bug #): 716439
User impact if declined: possible incorrect rendering
Testing completed (on m-c, etc.): m-c
Risk to taking this patch (and alternatives if risky): no obvious risks
String or UUID changes made by this patch: none
Attachment #636583 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 8•12 years ago
|
||
Comment on attachment 636586 [details] [diff] [review]
test
[Approval Request Comment]
test, see approval request comment above
Attachment #636586 -
Flags: approval-mozilla-aurora?
Comment 9•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1766d4a0c876
https://hg.mozilla.org/mozilla-central/rev/1209bfaff4b9
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
Updated•12 years ago
|
Attachment #636583 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•12 years ago
|
Attachment #636586 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 10•12 years ago
|
||
Status: RESOLVED → UNCONFIRMED
Ever confirmed: false
Resolution: FIXED → ---
Target Milestone: mozilla16 → ---
Assignee | ||
Updated•12 years ago
|
Status: UNCONFIRMED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
Comment 11•12 years ago
|
||
Did it land in Aurora?
Assignee | ||
Comment 12•12 years ago
|
||
Yes, comment 10, I'm pretty sure it stuck.
You need to log in
before you can comment on or make changes to this bug.
Description
•