Closed
Bug 397871
Opened 18 years ago
Closed 18 years ago
[FIX]###!!! ASSERTION: Negative width passed to nsWindow::Resize: '(aWidth >=0 )', file c:/suiterunner/mozilla/widget/src/windows/nsWindow.cpp, line 2155
Categories
(Core :: Layout: Images, Video, and HTML Frames, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9beta2
People
(Reporter: neil, Assigned: bzbarsky)
References
Details
Attachments
(1 file)
|
1.40 KB,
patch
|
roc
:
review+
roc
:
superreview+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
Steps to reproduce problem:
1. seamonkey -mail
Only (sigh) on my non-debug build I can also trigger it by toggling the message pane (e.g. via F8).
Seems to be caused by the border on a collapsed <browser> element.
| Assignee | ||
Comment 1•18 years ago
|
||
Hmm. So I think we do want to PR_MAX this with 0 to handle border-box sizing with too-small width and the like. But in this particular case, shouldn't the XUL code that does wacky things with |visibility: collapse| set the used border and padding to 0?
Flags: blocking1.9?
Flags: blocking1.9? → blocking1.9-
| Assignee | ||
Comment 2•18 years ago
|
||
I don't see a sane way to do make the used border correct here; I'll just file a followup bug on that and hack in the PR_MAX here once bug 399410 lands.
Depends on: 399410
| Assignee | ||
Comment 3•18 years ago
|
||
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #289779 -
Flags: superreview?(roc)
Attachment #289779 -
Flags: review?(roc)
| Assignee | ||
Updated•18 years ago
|
Summary: ###!!! ASSERTION: Negative width passed to nsWindow::Resize: '(aWidth >=0 )', file c:/suiterunner/mozilla/widget/src/windows/nsWindow.cpp, line 2155 → [FIX]###!!! ASSERTION: Negative width passed to nsWindow::Resize: '(aWidth >=0 )', file c:/suiterunner/mozilla/widget/src/windows/nsWindow.cpp, line 2155
Target Milestone: --- → mozilla1.9 M10
Attachment #289779 -
Flags: superreview?(roc)
Attachment #289779 -
Flags: superreview+
Attachment #289779 -
Flags: review?(roc)
Attachment #289779 -
Flags: review+
| Assignee | ||
Comment 4•18 years ago
|
||
Comment on attachment 289779 [details] [diff] [review]
Simple fix to not end up with negative widget sizes
Simple sanity fix.
Attachment #289779 -
Flags: approval1.9?
Comment 5•18 years ago
|
||
Comment on attachment 289779 [details] [diff] [review]
Simple fix to not end up with negative widget sizes
a=beltzner on behalf of drivers
Attachment #289779 -
Flags: approval1.9? → approval1.9+
| Assignee | ||
Comment 6•18 years ago
|
||
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Updated•7 years ago
|
Product: Core → Core Graveyard
Updated•7 years ago
|
Component: Layout: HTML Frames → Layout: Images
Product: Core Graveyard → Core
You need to log in
before you can comment on or make changes to this bug.
Description
•