The default bug view has changed. See this FAQ.

Move null check into isElementVisible() like Friedfox.

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
General
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Philip Chee, Assigned: Philip Chee)

Tracking

Trunk
seamonkey2.1b2

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
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?
(Assignee)

Updated

7 years ago
Blocks: 410613
(Assignee)

Comment 1

7 years ago
Created attachment 482145 [details] [diff] [review]
Proposed fix v1.0

Move null check to isElementVisible() and fix callers.
Attachment #482145 - Flags: review?(neil)

Updated

7 years ago
Attachment #482145 - Flags: review?(neil) → review+
(Assignee)

Comment 2

7 years ago
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/68913ca8ed66
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Updated

7 years ago
Target Milestone: --- → seamonkey2.1b2
You need to log in before you can comment on or make changes to this bug.