[FIX]###!!! ASSERTION: Negative width passed to nsWindow::Resize: '(aWidth >=0 )', file c:/suiterunner/mozilla/widget/src/windows/nsWindow.cpp, line 2155

RESOLVED FIXED in mozilla1.9beta2

Status

()

Core
Layout: HTML Frames
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: neil@parkwaycc.co.uk, Assigned: bz)

Tracking

Trunk
mozilla1.9beta2
x86
Windows 2000
Points:
---
Dependency tree / graph
Bug Flags:
blocking1.9 -
in-testsuite ?

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
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
Created attachment 289779 [details] [diff] [review]
Simple fix to not end up with negative widget sizes
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
Last Resolved: 10 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.