Closed Bug 283522 Opened 20 years ago Closed 20 years ago

SetPageProxyState no longer needs a URI parameter

Categories

(Firefox :: General, defect, P3)

Product:
Firefox
Component:
General
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: Gavin, Assigned: Gavin)

References

(
URL
)

Details

Attachments

(1 file, 1 obsolete file)

Unbitrotted
4.62 KB, patch
mconnor
: review+
asa
: approval-aviary1.1a1+
Details | Diff | Splinter Review
The security check was removed in bug 252288's attachment 154061 [details] [diff] [review], so the parameter is no longer used. This removes the need to create a URI at: http://lxr.mozilla.org/mozilla/source/browser/base/content/browser.js#2720
Attached patch Patch (obsolete) — — Splinter Review
Attachment #175488 - Flags: review?(mconnor)
Let's try that again... This removes the need to create a URI at: http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/browser/base/content/browser.js&rev=1.386#2773
Status: NEW → ASSIGNED
Whiteboard: [patch-r?]
Priority: -- → P3
Attached patch Unbitrotted — — Splinter Review
Attachment #178160 - Flags: review?(mconnor)
Attachment #175488 - Attachment is obsolete: true
Attachment #175488 - Flags: review?(mconnor)
Attachment #178160 - Flags: review?(mconnor) → review+
Whiteboard: [patch-r?] → [patch-r+] [checkin needed]
Comment on attachment 178160 [details] [diff] [review] Unbitrotted Removes unused code.
Attachment #178160 - Flags: approval-aviary1.1a?
Comment on attachment 178160 [details] [diff] [review] Unbitrotted a=asa
Attachment #178160 - Flags: approval-aviary1.1a? → approval-aviary1.1a+
Whiteboard: [patch-r+] [checkin needed] → [patch-r+][checkin needed][a+]
Checked in by timeless.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Whiteboard: [patch-r+][checkin needed][a+]
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: