Closed Bug 202707 Opened 21 years ago Closed 21 years ago

cookie and image manager sort entries in descending order

Categories

(Core :: Networking: Cookies, defect)

Product:
Core
Component:
Networking: Cookies
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: svl-bmo, Assigned: dwitte)

Details

(Keywords: regression)

Attachments

(1 file)

fix
1.05 KB, patch
mvl
: review+
bzbarsky
: superreview+
Details | Diff | Splinter Review 
Using 2003041908 (win98), cookie manager and image manager show all entries in
descending order, rather than the usual ascending order.
This was caused by the fix for bug 202131; switching the -1 and 1 (both for the
if and the else) in compareFunc here:
http://lxr.mozilla.org/seamonkey/source/extensions/wallet/cookieviewer/nsWalletTreeUtils.js#130
fixes.
Attached patch fix — — Splinter Review 
indeed. neil actually pointed this out in his patch but i guess i forgot to
carry over the change :/

nice catch, thanks.
Attachment #121124 - Flags: superreview+
checked in by ere @ 1331.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: