190886 - Non-Ascii Anchor does not work

Status: VERIFIED FIXED

(Core :: Internationalization, defect, P1)

Description

[Teruko Kobayashi]

Named anchors with Japanese text does not work.  

Steps of reproduce

1. Go to above url
2. Click on Japanese anchor

ACTUAL RESULTS
Nothing happens, only the URLbar updates to
http://babel/Tests/Browser/HTML/sjis/a_name_sjis.html#%93%FA%96{%8C%EA

EXPECTED RESULTS:
The page scrolls down

Tested 1-27-09 Win32 Trunk build on WinXPJ and Win98 J.  This is reproduciable
in 1-27 MachO trunk build. 

This is a regression bug fro 7.01.

Comment 1

[Yuying Long]

Attachment: test page include work and not work link

The first one is same as in the original report; 
the sencond one works if including the whole path.

Is this because of start with "#" inside the anchor statement?

Comment 2

[nhottanscp]

>This is a regression bug fro 7.01.
Is this a recent regression for mozilla 1.3b? Does mozilla 1.3a have this?

Comment 3

[Teruko Kobayashi]

>Does mozilla 1.3a have this?
I checked the 1.3a 12-12-15 Win32 build.  This works fine in the build.

Comment 4

[Teruko Kobayashi]

I found out that this started to happen from 1-14-08 Trunk build.  This worked
fine until 1-13-08 Trunk build.

Comment 5

[Boris Zbarsky [:bzbarsky]]

It also does not work in the 13-21 build.  I suspect bug 188734 at the moment...

Comment 6

[Boris Zbarsky [:bzbarsky]]

Attachment: patch

Comment 7

[Boris Zbarsky [:bzbarsky]]

taking

Comment 8

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 112897 [details] [diff] [review]
patch

r+sr=dbaron, but please comment explaining why you're doing this (i.e.,
conversion errors, I assume)

Comment 9

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 112897 [details] [diff] [review]
patch

This will affect a lot of non-English sites.. simple very safe fix.  Could this
please be approved for 1.3b?

Comment 10

[Asa Dotzler [:asa]]

Comment on attachment 112897 [details] [diff] [review]
patch

a=asa (on behalf of drivers) for checkin to 1.3beta.

Comment 11

[Boris Zbarsky [:bzbarsky]]

fixed.

Comment 12

[Teruko Kobayashi]

I verified this as fixed with 1-29 Win32, MacOS, and Linux builds.