Last Comment Bug 768079 - "ASSERTION: How did we end up with a 3D transform here?!"
: "ASSERTION: How did we end up with a 3D transform here?!"
Status: RESOLVED FIXED
: assertion, testcase
Product: Core
Classification: Components
Component: Graphics (show other bugs)
: Trunk
: x86_64 Mac OS X
: -- normal (vote)
: mozilla16
Assigned To: Nick Cameron [:nrc]
:
Mentors:
Depends on:
Blocks: randomstyles
  Show dependency treegraph
 
Reported: 2012-06-25 10:27 PDT by Jesse Ruderman
Modified: 2012-07-09 15:18 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
testcase (233 bytes, text/html)
2012-06-25 10:27 PDT, Jesse Ruderman
no flags Details
stack trace (2.41 KB, text/plain)
2012-06-25 10:27 PDT, Jesse Ruderman
no flags Details
fix (1.21 KB, patch)
2012-06-25 20:24 PDT, Nick Cameron [:nrc]
roc: review+
lukasblakk+bugs: approval‑mozilla‑aurora+
Details | Diff | Splinter Review
test (1.11 KB, patch)
2012-06-25 20:43 PDT, Nick Cameron [:nrc]
no flags Details | Diff | Splinter Review
test (949 bytes, patch)
2012-06-25 20:52 PDT, Nick Cameron [:nrc]
roc: review+
lukasblakk+bugs: approval‑mozilla‑aurora+
Details | Diff | Splinter Review

Description Jesse Ruderman 2012-06-25 10:27:19 PDT
Created attachment 636376 [details]
testcase

###!!! 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.
Comment 1 Jesse Ruderman 2012-06-25 10:27:36 PDT
Created attachment 636378 [details]
stack trace
Comment 2 Nick Cameron [:nrc] 2012-06-25 16:50:50 PDT
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.
Comment 3 Nick Cameron [:nrc] 2012-06-25 20:24:02 PDT
Created attachment 636583 [details] [diff] [review]
fix
Comment 4 Nick Cameron [:nrc] 2012-06-25 20:43:43 PDT
Created attachment 636585 [details] [diff] [review]
test
Comment 5 Nick Cameron [:nrc] 2012-06-25 20:52:03 PDT
Created attachment 636586 [details] [diff] [review]
test
Comment 7 Nick Cameron [:nrc] 2012-06-26 20:38:02 PDT
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
Comment 8 Nick Cameron [:nrc] 2012-06-26 20:42:25 PDT
Comment on attachment 636586 [details] [diff] [review]
test

[Approval Request Comment]
test, see approval request comment above
Comment 11 Scoobidiver (away) 2012-07-09 14:42:04 PDT
Did it land in Aurora?
Comment 12 Nick Cameron [:nrc] 2012-07-09 15:18:25 PDT
Yes, comment 10, I'm pretty sure it stuck.

Note You need to log in before you can comment on or make changes to this bug.