When a link in a non-UTF8 document links to a page with special characters, those characters are decoded correctly in the status bar when the link is hovered, but they are not decoded correctly in the URLbar when the link is visited. Steps to reproduce problem: 1. Visit the linked URL 2. Examine the status bar as each link is hovered 3. Visit each link and note the URLbar contents
Created attachment 8373082 [details] [diff] [review] Proposed patch The gotcha with unEscapeURIForUI is that it returns the original value rather than throwing an exception.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #8373082 - Flags: review?(philip.chee)
[comment] Please file a followup bug to port the relevant bits from: Bug 909264 - ASCII control characters stripped from address bar Bug 582186 - Zero-Width-Non-Joiner not rendered correctly in the address bar Bug 922530 - U+061C ARABIC LETTER MARK displays as tofu (i.e. is not invisible) http://hg.mozilla.org/mozilla-central/rev/30a7b37f0892#l1.12
Comment on attachment 8373082 [details] [diff] [review] Proposed patch r+ since it works if you left-click to follow the links.
Attachment #8373082 - Flags: review?(philip.chee) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Pushing to comm-release for SeaMonkey 2.26.1 $ hg tip changeset: 20184:fd910d1fd8f0 branch: SEA_2_26_1_RELBRANCH tag: tip user: Neil Rashbrook <firstname.lastname@example.org> date: Fri Feb 21 01:03:14 2014 +0000 summary: Bug 970121 Try to decode the URL in more cases r=Ratty
status-seamonkey2.26: --- → fixed
This caused a regression, because UnEscapeURIForUI unescapes reserved characters (so for instance %2F in a link shows as / in the status bar and now in the URL bar too).
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to comment #4) > Pushed comm-central changeset ed8c317c7a96. Backed out in comm-central changeset 3fdd4044a6e1.
Status: REOPENED → RESOLVED
Last Resolved: 4 years ago → 4 years ago
status-seamonkey2.27: --- → unaffected
status-seamonkey2.28: --- → affected
status-seamonkey2.29: --- → affected
status-seamonkey2.30: --- → wontfix
Resolution: --- → WONTFIX
status-seamonkey2.28: affected → wontfix
status-seamonkey2.29: affected → wontfix
You need to log in before you can comment on or make changes to this bug.