<?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>25657</bug_id>
          
          <creation_ts>2009-05-08 17:12:00 -0700</creation_ts>
          <short_desc>[Chromium] Non-BMP characters are not rendered correctly</short_desc>
          <delta_ts>2009-06-19 10:49:53 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Layout and Rendering</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</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>Minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jungshik Shin">jshin</reporter>
          <assigned_to name="Jungshik Shin">jshin</assigned_to>
          <cc>agl</cc>
    
    <cc>ap</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>120492</commentid>
    <comment_count>0</comment_count>
    <who name="Jungshik Shin">jshin</who>
    <bug_when>2009-05-08 17:12:00 -0700</bug_when>
    <thetext>In getFontForCharacters in FontCacheLinux.cpp in platform/graphics/chromium does not handle surrogate pairs for non-BMP characters properly. 

It is trivial to fix, but I need to make a simple TTF supporting a few non-BMP characters to check in for layout test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>120498</commentid>
    <comment_count>1</comment_count>
    <who name="Adam Langley">agl</who>
    <bug_when>2009-05-08 18:25:24 -0700</bug_when>
    <thetext>There are lots of changes pending in the Linux font code. I&apos;m waiting on the Skia unforking. Please don&apos;t try and fix anything, you&apos;ll just make merging harder!

Thanks</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>125026</commentid>
    <comment_count>2</comment_count>
    <who name="Jungshik Shin">jshin</who>
    <bug_when>2009-06-09 15:38:39 -0700</bug_when>
    <thetext>Adam, are you done with the Skia unforking? 

Actually, the change does not touch Skia at all. It&apos;s like this in FontCacheLinux.cpp. 

+#include &lt;unicode/utf16.h&gt;
 #include &lt;wtf/Assertions.h&gt;
 
 namespace WebCore {
@@ -59,8 +60,11 @@
                                                           int length)
 {
     FcCharSet* cset = FcCharSetCreate();
-    for (int i = 0; i &lt; length; ++i)
-        FcCharSetAddChar(cset, characters[i]);
+    for (int i = 0; i &lt; length; ++i) {
+        UChar32 ucs4 = 0;
+        U16_NEXT(characters, i, length, ucs4);
+        FcCharSetAddChar(cset, ucs4);
+    }
 
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>125028</commentid>
    <comment_count>3</comment_count>
    <who name="Adam Langley">agl</who>
    <bug_when>2009-06-09 15:45:29 -0700</bug_when>
    <thetext>Yes, Skia is unforked. I just don&apos;t have time to work on complex text stuff right now. Feel free to land a patch if you like.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>125392</commentid>
    <comment_count>4</comment_count>
      <attachid>31175</attachid>
    <who name="Jungshik Shin">jshin</who>
    <bug_when>2009-06-11 16:12:19 -0700</bug_when>
    <thetext>Created attachment 31175
patch without the layout test result

I have yet to make a freely-distributable font with a few non-BMP glyphs to use for layout test. Therefore, the test is attached without a result.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>125779</commentid>
    <comment_count>5</comment_count>
      <attachid>31175</attachid>
    <who name="Adam Langley">agl</who>
    <bug_when>2009-06-14 10:28:37 -0700</bug_when>
    <thetext>Comment on attachment 31175
patch without the layout test result

&gt; +    for (int i = 0; i &lt; length; ++i) {
&gt; +        UChar32 ucs4 = 0;

Setting this to zero every time would appear to be a waste.

&gt; +        U16_NEXT(characters, i, length, ucs4);

U16_NEXT is a terrible name for this macro! (Nothing to do with with patch though).

From looking at the macro, it appears that it increments |i| for you:
  https://dev.mobileread.com/trac/webkitbrowser/browser/trunk/WebKit-r30377/WebKit/mac/icu/unicode/utf16.h?rev=3
However, you are also incrementing i in the for loop. Might not you be skipping charactors?



AGL</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>126297</commentid>
    <comment_count>6</comment_count>
      <attachid>31376</attachid>
    <who name="Jungshik Shin">jshin</who>
    <bug_when>2009-06-16 14:38:33 -0700</bug_when>
    <thetext>Created attachment 31376
patch with a test put in manual-tests

I put non-bmp.html in WebCore/manual-tests. Once I make a small font covering non-BMP and contribute to the webkit, I&apos;ll move the test to fast/text.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>126307</commentid>
    <comment_count>7</comment_count>
    <who name="Jungshik Shin">jshin</who>
    <bug_when>2009-06-16 14:47:30 -0700</bug_when>
    <thetext>(In reply to comment #5)


&gt; 
&gt; From looking at the macro, it appears that it increments |i| for you:
&gt;  
&gt; https://dev.mobileread.com/trac/webkitbrowser/browser/trunk/WebKit-r30377/WebKit/mac/icu/unicode/utf16.h?rev=3
&gt; However, you are also incrementing i in the for loop. Might not you be skipping
&gt; charactors?

You&apos;re right. I shouldn&apos;t have resurrected the patch from the memory. When I tested it on Linux, I used while loop. 
Thanks for catching it. 
 
&gt; 
&gt; AGL
&gt; 

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>126380</commentid>
    <comment_count>8</comment_count>
      <attachid>31398</attachid>
    <who name="Jungshik Shin">jshin</who>
    <bug_when>2009-06-16 21:58:58 -0700</bug_when>
    <thetext>Created attachment 31398
patch with a test put in manual-tests

fixed a &apos;double&apos; iteratorion issue pointed out by agl.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>126749</commentid>
    <comment_count>9</comment_count>
      <attachid>31398</attachid>
    <who name="David Levin">levin</who>
    <bug_when>2009-06-18 12:35:45 -0700</bug_when>
    <thetext>Comment on attachment 31398
patch with a test put in manual-tests

There are tabs in the changelog which will need to be fixed on landing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>126962</commentid>
    <comment_count>10</comment_count>
    <who name="Jungshik Shin">jshin</who>
    <bug_when>2009-06-19 10:49:53 -0700</bug_when>
    <thetext>Landed in r44858 ( http://trac.webkit.org/changeset/44858)
Sending        WebCore/ChangeLog
Adding         WebCore/manual-tests/non-bmp.html
Sending        WebCore/platform/graphics/chromium/FontCacheLinux.cpp

</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>31175</attachid>
            <date>2009-06-11 16:12:19 -0700</date>
            <delta_ts>2009-06-16 14:38:33 -0700</delta_ts>
            <desc>patch without the layout test result</desc>
            <filename>25657.patch.1</filename>
            <type>text/plain</type>
            <size>2736</size>
            <attacher name="Jungshik Shin">jshin</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0NDYxOSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTggQEAKKzIwMDktMDYtMTEgIEp1bmdzaGlrIFNoaW4gIDxqc2hpbkBjaHJvbWl1
bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI1NjU3CisgICAgICAgIENo
cm9taXVtIExpbnV4IHBvcnQgZG9lcyBub3QgaGFuZGxlIG5vbi1CTVAgY2hhcmFjdGVycyBwcm9w
ZXJseS4KKyAgICAgICAgSXQncyBmaXhlZCB3aXRoIGEgJ1VURi0xNiBpdGVyYXRvciBtYWNybycg
dG8gZXh0cmFjdCBVbmljb2RlCisgICAgICAgIGNvZGVwb2ludHMgb3V0IG9mIGEgVVRGLTE2IGlu
cHV0IHN0cmluZy4KKworICAgICAgICBUZXN0OiBmYXN0L3RleHQvbm9uLWJtcC5odG1sCisKKyAg
ICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9jaHJvbWl1bS9Gb250Q2FjaGVMaW51eC5jcHA6Cisg
ICAgICAgIChXZWJDb3JlOjpGb250Q2FjaGU6OmdldEZvbnREYXRhRm9yQ2hhcmFjdGVycyk6CisK
KwogMjAwOS0wNi0wMyAgRXJpYyBTZWlkZWwgIDxlcmljQHdlYmtpdC5vcmc+CiAKICAgICAgICAg
UmV2aWV3ZWQgYnkgRGFyaW4gQWRsZXIuCkluZGV4OiBXZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNz
L2Nocm9taXVtL0ZvbnRDYWNoZUxpbnV4LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL3BsYXRm
b3JtL2dyYXBoaWNzL2Nocm9taXVtL0ZvbnRDYWNoZUxpbnV4LmNwcAkocmV2aXNpb24gNDQ2MTkp
CisrKyBXZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2Nocm9taXVtL0ZvbnRDYWNoZUxpbnV4LmNw
cAkod29ya2luZyBjb3B5KQpAQCAtNDYsNiArNDYsNyBAQAogI2luY2x1ZGUgIlNrVHlwZWZhY2Uu
aCIKICNpbmNsdWRlICJTa1V0aWxzLmgiCiAKKyNpbmNsdWRlIDx1bmljb2RlL3V0ZjE2Lmg+CiAj
aW5jbHVkZSA8d3RmL0Fzc2VydGlvbnMuaD4KIAogbmFtZXNwYWNlIFdlYkNvcmUgewpAQCAtNTks
OCArNjAsMTEgQEAgY29uc3QgU2ltcGxlRm9udERhdGEqIEZvbnRDYWNoZTo6Z2V0Rm9udAogICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIGlu
dCBsZW5ndGgpCiB7CiAgICAgRmNDaGFyU2V0KiBjc2V0ID0gRmNDaGFyU2V0Q3JlYXRlKCk7Ci0g
ICAgZm9yIChpbnQgaSA9IDA7IGkgPCBsZW5ndGg7ICsraSkKLSAgICAgICAgRmNDaGFyU2V0QWRk
Q2hhcihjc2V0LCBjaGFyYWN0ZXJzW2ldKTsKKyAgICBmb3IgKGludCBpID0gMDsgaSA8IGxlbmd0
aDsgKytpKSB7CisgICAgICAgIFVDaGFyMzIgdWNzNCA9IDA7CisgICAgICAgIFUxNl9ORVhUKGNo
YXJhY3RlcnMsIGksIGxlbmd0aCwgdWNzNCk7CisgICAgICAgIEZjQ2hhclNldEFkZENoYXIoY3Nl
dCwgdWNzNCk7CisgICAgfQogCiAgICAgRmNQYXR0ZXJuKiBwYXR0ZXJuID0gRmNQYXR0ZXJuQ3Jl
YXRlKCk7CiAKSW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlv
dXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlzaW9uIDQ0NjE5KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMDktMDYtMTEgIEp1bmdzaGlr
IFNoaW4gIDxqc2hpbkBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTI1NjU3CisKKyAgICAgICAgVGVzdCBmb3Igbm9uLUJNUCBjaGFyYWN0ZXIgcmVuZGVyaW5n
LgorCisgICAgICAgICogZmFzdC90ZXh0L25vbi1ibXAuaHRtbDogQWRkZWQuCisKIDIwMDktMDYt
MTEgIEVyaWMgU2VpZGVsICA8ZXJpY0B3ZWJraXQub3JnPgogCiAgICAgICAgIFJ1YmJlciBzdGFt
cGVkIGJ5IERhdmUgTGV2aW4uCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L3RleHQvbm9uLWJtcC5o
dG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvdGV4dC9ub24tYm1wLmh0bWwJKHJl
dmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L3RleHQvbm9uLWJtcC5odG1sCShyZXZpc2lv
biAwKQpAQCAtMCwwICsxLDEyIEBACis8aHRtbD4KKzxoZWFkPgorPHRpdGxlPkJ1ZyAyNTY1NyA6
IE5vbi1CTVAgY2hhcmFjdGVyczwvdGl0bGU+Cis8L2hlYWQ+Cis8Ym9keT4KK1Rlc3QgZm9yIDxh
IGhyZWY9Imh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI1NjU3Ij5idWcg
MjU2NTc8L2E+IDogCitMaXN0ZWQgYmVsb3cgYXJlIE1hdGhlbWF0aWNhbCBCb2xkIENhcGl0YWwg
QSB0aHJvdWdoIEUgKFUrMUQ0MDAgLi4gVSsxRDQwNCkuCis8ZGl2PgorJiN4MWQ0MDA7JiN4MWQ0
MDE7JiN4MWQ0MDI7JiN4MWQ0MDM7JiN4MWQ0MDQ7Cis8L2Rpdj4KKzwvYm9keT4KKzwvaHRtbD4K
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>31376</attachid>
            <date>2009-06-16 14:38:33 -0700</date>
            <delta_ts>2009-06-16 14:47:52 -0700</delta_ts>
            <desc>patch with a test put in manual-tests</desc>
            <filename>25657.patch.2</filename>
            <type>text/plain</type>
            <size>2481</size>
            <attacher name="Jungshik Shin">jshin</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0NDczOCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMjEgQEAKKzIwMDktMDYtMTYgIEp1bmdzaGlrIFNoaW4gIDxqc2hpbkBjaHJvbWl1
bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI1NjU3CisKKyAgICAgICAg
Q2hyb21pdW0gTGludXggcG9ydCBkb2VzIG5vdCBoYW5kbGUgbm9uLUJNUCBjaGFyYWN0ZXJzIHBy
b3Blcmx5LgorICAgICAgICBJdCdzIGZpeGVkIHdpdGggYSAnVVRGLTE2IGl0ZXJhdG9yIG1hY3Jv
JyB0byBleHRyYWN0IFVuaWNvZGUKKyAgICAgICAgY29kZXBvaW50cyBvdXQgb2YgYSBVVEYtMTYg
aW5wdXQgc3RyaW5nLgorCUEgbWFudWFsIHRlc3QgaXMgYWRkZWQgZm9yIG5vbi1CTVAgY2hhcmFj
dGVyIHJlbmRlcmluZywKKwl3aGljaCB3ZSBjYW4gdXNlIHVudGlsIGEgc21hbGwgZnJlZWx5LWRp
c3RyaWJ1dGFibGUgZm9udAorCWNvdmVyaW5nIG5vbi1CTVAgaXMgYWRkZWQgdG8gdGhlIFdlYmtp
dCBzb3VyY2UgdHJlZS4KKworCisgICAgICAgICogcGxhdGZvcm0vZ3JhcGhpY3MvY2hyb21pdW0v
Rm9udENhY2hlTGludXguY3BwOgorICAgICAgICAoV2ViQ29yZTo6Rm9udENhY2hlOjpnZXRGb250
RGF0YUZvckNoYXJhY3RlcnMpOgorICAgICAgICAqIG1hbnVhbC10ZXN0cy9ub24tYm1wLmh0bWw6
IEFkZGVkLgorCiAyMDA5LTA2LTE2ICBFcmljIENhcmxzb24gIDxlcmljLmNhcmxzb25AYXBwbGUu
Y29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IFNpbW9uIEZyYXNlci4KSW5kZXg6IFdlYkNvcmUv
bWFudWFsLXRlc3RzL25vbi1ibXAuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL21hbnVhbC10
ZXN0cy9ub24tYm1wLmh0bWwJKHJldmlzaW9uIDApCisrKyBXZWJDb3JlL21hbnVhbC10ZXN0cy9u
b24tYm1wLmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMTIgQEAKKzxodG1sPgorPGhlYWQ+
Cis8dGl0bGU+QnVnIDI1NjU3IDogTm9uLUJNUCBjaGFyYWN0ZXJzPC90aXRsZT4KKzwvaGVhZD4K
Kzxib2R5PgorVGVzdCBmb3IgPGEgaHJlZj0iaHR0cDovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MjU2NTciPmJ1ZyAyNTY1NzwvYT4gOiAKK0xpc3RlZCBiZWxvdyBhcmUgTWF0aGVt
YXRpY2FsIEJvbGQgQ2FwaXRhbCBBIHRocm91Z2ggRSAoVSsxRDQwMCAuLiBVKzFENDA0KS4gTm90
ZSB0aGF0IHlvdSBuZWVkIHRvIGluc3RhbGwgYSBmb250IGNvdmVyaW5nIFBsYW5lIDEgbGlrZSBD
b2RlMjAwMSBmb3IgdGhpcyB0ZXN0LgorPGRpdj4KKyYjeDFkNDAwOyYjeDFkNDAxOyYjeDFkNDAy
OyYjeDFkNDAzOyYjeDFkNDA0OworPC9kaXY+Cis8L2JvZHk+Cis8L2h0bWw+CkluZGV4OiBXZWJD
b3JlL3BsYXRmb3JtL2dyYXBoaWNzL2Nocm9taXVtL0ZvbnRDYWNoZUxpbnV4LmNwcAo9PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09Ci0tLSBXZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2Nocm9taXVtL0ZvbnRDYWNoZUxpbnV4
LmNwcAkocmV2aXNpb24gNDQ3MzEpCisrKyBXZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2Nocm9t
aXVtL0ZvbnRDYWNoZUxpbnV4LmNwcAkod29ya2luZyBjb3B5KQpAQCAtNDYsNiArNDYsNyBAQAog
I2luY2x1ZGUgIlNrVHlwZWZhY2UuaCIKICNpbmNsdWRlICJTa1V0aWxzLmgiCiAKKyNpbmNsdWRl
IDx1bmljb2RlL3V0ZjE2Lmg+CiAjaW5jbHVkZSA8d3RmL0Fzc2VydGlvbnMuaD4KIAogbmFtZXNw
YWNlIFdlYkNvcmUgewpAQCAtNTksOCArNjAsMTEgQEAgY29uc3QgU2ltcGxlRm9udERhdGEqIEZv
bnRDYWNoZTo6Z2V0Rm9udAogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgIGludCBsZW5ndGgpCiB7CiAgICAgRmNDaGFyU2V0KiBjc2V0ID0g
RmNDaGFyU2V0Q3JlYXRlKCk7Ci0gICAgZm9yIChpbnQgaSA9IDA7IGkgPCBsZW5ndGg7ICsraSkK
LSAgICAgICAgRmNDaGFyU2V0QWRkQ2hhcihjc2V0LCBjaGFyYWN0ZXJzW2ldKTsKKyAgICBmb3Ig
KGludCBpID0gMDsgaSA8IGxlbmd0aDsgKytpKSB7CisgICAgICAgIFVDaGFyMzIgdWNzNCA9IDA7
CisgICAgICAgIFUxNl9ORVhUKGNoYXJhY3RlcnMsIGksIGxlbmd0aCwgdWNzNCk7CisgICAgICAg
IEZjQ2hhclNldEFkZENoYXIoY3NldCwgdWNzNCk7CisgICAgfQogCiAgICAgRmNQYXR0ZXJuKiBw
YXR0ZXJuID0gRmNQYXR0ZXJuQ3JlYXRlKCk7CiAK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>31398</attachid>
            <date>2009-06-16 21:58:58 -0700</date>
            <delta_ts>2009-06-18 12:35:45 -0700</delta_ts>
            <desc>patch with a test put in manual-tests</desc>
            <filename>25657.patch.2a</filename>
            <type>text/plain</type>
            <size>2478</size>
            <attacher name="Jungshik Shin">jshin</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0NDczOCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMjEgQEAKKzIwMDktMDYtMTYgIEp1bmdzaGlrIFNoaW4gIDxqc2hpbkBjaHJvbWl1
bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI1NjU3CisKKyAgICAgICAg
Q2hyb21pdW0gTGludXggcG9ydCBkb2VzIG5vdCBoYW5kbGUgbm9uLUJNUCBjaGFyYWN0ZXJzIHBy
b3Blcmx5LgorICAgICAgICBJdCdzIGZpeGVkIHdpdGggYSAnVVRGLTE2IGl0ZXJhdG9yIG1hY3Jv
JyB0byBleHRyYWN0IFVuaWNvZGUKKyAgICAgICAgY29kZXBvaW50cyBvdXQgb2YgYSBVVEYtMTYg
aW5wdXQgc3RyaW5nLgorCUEgbWFudWFsIHRlc3QgaXMgYWRkZWQgZm9yIG5vbi1CTVAgY2hhcmFj
dGVyIHJlbmRlcmluZywKKwl3aGljaCB3ZSBjYW4gdXNlIHVudGlsIGEgc21hbGwgZnJlZWx5LWRp
c3RyaWJ1dGFibGUgZm9udAorCWNvdmVyaW5nIG5vbi1CTVAgaXMgYWRkZWQgdG8gdGhlIFdlYmtp
dCBzb3VyY2UgdHJlZS4KKworCisgICAgICAgICogcGxhdGZvcm0vZ3JhcGhpY3MvY2hyb21pdW0v
Rm9udENhY2hlTGludXguY3BwOgorICAgICAgICAoV2ViQ29yZTo6Rm9udENhY2hlOjpnZXRGb250
RGF0YUZvckNoYXJhY3RlcnMpOgorICAgICAgICAqIG1hbnVhbC10ZXN0cy9ub24tYm1wLmh0bWw6
IEFkZGVkLgorCiAyMDA5LTA2LTE2ICBFcmljIENhcmxzb24gIDxlcmljLmNhcmxzb25AYXBwbGUu
Y29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IFNpbW9uIEZyYXNlci4KSW5kZXg6IFdlYkNvcmUv
bWFudWFsLXRlc3RzL25vbi1ibXAuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL21hbnVhbC10
ZXN0cy9ub24tYm1wLmh0bWwJKHJldmlzaW9uIDApCisrKyBXZWJDb3JlL21hbnVhbC10ZXN0cy9u
b24tYm1wLmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMTIgQEAKKzxodG1sPgorPGhlYWQ+
Cis8dGl0bGU+QnVnIDI1NjU3IDogTm9uLUJNUCBjaGFyYWN0ZXJzPC90aXRsZT4KKzwvaGVhZD4K
Kzxib2R5PgorVGVzdCBmb3IgPGEgaHJlZj0iaHR0cDovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MjU2NTciPmJ1ZyAyNTY1NzwvYT4gOiAKK0xpc3RlZCBiZWxvdyBhcmUgTWF0aGVt
YXRpY2FsIEJvbGQgQ2FwaXRhbCBBIHRocm91Z2ggRSAoVSsxRDQwMCAuLiBVKzFENDA0KS4gTm90
ZSB0aGF0IHlvdSBuZWVkIHRvIGluc3RhbGwgYSBmb250IGNvdmVyaW5nIFBsYW5lIDEgbGlrZSBD
b2RlMjAwMSBmb3IgdGhpcyB0ZXN0LgorPGRpdj4KKyYjeDFkNDAwOyYjeDFkNDAxOyYjeDFkNDAy
OyYjeDFkNDAzOyYjeDFkNDA0OworPC9kaXY+Cis8L2JvZHk+Cis8L2h0bWw+CkluZGV4OiBXZWJD
b3JlL3BsYXRmb3JtL2dyYXBoaWNzL2Nocm9taXVtL0ZvbnRDYWNoZUxpbnV4LmNwcAo9PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09Ci0tLSBXZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2Nocm9taXVtL0ZvbnRDYWNoZUxpbnV4
LmNwcAkocmV2aXNpb24gNDQ3MzEpCisrKyBXZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2Nocm9t
aXVtL0ZvbnRDYWNoZUxpbnV4LmNwcAkod29ya2luZyBjb3B5KQpAQCAtNDYsNiArNDYsNyBAQAog
I2luY2x1ZGUgIlNrVHlwZWZhY2UuaCIKICNpbmNsdWRlICJTa1V0aWxzLmgiCiAKKyNpbmNsdWRl
IDx1bmljb2RlL3V0ZjE2Lmg+CiAjaW5jbHVkZSA8d3RmL0Fzc2VydGlvbnMuaD4KIAogbmFtZXNw
YWNlIFdlYkNvcmUgewpAQCAtNTksOCArNjAsMTEgQEAgY29uc3QgU2ltcGxlRm9udERhdGEqIEZv
bnRDYWNoZTo6Z2V0Rm9udAogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgIGludCBsZW5ndGgpCiB7CiAgICAgRmNDaGFyU2V0KiBjc2V0ID0g
RmNDaGFyU2V0Q3JlYXRlKCk7Ci0gICAgZm9yIChpbnQgaSA9IDA7IGkgPCBsZW5ndGg7ICsraSkK
LSAgICAgICAgRmNDaGFyU2V0QWRkQ2hhcihjc2V0LCBjaGFyYWN0ZXJzW2ldKTsKKyAgICBmb3Ig
KGludCBpID0gMDsgaSA8IGxlbmd0aDsgKSB7CisgICAgICAgIFVDaGFyMzIgdWNzNCA9IDA7Cisg
ICAgICAgIFUxNl9ORVhUKGNoYXJhY3RlcnMsIGksIGxlbmd0aCwgdWNzNCk7CisgICAgICAgIEZj
Q2hhclNldEFkZENoYXIoY3NldCwgdWNzNCk7CisgICAgfQogCiAgICAgRmNQYXR0ZXJuKiBwYXR0
ZXJuID0gRmNQYXR0ZXJuQ3JlYXRlKCk7CiAK
</data>
<flag name="review"
          id="16065"
          type_id="1"
          status="+"
          setter="levin"
    />
          </attachment>
      

    </bug>

</bugzilla>