"Assertion failure: hasPerspective (Why did we create nsDisplayPerspective?)"

RESOLVED FIXED in Firefox 46

Status

()

--
critical
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jruderman, Assigned: mattwoodrow)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
mozilla46
assertion, testcase
Points:
---

Firefox Tracking Flags

(firefox45 affected, firefox46 fixed)

Details

Attachments

(3 attachments)

(Reporter)

Description

3 years ago
Created attachment 8696124 [details]
testcase

Assertion failure: hasPerspective (Why did we create nsDisplayPerspective?), at layout/base/nsDisplayList.cpp:6093
(Reporter)

Updated

3 years ago
Severity: normal → critical
(Reporter)

Comment 1

3 years ago
Created attachment 8696125 [details]
stack
Probably one of Thinker or Matt is the right person to look into this.
Flags: needinfo?(tlee)
Flags: needinfo?(matt.woodrow)

Comment 3

3 years ago
I think Matt know better than me.  If he is busy, I may look into it.
Flags: needinfo?(tlee)
(Assignee)

Comment 4

3 years ago
Created attachment 8697006 [details] [diff] [review]
perspective-assertion

This is such a small perspective value that it rounds to 0 in app units.

We should still treat this as true for having perspective (since the rest of our code does), and just use an identity matrix.
Assignee: nobody → matt.woodrow
Flags: needinfo?(matt.woodrow)
Attachment #8697006 - Flags: review?(roc)

Comment 6

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/646484054aea
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.