Closed Bug 64588 Opened 24 years ago Closed 23 years ago

javascript strict warnings in CookieViewer.js

Categories

(Core :: Networking: Cookies, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla0.9

People

(Reporter: bugzilla, Assigned: morse)

Details

Attachments

(2 files)

JavaScript strict warning:
chrome://communicator/content/wallet/CookieViewer.js line 200: assignment to 
undeclared variable x

JavaScript strict warning:
chrome://communicator/content/wallet/CookieViewer.js line 210: assignment to 
undeclared variable rows

JavaScript strict warning:
chrome://communicator/content/wallet/CookieViewer.js line 212: assignment to 
undeclared variable i
Keywords: patch, review
r=morse
Fix checked in.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified with build 20010126 on win2k
Status: RESOLVED → VERIFIED
UPS:

JavaScript strict warning:
chrome://communicator/content/wallet/CookieViewer.js line 495: assignment to 
undeclared variable selitems
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Good catch gemal.  Please review the patch that I just attached and, if you 
agree with it, add r=gemal to this bug report  Thanks.
Status: REOPENED → ASSIGNED
r=gemal
This was checked in a long time ago.  Forgot to close the report.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago23 years ago
Resolution: --- → FIXED
Todays build:

JavaScript strict warning:
chrome://communicator/content/wallet/CookieViewer.js line 235: assignment to 
undeclared variable rows

JavaScript strict warning:
chrome://communicator/content/wallet/CookieViewer.js line 236: assignment to 
undeclared variable k
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → ASSIGNED
Target Milestone: --- → mozilla0.9
Mass moving most of mozilla0.9 bugs to mozilla0.8.1
Target Milestone: mozilla0.9 → mozilla0.8.1
Target Milestone: mozilla0.8.1 → mozilla0.9
This is currently a major reorg going on in bug 46783 for all the cookies files.  
Have put the fix for this into that patch.  So when I checkin that patch I will 
close out this bug report.
Fixed as part of the checkin for bug 46783 that just occured.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago23 years ago
Resolution: --- → FIXED
verified
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: