Closed
Bug 389101
Opened 17 years ago
Closed 17 years ago
Don't claim to handle the gopher protocol
Categories
(Camino Graveyard :: OS Integration, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: alqahira, Assigned: alqahira)
Details
(Keywords: fixed1.8.1.8)
Attachments
(2 files, 1 obsolete file)
1.99 KB,
patch
|
jaas
:
review+
stuart.morgan+bugzilla
:
superreview+
|
Details | Diff | Splinter Review |
1.91 KB,
patch
|
jaas
:
review+
stuart.morgan+bugzilla
:
superreview+
|
Details | Diff | Splinter Review |
In bug 388192, Firefox removed their LS entries claiming to handle the gopher protocol for security reasons. I'm not clear on all the implications, but gopher's pretty dead anyway, so not telling the OS that we can handle gopher doesn't seem like a big loss. This should apply on the branch, too. If the issue is really serious, we should consider making 1.5-branch patch, too.
Attachment #273251 -
Flags: review?(joshmoz)
Assignee | ||
Comment 1•17 years ago
|
||
Comment on attachment 273251 [details] [diff] [review] gopherectomy for Info-Camino.plist Er, this isn't "complete" (in the sense the Fx patch was), because we *do* set ourselves as the default gopher client when we set ourselves as the default browser.
Attachment #273251 -
Attachment is obsolete: true
Attachment #273251 -
Flags: review?(joshmoz)
Assignee | ||
Comment 2•17 years ago
|
||
The LaunchServices functions are different on branch and trunk, so we need 2 patches. Needless to say, these haven't been tested since I can't build anymore.
Assignee | ||
Comment 3•17 years ago
|
||
Attachment #273263 -
Flags: review?(joshmoz)
Attachment #273261 -
Flags: review?(joshmoz) → review+
Attachment #273263 -
Flags: review?(joshmoz) → review+
Assignee | ||
Updated•17 years ago
|
Attachment #273261 -
Flags: superreview?(stuart.morgan)
Assignee | ||
Updated•17 years ago
|
Attachment #273263 -
Flags: superreview?(stuart.morgan)
Comment 4•17 years ago
|
||
Landed on trunk and MOZILLA_1_8_BRANCH.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•17 years ago
|
Attachment #273261 -
Flags: superreview?(stuart.morgan) → superreview+
Updated•17 years ago
|
Attachment #273263 -
Flags: superreview?(stuart.morgan) → superreview+
Assignee | ||
Updated•17 years ago
|
Keywords: fixed1.8.1.7
You need to log in
before you can comment on or make changes to this bug.
Description
•