Closed Bug 331674 Opened 19 years ago Closed 19 years ago

Replace QI call in nsBrowserStatusHandler with instanceof

Categories

(SeaMonkey :: UI Design, defect)

x86
Windows XP
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: WeirdAl, Assigned: WeirdAl)

Details

(Keywords: fixed-seamonkey1.1a)

Attachments

(1 file)

This is a little tweak; it gets rid of a try...catch block around a line of code that triggers an exception. I discovered it while Venkman was running and I dragged & dropped a file into the browser.
Attached patch patchSplinter Review
This patch is only ten lines, and produces no outwardly visible changes to the user.
Attachment #216195 - Flags: superreview?(jag)
Attachment #216195 - Flags: review?(jag)
Attachment #216195 - Flags: superreview?(neil)
Attachment #216195 - Flags: superreview?(jag)
Attachment #216195 - Flags: review?(jag)
Attachment #216195 - Flags: review+
No worries, that's truly me, just another account. r=jag
Comment on attachment 216195 [details] [diff] [review] patch You can check this in on the 1.8 branch too if you like.
Attachment #216195 - Flags: superreview?(neil) → superreview+
checked into trunk and 1_8 branch
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: