Bug 15680

Summary:

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

Product:

WebKit

Reporter:

Adele Peterson <adele>

Component:

Text

Assignee:

Alexey Proskuryakov <ap>

Status:

NEW

Severity:

Normal

CC:

Priority:

Keywords:

InRadar

Version:

523.x (Safari 3)

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
original test case	none
reduction that we fixed in r25429	none
reduction that's still broken	none
example of which characters to select	none
patch to use TextIterator in _web_attributedStringFromRange	none

Adele Peterson

Reported 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 "明日も" 4) Do reconversion by selecting "Reverse conversion" item from Text Input Menu, or hitting short cuts shift + control + r. In inlinehole, the first character "明" disappears and the reversion is done against the characters "日も猛". As expected, the reversion should be done against "明日も".

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 attachment proposed patch, testcase, etc.

Adele Peterson

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

Created attachment 16843 [details] original test case

Adele Peterson

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

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

Adele Peterson

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

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

Adele Peterson

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

Comment on attachment 16843 [details] original test case To reproduce with the original case, select the second "明日も".

Adele Peterson

Comment 5 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

Adele Peterson

Comment 6 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.

Adele Peterson

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

<rdar://problem/5428427>

Alexey Proskuryakov

Comment 8 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).

Adele Peterson

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

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

Alexey Proskuryakov

Comment 10 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>.

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