15680 – Wrong characters are re-converted after doing recoversion with Kotoeri with particular HTML markup

Bug 15680 - Wrong characters are re-converted after doing recoversion with Kotoeri with particular HTML markup

Summary: Wrong characters are re-converted after doing recoversion with Kotoeri with p...

Status:	NEW

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	Text (show other bugs)
Version:	523.x (Safari 3)
Hardware:	Mac OS X 10.4

Importance:	P1 Normal
Assignee:	Alexey Proskuryakov

URL:
Keywords:	InRadar

Depends on:
Blocks:

Reported:	2007-10-24 14:34 PDT by Adele Peterson
Modified:	2008-05-15 06:47 PDT (History)
CC List:	1 user (show)

See Also:

Attachments
original test case (1.69 KB, text/html) 2007-10-24 14:48 PDT, Adele Peterson	no flags	Details
reduction that we fixed in r25429 (198 bytes, text/html) 2007-10-24 14:49 PDT, Adele Peterson	no flags	Details
reduction that's still broken (209 bytes, text/html) 2007-10-24 14:50 PDT, Adele Peterson	no flags	Details
example of which characters to select (6.80 KB, image/png) 2007-10-24 14:56 PDT, Adele Peterson	no flags	Details
patch to use TextIterator in _web_attributedStringFromRange (26.70 KB, text/plain) 2007-10-24 15:01 PDT, Adele Peterson	no flags	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Adele Peterson 2007-10-24 14:34:31 PDT

Steps to reproduce:
1) Enable Kotoeri input method on International preferences.
2) Open test cases in Safari
3) Select "&#26126;&#26085;&#12418;"
4) Do reconversion by selecting "Reverse conversion" item from Text Input Menu, or hitting short cuts shift + control + r.

In inlinehole, the first character "&#26126;" disappears and the reversion is done against the characters "&#26085;&#12418;&#29467;".
As expected, the reversion should be done against "&#26126;&#26085;&#12418;".

Comment 1 Adele Peterson 2007-10-24 14:48:43 PDT

Created attachment 16843 [details]
original test case

Comment 2 Adele Peterson 2007-10-24 14:49:53 PDT

Created attachment 16844 [details]
reduction that we fixed in r25429

Comment 3 Adele Peterson 2007-10-24 14:50:14 PDT

Created attachment 16845 [details]
reduction that's still broken

Comment 4 Adele Peterson 2007-10-24 14:51:59 PDT

Comment on attachment 16843 [details]
original test case

To reproduce with the original case, select the second "&#26126;&#26085;&#12418;".

Comment 5 Adele Peterson 2007-10-24 14:56:37 PDT

Created attachment 16846 [details]
example of which characters to select

Here's a picture of which characters to select, since they're escaped in the steps to reproduce

Comment 6 Adele Peterson 2007-10-24 15:01:32 PDT

Created attachment 16847 [details]
patch to use TextIterator in _web_attributedStringFromRange

Although this patch doesn't fix the bug, I think its an improvement we should make, and it will probably make analysis of these types of bugs easier.

Comment 7 Adele Peterson 2007-10-24 15:02:15 PDT

<rdar://problem/5428427>

Comment 8 Alexey Proskuryakov 2007-10-25 04:19:31 PDT

> reduction that's still broken

FWIW, I cannot reproduce this on Tiger with r26989 nightly, or with r27033 local debug build. Even Tiger Safari/WebKit work fine for me with this particular selection (but not with others).

Comment 9 Adele Peterson 2007-10-25 20:19:59 PDT

hmm maybe its a leopard-only bug.  maybe a change in NSAttributedString.

Comment 10 Alexey Proskuryakov 2008-05-15 06:47:19 PDT

(In reply to comment #6)
> Although this patch doesn't fix the bug, I think its an improvement we should
> make, and it will probably make analysis of these types of bugs easier.

Landed in <http://trac.webkit.org/changeset/33486>.