"ASSERTION: parentSize should be valid" with huge padding

RESOLVED FIXED in mozilla16

Status

()

Core
Layout
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Jesse Ruderman, Assigned: Ehsan)

Tracking

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

Trunk
mozilla16
x86_64
Mac OS X
assertion, regression, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

5 years ago
Created attachment 631688 [details]
testcase

###!!! ASSERTION: parentSize should be valid: 'parentSize.width != NS_INTRINSICSIZE && parentSize.height != NS_INTRINSICSIZE', file layout/base/nsCSSFrameConstructor.cpp, line 11994

This assertion was added in bug 157681.
(Reporter)

Comment 1

5 years ago
Created attachment 631689 [details]
stack trace
So the right padding here is a large value, which causes tons of these types of warnings: "have unconstrained width; this should only result from very large sizes, not attempts at intrinsic width calculation: 'aContainingBlockWidth != NS_UNCONSTRAINEDSIZE'", and also causes the width value to be unconstrained size, which triggers this assertion.

How are we supposed to handle cases like this with huge values in layout?
We don't really have a plan for these.  :(
So there isn't too much we can do about this assertion, except for taking it out?
(Reporter)

Comment 5

5 years ago
I have an idea in bug 765861.  For now, this should probably be downgraded to a warning.
Created attachment 634258 [details] [diff] [review]
Patch (v1)

This patch converts the assertion into a warning.
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #634258 - Flags: review?(dbaron)
Comment on attachment 634258 [details] [diff] [review]
Patch (v1)

Please use NS_WARN_IF_FALSE instead of the #ifdef and NS_WARNING.  r=dbaron with that
Attachment #634258 - Flags: review?(dbaron) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/a7f575fe03e0
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/a7f575fe03e0
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.