Closed
Bug 824300
Opened 12 years ago
Closed 7 years ago
"ASSERTION: GetVisualOverflowRect() won't return the pre-effects rect!" with {ib}, CSS filter
Categories
(Core :: SVG, defect)
Tracking
()
RESOLVED
FIXED
mozilla54
People
(Reporter: jruderman, Assigned: u459114)
References
Details
(Keywords: assertion, testcase)
Attachments
(4 files)
###!!! ASSERTION: How did we getting here, then?: 'aFrame->GetParent()->GetStyleContext()->GetPseudo() == nsCSSAnonBoxes::mozAnonymousBlock', file layout/svg/nsSVGIntegrationUtils.cpp, line 109 ###!!! ASSERTION: GetVisualOverflowRect() won't return the pre-effects rect!: '!aFrame->Properties().Get( aFrame->PreTransformOverflowAreasProperty())', file layout/svg/nsSVGIntegrationUtils.cpp, line 112
|
Reporter | |
Comment 1•12 years ago
|
||
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
Attachment #8833635 -
Flags: review?(cam)
Attachment #8833636 -
Flags: review?(cam)
|
||
Comment 5•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8833635 [details] Bug 824300 - Part 1. Correct assert condition and promote NS_ASSERTION to MOZ_ASSERT https://reviewboard.mozilla.org/r/109848/#review114024
Attachment #8833635 -
Flags: review?(cam) → review+
|
|
||
Comment 6•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8833636 [details] Bug 824300 - Part 2. Crash test. https://reviewboard.mozilla.org/r/109850/#review114028 ::: layout/base/crashtests/crashtests.list:486 (Diff revision 2) > load 1299736-1.html > load 1308793.svg > load 1308848-1.html > load 1308848-2.html > load 842114.html > +load 824300.html Put this in sorted order too.
Attachment #8833636 -
Flags: review?(cam) → review+
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
Pushed by cku@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7ff2deda019f Part 1. Correct assert condition and promote NS_ASSERTION to MOZ_ASSERT r=heycam https://hg.mozilla.org/integration/autoland/rev/fb2b5dae66c7 Part 2. Crash test. r=heycam
|
||
Comment 10•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/7ff2deda019f https://hg.mozilla.org/mozilla-central/rev/fb2b5dae66c7
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
|
||
Comment 11•7 years ago
|
||
Is this worth backporting to 53 or should we let it ride the 54 train?
status-firefox51:
--- → wontfix
status-firefox52:
--- → wontfix
status-firefox53:
--- → affected
status-firefox-esr52:
--- → wontfix
Flags: needinfo?(cku)
Flags: in-testsuite+
|
Assignee | |
Comment 12•7 years ago
|
||
Probably not, the reason is the same with bug 1340257 comment 11
Flags: needinfo?(cku)
|
|
||
Updated•7 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•