If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

browser.block.target_new_window listed twice in all.js

VERIFIED FIXED in mozilla1.0

Status

()

Core
Security: CAPS
P3
trivial
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: Jonas Jørgensen, Assigned: Christopher Aillon (sabbatical, not receiving bugmail))

Tracking

Trunk
mozilla1.0
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

471 bytes, patch
Christopher Aillon (sabbatical, not receiving bugmail)
: review+
Details | Diff | Splinter Review
(Reporter)

Description

16 years ago
caillon: With your checkin for bug 117707, you added
browser.block.target_new_window to the Scripts & Windows section of all.js, but
that pref was already listed in all.js, so now it is listed twice.
(Reporter)

Comment 1

16 years ago
Created attachment 78893 [details] [diff] [review]
patch

caillon, could you r=? jst, could you sr=?
Comment on attachment 78893 [details] [diff] [review]
patch

Oops.  r=caillon
Attachment #78893 - Flags: review+
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla1.0

Comment 3

16 years ago
Comment on attachment 78893 [details] [diff] [review]
patch

sr=tor
Attachment #78893 - Flags: superreview+

Comment 4

16 years ago
Comment on attachment 78893 [details] [diff] [review]
patch

a=asa (on behalf of drivers) for checkin to the 1.0 branch
Attachment #78893 - Flags: approval+
Checked in on the trunk:  04/13/2002 12:22
Checked in on the branch: 04/17/2002 07:17
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
(Reporter)

Comment 6

16 years ago
Verified: Searching for "target_new_window" in prefs.js retrieved through LXR
(both trunk and branch) returns only one hit.
Status: RESOLVED → VERIFIED
Keywords: verified1.0.0
You need to log in before you can comment on or make changes to this bug.