Closed
Bug 276617
Opened 20 years ago
Closed 20 years ago
Cookie Exceptions cannot prompt for Invalid Hostname
Categories
(Firefox :: Settings UI, defect)
Firefox
Settings UI
Tracking
()
RESOLVED
FIXED
People
(Reporter: iannbugzilla, Assigned: iannbugzilla)
Details
Attachments
(1 file, 1 obsolete file)
|
1.44 KB,
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
At the moment CookieExceptions.js would error when trying to display a prompt about an Invalid Hostname being entered. stringBundle not defined promptservice not defined
stringBundle -> this._bundle promptservice -> promptService
Comment 2•20 years ago
|
||
Comment on attachment 169999 [details] [diff] [review] Renames things as they should be v0.1 hmm, I'd look for when this broke, but I don't know if/when it ever gets hit.
Attachment #169999 -
Flags: review?(mconnor) → review+
This zaps the textbox and returns early otherwise the "invalid" site still gets added. I'm not sure it does get hit as newURI seems to generate a uri no matter what rubbish you enter, but just in case...
Attachment #169999 -
Attachment is obsolete: true
Attachment #170000 -
Flags: review?(mconnor)
Comment 4•20 years ago
|
||
Comment on attachment 170000 [details] [diff] [review] Renames things and returns early v0.2 (Checked in) that'll work too.
Attachment #170000 -
Flags: review?(mconnor) → review+
Comment on attachment 170000 [details] [diff] [review] Renames things and returns early v0.2 (Checked in) Checking in CookieExceptions.js; /cvsroot/mozilla/browser/components/cookieviewer/content/CookieExceptions.js,v <-- CookieExceptions.js new revision: 1.10; previous revision: 1.9 done
Attachment #170000 -
Attachment description: Renames things and returns early v0.2 → Renames things and returns early v0.2
(Checked in)
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment 6•18 years ago
|
||
sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs, filter on "beltznerLovesGoats" to get rid of this mass change
QA Contact: mconnor → preferences
You need to log in
before you can comment on or make changes to this bug.
Description
•