<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>73516</bug_id>
          
          <creation_ts>2011-11-30 20:59:05 -0800</creation_ts>
          <short_desc>[EFL] Fix broken UTF-8 characters issue on editor command.</short_desc>
          <delta_ts>2011-12-04 03:32:46 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit EFL</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Kangil Han">kangil.han</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>eric</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>leandro</cc>
    
    <cc>lucas.de.marchi</cc>
    
    <cc>rakuco</cc>
    
    <cc>rniwa</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>512408</commentid>
    <comment_count>0</comment_count>
    <who name="Kangil Han">kangil.han</who>
    <bug_when>2011-11-30 20:59:05 -0800</bug_when>
    <thetext>The issue has been described that Korean character is broken on pasting copied text.
In my analysis, this is from lack of format change between C style UTF-8 character pointer and C++ style String reference.
Therefore, this patch will convert types to resolve the problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>512413</commentid>
    <comment_count>1</comment_count>
      <attachid>117323</attachid>
    <who name="Kangil Han">kangil.han</who>
    <bug_when>2011-11-30 21:02:53 -0800</bug_when>
    <thetext>Created attachment 117323
patch file for this bug</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>512439</commentid>
    <comment_count>2</comment_count>
      <attachid>117323</attachid>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2011-11-30 21:37:01 -0800</bug_when>
    <thetext>Comment on attachment 117323
patch file for this bug

LGTM.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>512700</commentid>
    <comment_count>3</comment_count>
    <who name="Raphael Kubo da Costa (:rakuco)">rakuco</who>
    <bug_when>2011-12-01 05:10:20 -0800</bug_when>
    <thetext>The patch looks OK. Do you have any test case, or does it break any of the current layout tests?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>513652</commentid>
    <comment_count>4</comment_count>
    <who name="Kangil Han">kangil.han</who>
    <bug_when>2011-12-02 03:32:12 -0800</bug_when>
    <thetext>I&apos;ve heard that someone is working for api test case framework so this time I would like to land source patch only.

(In reply to comment #3)
&gt; The patch looks OK. Do you have any test case, or does it break any of the current layout tests?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>514580</commentid>
    <comment_count>5</comment_count>
      <attachid>117323</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2011-12-03 14:17:44 -0800</bug_when>
    <thetext>Comment on attachment 117323
patch file for this bug

How do you test this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>514666</commentid>
    <comment_count>6</comment_count>
      <attachid>117323</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-12-04 03:32:42 -0800</bug_when>
    <thetext>Comment on attachment 117323
patch file for this bug

Clearing flags on attachment: 117323

Committed r101952: &lt;http://trac.webkit.org/changeset/101952&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>514667</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-12-04 03:32:46 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>117323</attachid>
            <date>2011-11-30 21:02:53 -0800</date>
            <delta_ts>2011-12-04 03:32:41 -0800</delta_ts>
            <desc>patch file for this bug</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>1691</size>
            <attacher name="Kangil Han">kangil.han</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvZWZsL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQv
ZWZsL0NoYW5nZUxvZwppbmRleCBmZDQxNjhiLi4zOGI3ZjVmIDEwMDc1NQotLS0gYS9Tb3VyY2Uv
V2ViS2l0L2VmbC9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdC9lZmwvQ2hhbmdlTG9nCkBA
IC0xLDMgKzEsMTcgQEAKKzIwMTEtMTEtMzAgIEthbmdpbCBIYW4gIDxrYW5naWwuaGFuQHNhbXN1
bmcuY29tPgorCisgICAgICAgIFtFRkxdIEZpeCBicm9rZW4gVVRGLTggY2hhcmFjdGVycyBpc3N1
ZSBvbiBlZGl0b3IgY29tbWFuZC4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTczNTE2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgVGhlIGlzc3VlIGhhcyBiZWVuIGRlc2NyaWJlZCB0aGF0IEtvcmVhbiBjaGFy
YWN0ZXIgaXMgYnJva2VuIG9uIHBhc3RpbmcgY29waWVkIHRleHQuCisgICAgICAgIEluIG15IGFu
YWx5c2lzLCB0aGlzIGlzIGZyb20gbGFjayBvZiBmb3JtYXQgY2hhbmdlIGJldHdlZW4gQyBzdHls
ZSBVVEYtOCBjaGFyYWN0ZXIgcG9pbnRlciBhbmQgQysrIHN0eWxlIFN0cmluZyByZWZlcmVuY2Uu
CisgICAgICAgIFRoZXJlZm9yZSwgdGhpcyBwYXRjaCB3aWxsIGNvbnZlcnQgdHlwZXMgdG8gcmVz
b2x2ZSB0aGUgcHJvYmxlbS4KKworICAgICAgICAqIGV3ay9ld2tfdmlldy5jcHA6CisgICAgICAg
IChfZXdrX3ZpZXdfZWRpdG9yX2NvbW1hbmQpOgorCiAyMDExLTExLTMwICBHeXV5b3VuZyBLaW0g
IDxneXV5b3VuZy5raW1Ac2Ftc3VuZy5jb20+CiAKICAgICAgICAgW0VGTF0gUmVtb3ZlIGR1cGxp
Y2F0ZWQgVUEgaW5mb3JtYXRpb24gZnVuY3Rpb24uCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0
L2VmbC9ld2svZXdrX3ZpZXcuY3BwIGIvU291cmNlL1dlYktpdC9lZmwvZXdrL2V3a192aWV3LmNw
cAppbmRleCA4MjgwMmMyLi41MDExN2UyIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0L2VmbC9l
d2svZXdrX3ZpZXcuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvZWZsL2V3ay9ld2tfdmlldy5jcHAK
QEAgLTE0MjMsNyArMTQyMyw3IEBAIGNoYXIqIGV3a192aWV3X3NlbGVjdGlvbl9nZXQoY29uc3Qg
RXZhc19PYmplY3QqIGV3a1ZpZXcpCiAKIHN0YXRpYyBFaW5hX0Jvb2wgX2V3a192aWV3X2VkaXRv
cl9jb21tYW5kKEV3a19WaWV3X1ByaXZhdGVfRGF0YSogcHJpdiwgY29uc3QgY2hhciogY29tbWFu
ZCwgY29uc3QgY2hhciogdmFsdWUgPSAwKQogewotICAgIHJldHVybiBwcml2LT5wYWdlLT5mb2N1
c0NvbnRyb2xsZXIoKS0+Zm9jdXNlZE9yTWFpbkZyYW1lKCktPmVkaXRvcigpLT5jb21tYW5kKFdU
Rjo6U3RyaW5nOjpmcm9tVVRGOChjb21tYW5kKSkuZXhlY3V0ZSh2YWx1ZSk7CisgICAgcmV0dXJu
IHByaXYtPnBhZ2UtPmZvY3VzQ29udHJvbGxlcigpLT5mb2N1c2VkT3JNYWluRnJhbWUoKS0+ZWRp
dG9yKCktPmNvbW1hbmQoV1RGOjpTdHJpbmc6OmZyb21VVEY4KGNvbW1hbmQpKS5leGVjdXRlKFdU
Rjo6U3RyaW5nOjpmcm9tVVRGOCh2YWx1ZSkpOwogfQogCiBFaW5hX0Jvb2wgZXdrX3ZpZXdfZXhl
Y3V0ZV9lZGl0b3JfY29tbWFuZChFdmFzX09iamVjdCogZXdrVmlldywgY29uc3QgRXdrX0VkaXRv
cl9Db21tYW5kIGNvbW1hbmQsIGNvbnN0IGNoYXIqIHZhbHVlKQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>