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)
SeaMonkey
General
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.22
People
(Reporter: iannbugzilla, Assigned: ewong)
References
Details
Attachments
(1 file, 1 obsolete file)
|
8.89 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
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
Comment 1•12 years ago
|
||
> for example http://mxr.mozilla.org/comm-central/source/suite/common/src/nsSuiteGlue.js#197
Permalink: http://hg.mozilla.org/comm-central/annotate/723ed450d2e7/suite/common/src/nsSuiteGlue.js#l197
OS: Linux → All
Hardware: x86_64 → All
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → ewong
Status: NEW → ASSIGNED
| Assignee | ||
Comment 2•12 years ago
|
||
Attachment #775139 -
Flags: review?(neil)
Comment 3•12 years ago
|
||
I think tabbrowser.xml also has a few places which should be fixed as well.
| Assignee | ||
Comment 4•12 years ago
|
||
Attachment #775139 -
Attachment is obsolete: true
Attachment #775139 -
Flags: review?(neil)
Attachment #775331 -
Flags: review?(neil)
Comment 5•12 years ago
|
||
Comment on attachment 775331 [details] [diff] [review]
Make use of .isTopLevel (v2)
Seems reasonable.
Attachment #775331 -
Flags: review?(neil) → review+
| Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Comment 6•12 years ago
|
||
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.
Description
•