Closed Bug 310379 Opened 19 years ago Closed 19 years ago

Crash [@ nsMacEventHandler::Scroll ]

Categories

(Core Graveyard :: Widget: Mac, defect)

Product:
Core Graveyard
Component:
Widget: Mac
Platform:
PowerPC
macOS
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: asaf, Assigned: mark)

Details

(Keywords: crash, regression, verified1.8)

Crash Data

Attachments

(1 file)

Fix
1.54 KB, patch
asaf
: review+
sfraser_bugs
: superreview+
asa
: approval1.8b5+
Details | Diff | Splinter Review 
See TB9842583Y
str:
1. Scrollwheel the window titlebar
2. move the mouse into the window.
Assignee: joshmoz → mark
Status: NEW → ASSIGNED
Flags: blocking1.8b5?
Keywords: crash, regression
Attached patch Fix — — Splinter Review 
Ooh, look at that pretty stack!
Attachment #197783 - Flags: superreview?(sfraser_bugs)
Attachment #197783 - Flags: review?(bugs.mano)
Comment on attachment 197783 [details] [diff] [review]
Fix


>+  if (nsnull == widgetToScroll)

If you don't insist, make that |if (!widgetToScroll)|.

r=mano either way.
Attachment #197783 - Flags: review?(bugs.mano) → review+
Severity: normal → critical
Comment on attachment 197783 [details] [diff] [review]
Fix

I'd prefer
if (!widgetToScroll)
too.
Attachment #197783 - Flags: superreview?(sfraser_bugs) → superreview+
Attachment #197783 - Flags: approval1.8b5?
Checked in on trunk.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Attachment #197783 - Flags: approval1.8b5? → approval1.8b5+
and branch.
Keywords: fixed1.8
Flags: blocking1.8b5?
Looks good using Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US;
rv:1.8b5) Gecko/20051018 Firefox/1.4.1. Add verified keyword.
Keywords: fixed1.8 → verified1.8
Product: Core → Core Graveyard
Summary: Crash [ @ nsMacEventHandler::Scroll ] → Crash [@ nsMacEventHandler::Scroll ]
Crash Signature: [@ nsMacEventHandler::Scroll ]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: