1623222 - Status tip shows inconsistent result between find match and hover link

Status: RESOLVED FIXED

(Toolkit :: Find Toolbar, defect)

Description

[Masatoshi Kimura [:emk]]

Steps to reproduce:

1. Navigate to the URL.
2. Open findbar and search for あ.
3. Hover the mouse cursor over the link.

Actual result:
Step 2 Will show the status tip with http://example.com?あ.
Step 3 Will show the status tip with http://example.com?%82%A0.

Expected result:
Both step 2 and 3 should show the consistent result.

Comment 1

[Masatoshi Kimura [:emk]]

Attachment: Bug 1623222 - Remove the charset parameter from nsITextToSubURI::UnEscapeURIForUI. r?hsivonen

Only 3 callers are using a non-UTF-8 charset as the first parameter.

• MediaDocument.cpp: This does not make sense because the "filename" part of
  URLs will always be encoded with UTF-8.
• nsContextMenu.js: This is wrong because "mailto:" URLs don't care about the
  document charset.
• Finder.jsm: This caused bug 1623222.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f99558b13579
Remove the charset parameter from nsITextToSubURI::UnEscapeURIForUI. r=hsivonen

Comment 3

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/f99558b13579