Open
Bug 611532
Opened 14 years ago
Updated 2 years ago
"ASSERTION: We should have padding here! (out of memory?)"
Categories
(Core :: XUL, defect)
Core
XUL
Tracking
()
NEW
Tracking | Status | |
---|---|---|
blocking2.0 | --- | - |
People
(Reporter: jruderman, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: assertion, testcase)
Attachments
(2 files)
684 bytes,
application/xhtml+xml
|
Details | |
1.30 KB,
patch
|
Details | Diff | Splinter Review |
###!!! ASSERTION: We should have padding here! (out of memory?): 'hasPadding', file layout/generic/nsFrame.cpp, line 693
Reporter | ||
Comment 1•14 years ago
|
||
I don't know why bug 550325 was a blocker, but this might need to be a blocker for the same reason.
blocking2.0: --- → ?
I don't know either.
blocking2.0: ? → -
Comment 3•14 years ago
|
||
I guess bug 550325 was a blocker because it's pure HTML+CSS and there was an actual layout error involved, so it could potentially have broken some site. This bug (which is the padding variant of bug 586628) involves XUL boxes in HTML and it's just a harmless assertion - no layout regression. I think we should just mute these assertions for XUL boxes until we actually implement percentage padding/margin for them.
Comment 4•14 years ago
|
||
Assignee: nobody → matspal
Updated•14 years ago
|
Whiteboard: [post-2.0]
Comment 5•13 years ago
|
||
Noting here that attachment 508920 [details] [diff] [review] should be backed out when this bug is fixed.
Comment 6•10 years ago
|
||
Only happens for XUL frames.
Assignee: mats → nobody
Component: Layout → XUL
Whiteboard: [post-2.0]
Comment 7•9 years ago
|
||
Still affects Windows at least (other platforms don't seem to reproduce easily on Try anyway).
Updated•2 years ago
|
Severity: minor → S4
You need to log in
before you can comment on or make changes to this bug.
Description
•