JavaScript code bug in pref-crlupdate.js

VERIFIED FIXED in psm2.4

Status

P2
normal
VERIFIED FIXED
16 years ago
2 years ago

People

(Reporter: kaie, Assigned: kaie)

Tracking

Other Branch
psm2.4
x86
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
File pref-crlupdate.js contains a call to updateSelectedURLControls(), but no
such function exists.

Should the correct call possibly be updateSelectedTimingControls() ?

Comment 1

16 years ago
Actually, this line should have been removed - originally, the update url was 
editable - that was changed later, but this line wasn't removed.

updateSelectedTimingControls() would automatically be invoked in Initialize - 
so, we should remove this line

Comment 2

16 years ago
Kai.
Assignee: ssaux → kaie
Priority: -- → P2
Target Milestone: --- → 2.4
(Assignee)

Comment 3

16 years ago
Created attachment 102314 [details] [diff] [review]
Patch v1


Rangan, can you please review?

Comment 4

16 years ago
Comment on attachment 102314 [details] [diff] [review]
Patch v1

looks fine
r=rangansen@netscape.com
Attachment #102314 - Flags: review+

Comment 5

16 years ago
Comment on attachment 102314 [details] [diff] [review]
Patch v1

sr=jag
Attachment #102314 - Flags: superreview+

Comment 6

16 years ago
Comment on attachment 102314 [details] [diff] [review]
Patch v1

a=asa for checkin to 1.2beta (on behalf of drivers)
Attachment #102314 - Flags: approval+
(Assignee)

Comment 7

16 years ago
fixed on trunk
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 8

16 years ago
Verified per kaie's comment.
Status: RESOLVED → VERIFIED

Updated

14 years ago
Component: Security: UI → Security: UI
Product: PSM → Core
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.