Closed Bug 601435 Opened 14 years ago Closed 14 years ago

Move null check into isElementVisible() like Friedfox.

Categories

(SeaMonkey :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b2

People

(Reporter: philip.chee, Assigned: philip.chee)

References

Details

Attachments

(1 file)

From 410613 Comment 61: > (In reply to comment #60) >> Firefox moved the null check into their isElementVisible(). We should too. > Good idea, could you file a followup for that?
Blocks: 410613
Attached patch Proposed fix v1.0 (deleted) — Splinter Review
Move null check to isElementVisible() and fix callers.
Attachment #482145 - Flags: review?(neil)
Attachment #482145 - Flags: review?(neil) → review+
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: