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)

x86
Windows 2000
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9beta2

People

(Reporter: neil, Assigned: bzbarsky)

References

Details

Attachments

(1 file)

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.
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-
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: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #289779 - Flags: superreview?(roc)
Attachment #289779 - Flags: review?(roc)
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+
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 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+
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Product: Core → Core Graveyard
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.

Attachment

General

Created:
Updated:
Size: