Closed Bug 137103 Opened 22 years ago Closed 22 years ago

browser.block.target_new_window listed twice in all.js

Categories

(Core :: Security: CAPS, defect, P3)

defect

Tracking

()

VERIFIED FIXED
mozilla1.0

People

(Reporter: jonasj, Assigned: caillon)

References

()

Details

Attachments

(1 file)

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.
Attached patch patchSplinter Review
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 on attachment 78893 [details] [diff] [review]
patch

sr=tor
Attachment #78893 - Flags: superreview+
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
Closed: 22 years ago
Resolution: --- → FIXED
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.

Attachment

General

Created:
Updated:
Size: