In <metadata.js>, "Warning: reference to undefined property tokens[0]"

RESOLVED FIXED in mozilla1.8alpha1

Status

SeaMonkey
General
--
minor
RESOLVED FIXED
14 years ago
13 years ago

People

(Reporter: sgautherie, Assigned: neil@parkwaycc.co.uk)

Tracking

Trunk
mozilla1.8alpha1

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

14 years ago
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7b) Gecko/20040421] (<-- 1.7rc1 !)
(W98SE)

1. Load URL
2. Right Click on "Téléchargement sur-le-champ" button
3. > Properties

The dialog opens "fine", but generates a warning:
{{
Warning: reference to undefined property tokens[0]
Source File: chrome://navigator/content/metadata.js
Line: 568
}}
(Reporter)

Comment 1

14 years ago
Created attachment 148314 [details] [diff] [review]
(Av1_M) <metadata.js>
Assignee: general → gautheri
Status: NEW → ASSIGNED
(Reporter)

Comment 2

14 years ago
Comment on attachment 148314 [details] [diff] [review]
(Av1_M) <metadata.js>


This fixes the warning,
but I'm not sure if it's the proper fix or not...
Attachment #148314 - Flags: review?(neil.parkwaycc.co.uk)
(Assignee)

Comment 3

14 years ago
Created attachment 148337 [details] [diff] [review]
More cleanup
(Assignee)

Updated

14 years ago
Assignee: gautheri → neil.parkwaycc.co.uk
Attachment #148314 - Attachment is obsolete: true
(Assignee)

Updated

14 years ago
Attachment #148337 - Flags: review?(db48x)
Comment on attachment 148337 [details] [diff] [review]
More cleanup

What about Firefox? Their copy looks to have the same problem.
(Assignee)

Comment 5

14 years ago
My understanding is that a separate bug has to be filed for the FireFox fix.
Comment on attachment 148337 [details] [diff] [review]
More cleanup

Ok. r=db48x
Attachment #148337 - Flags: review?(db48x) → review+
(Assignee)

Updated

14 years ago
Attachment #148337 - Flags: superreview?(alecf)

Comment 7

14 years ago
Comment on attachment 148337 [details] [diff] [review]
More cleanup

sr=alecf
Attachment #148337 - Flags: superreview?(alecf) → superreview+
(Assignee)

Comment 8

14 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
(Reporter)

Updated

14 years ago
Attachment #148314 - Flags: review?(neil.parkwaycc.co.uk)
(Reporter)

Updated

14 years ago
OS: Windows 98 → All
Hardware: PC → All
Target Milestone: --- → mozilla1.8alpha
(Reporter)

Comment 9

14 years ago
Comment on attachment 148337 [details] [diff] [review]
More cleanup

'approval1.7=?':
Trivial U.I. code cleanup, low risk.
Attachment #148337 - Flags: approval1.7?

Comment 10

14 years ago
Comment on attachment 148337 [details] [diff] [review]
More cleanup

warning cleanups can happen on the trunk. there's not much user value, even if
the risk is low -- and warning cleanups have broken features in the past.
Attachment #148337 - Flags: approval1.7? → approval1.7-
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.