Closed
Bug 169185
Opened 22 years ago
Closed 22 years ago
JavaScript code bug in pref-crlupdate.js
Categories
(Core Graveyard :: Security: UI, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
psm2.4
People
(Reporter: KaiE, Assigned: KaiE)
Details
Attachments
(1 file)
|
676 bytes,
patch
|
rangansen
:
review+
jag+mozilla
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
File pref-crlupdate.js contains a call to updateSelectedURLControls(), but no such function exists. Should the correct call possibly be updateSelectedTimingControls() ?
|
||
Comment 1•22 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
|
Assignee | |
Comment 3•22 years ago
|
||
Rangan, can you please review?
|
|
||
Comment 4•22 years ago
|
||
Comment on attachment 102314 [details] [diff] [review] Patch v1 looks fine r=rangansen@netscape.com
Attachment #102314 -
Flags: review+
|
||
Comment 5•22 years ago
|
||
Comment on attachment 102314 [details] [diff] [review] Patch v1 sr=jag
Attachment #102314 -
Flags: superreview+
|
||
Comment 6•22 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•22 years ago
|
||
fixed on trunk
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
|
||
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•