238080 - In <CookieViewer.js>, "Warning: redeclaration of var p"

Status: VERIFIED FIXED

(Core :: Networking: Cookies, defect)

Description

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7b) Gecko/20040316] (W98SE)

Spun off from bug 233910 comment 7. (already in v1.7a)

{{
Warning: redeclaration of var p
Source File: chrome://communicator/content/wallet/CookieViewer.js
Line: 619, Column: 13
Source Code:
    for (var p=0; p<deletedPermissions.length; p++)
}}

Updating:
*(A.To) darin#meer.net -> me.

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) <CookieViewer.js> [Checked in: Comment 5]

Comment 2

[jag (Peter Annema)]

Comment on attachment 144399 [details] [diff] [review]
(Av1) <CookieViewer.js>
[Checked in: Comment 5]

sr=jag

Comment 3

[Serge Gautherie (:sgautherie)]

Comment on attachment 144399 [details] [diff] [review]
(Av1) <CookieViewer.js>
[Checked in: Comment 5]


'approval1.7=?':
Trivial code fix, no risk;
(Trying v1.7 instead of waiting for v1.8a, as v1.7 will become new
long-lived...)

Comment 4

[Mike Kaply [:mkaply]]

Comment on attachment 144399 [details] [diff] [review]
(Av1) <CookieViewer.js>
[Checked in: Comment 5]

a=mkaply

Comment 5

[[e-mail]]

Fix checked in.

Comment 6

[benc]

V:fixed
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=CookieViewer.js&branch=&root=/cvsroot&subdir=mozilla/extensions/wallet/cookieviewer/resources/content&command=DIFF_FRAMESET&rev1=1.16&rev2=1.17