Closed
Bug 1007303
Opened 11 years ago
Closed 11 years ago
SeaMonkey should fix common scheme typos
Categories
(SeaMonkey :: Tabbed Browser, defect)
SeaMonkey
Tabbed Browser
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.29
People
(Reporter: neil, Assigned: neil)
Details
Attachments
(2 files)
|
3.12 KB,
patch
|
iannbugzilla
:
review+
philip.chee
:
feedback+
|
Details | Diff | Splinter Review |
|
10.03 KB,
patch
|
philip.chee
:
feedback+
|
Details | Diff | Splinter Review |
Basically a port of the front-end parts of bug 935377.
|
Assignee | |
Comment 1•11 years ago
|
||
Because we funnel almost everything through loadURI or addTab we don't have so many places where we need to specify the flags. I thought it was better to update editor's version of the open location dialog to use the new parameter format (postData + boolean fixup flag instead of integer load flags).
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #8418938 -
Flags: review?(iann_bugzilla)
Attachment #8418938 -
Flags: feedback?(philip.chee)
|
|
Assignee | |
Comment 2•11 years ago
|
||
Removed in bug 945627.
Attachment #8419047 -
Flags: feedback?(philip.chee)
Comment on attachment 8418938 [details] [diff] [review]
Proposed patch
Does anything need doing with regards to nsBrowserContentHandler.js and function resolveURIInternal?
r=me with that addressed.
Attachment #8418938 -
Flags: review?(iann_bugzilla) → review+
|
||
Comment 4•11 years ago
|
||
Comment on attachment 8419047 [details] [diff] [review]
With isUTF8 removal
Works as expected. [is there a pref to turn thi on/off?]
Attachment #8419047 -
Flags: feedback?(philip.chee) → feedback+
|
|
||
Updated•11 years ago
|
Attachment #8418938 -
Flags: feedback?(philip.chee) → feedback+
|
|
Assignee | |
Comment 5•11 years ago
|
||
(In reply to Philip Chee from comment #4)
> is there a pref to turn this on/off?
Yes, there's a hidden pref browser.fixup.typo.scheme which defaults to true.
|
|
Assignee | |
Comment 6•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.29
|
|
Assignee | |
Comment 7•11 years ago
|
||
Aargh I checked it in with r=Ratty for some reason :-(
You need to log in
before you can comment on or make changes to this bug.
Description
•