Closed
Bug 291096
Opened 20 years ago
Closed 20 years ago
Assertion not needed at widget/src/mac/nsWindow.cpp#2476
Categories
(Core Graveyard :: Widget: Mac, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bfowler, Assigned: jaas)
Details
Attachments
(1 file)
(deleted),
patch
|
mikepinkerton
:
review+
sfraser_bugs
:
superreview+
benjamin
:
approval1.8b4+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b2) Gecko/20050418 Firefox/1.0+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b2) Gecko/20050418 Firefox/1.0+
http://lxr.mozilla.org/seamonkey/source/widget/src/mac/nsWindow.cpp#2476 has
an assertion followed by normal error handling. This assertion can be replaced
by a warning, and this is desirable per Bug 279923 "Assertions should be fatal".
See Bug 268157 comment 9 , and attachment (id=181121) . (This patch is NOT needed
for work on that bug).
Reproducible: Always
I suspect that this needs to be reviewed by brianryner or pinkerton per
http://www.mozilla.org/owners.html#gfx-and-widget-mac
Updated•20 years ago
|
Assignee: firefox → joshmoz
Component: General → Widget: Mac
Product: Firefox → Core
QA Contact: general → mac
I assume you're talking about the assertions in:
NS_IMETHODIMP nsWindow::ResetInputState()
When the file changes, lxr links don't work any more. Don't use those in bug
descriptions please.
Reporter | ||
Comment 2•20 years ago
|
||
Yes.
If you want the assertion there, then just close this report as INVALID (by
design).
Sorry about the mis-use of LXR.
Just turn the assertions into warnings. I'm not comfortable removing them
myself. Pinkerton - should we remove them totally?
Attachment #190039 -
Flags: review?(pinkerton)
Comment 4•20 years ago
|
||
Comment on attachment 190039 [details] [diff] [review]
turn assertions into warnings
+ NS_WARN_IF_FALSE(kb, "cannot get parent");
+ if (kb) {
fix the tab here, r=pink.
Attachment #190039 -
Flags: review?(pinkerton) → review+
Attachment #190039 -
Flags: superreview?(sfraser_bugs)
Updated•20 years ago
|
Attachment #190039 -
Flags: superreview?(sfraser_bugs) → superreview+
Attachment #190039 -
Flags: approval1.8b4?
Updated•20 years ago
|
Attachment #190039 -
Flags: approval1.8b4? → approval1.8b4+
landed
Status: UNCONFIRMED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•