Closed Bug 884132 Opened 12 years ago Closed 12 years ago

Make use of .isTopLevel now fix to bug 666801 has landed

Categories

(SeaMonkey :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.22

People

(Reporter: iannbugzilla, Assigned: ewong)

References

Details

Attachments

(1 file, 1 obsolete file)

Now that the fix to bug 666801 has landed, we should be able to switch our code to use .isTopLevel where appropriate - for example http://mxr.mozilla.org/comm-central/source/suite/common/src/nsSuiteGlue.js#197
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attached patch Make use of .isTopLevel (v1) (obsolete) — Splinter Review
Attachment #775139 - Flags: review?(neil)
I think tabbrowser.xml also has a few places which should be fixed as well.
Attachment #775139 - Attachment is obsolete: true
Attachment #775139 - Flags: review?(neil)
Attachment #775331 - Flags: review?(neil)
Comment on attachment 775331 [details] [diff] [review] Make use of .isTopLevel (v2) Seems reasonable.
Attachment #775331 - Flags: review?(neil) → review+
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.22
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: