<?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>147504</bug_id>
          
          <creation_ts>2015-07-31 14:14:09 -0700</creation_ts>
          <short_desc>[Cocoa] Latin quotes are used with the system font on Chinese devices</short_desc>
          <delta_ts>2015-08-10 20:07:05 -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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Myles C. Maxfield">mmaxfield</reporter>
          <assigned_to name="Myles C. Maxfield">mmaxfield</assigned_to>
          <cc>dino</cc>
    
    <cc>jonlee</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>thorton</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1113843</commentid>
    <comment_count>0</comment_count>
    <who name="Myles C. Maxfield">mmaxfield</who>
    <bug_when>2015-07-31 14:14:09 -0700</bug_when>
    <thetext>[Cocoa] Latin quotes are used with the system font on Chinese devices</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1113844</commentid>
    <comment_count>1</comment_count>
      <attachid>257958</attachid>
    <who name="Myles C. Maxfield">mmaxfield</who>
    <bug_when>2015-07-31 14:15:40 -0700</bug_when>
    <thetext>Created attachment 257958
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1113845</commentid>
    <comment_count>2</comment_count>
    <who name="Myles C. Maxfield">mmaxfield</who>
    <bug_when>2015-07-31 14:16:09 -0700</bug_when>
    <thetext>&lt;rdar://problem/22047626&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1113924</commentid>
    <comment_count>3</comment_count>
    <who name="Myles C. Maxfield">mmaxfield</who>
    <bug_when>2015-07-31 17:00:59 -0700</bug_when>
    <thetext>Committed r187693: &lt;http://trac.webkit.org/changeset/187693&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1116038</commentid>
    <comment_count>4</comment_count>
      <attachid>257958</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2015-08-08 14:07:24 -0700</bug_when>
    <thetext>Comment on attachment 257958
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=257958&amp;action=review

&gt; Source/WebCore/platform/graphics/mac/GlyphPageMac.cpp:45
&gt; +    if (fontData-&gt;platformData().isCompositeFontReference() || fontData-&gt;isSystemFont())

So this means that system fonts will never used our fast code path? I guess that’s OK since we don’t expect most websites to use them?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1116458</commentid>
    <comment_count>5</comment_count>
    <who name="Myles C. Maxfield">mmaxfield</who>
    <bug_when>2015-08-10 20:07:05 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; Comment on attachment 257958 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=257958&amp;action=review
&gt; 
&gt; &gt; Source/WebCore/platform/graphics/mac/GlyphPageMac.cpp:45
&gt; &gt; +    if (fontData-&gt;platformData().isCompositeFontReference() || fontData-&gt;isSystemFont())
&gt; 
&gt; So this means that system fonts will never used our fast code path? I guess
&gt; that’s OK since we don’t expect most websites to use them?

Yes.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>257958</attachid>
            <date>2015-07-31 14:15:40 -0700</date>
            <delta_ts>2015-07-31 14:20:38 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-147504-20150731141538.patch</filename>
            <type>text/plain</type>
            <size>1780</size>
            <attacher name="Myles C. Maxfield">mmaxfield</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTg3NjY4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZTNhZmNmYjdhNjI1YzMx
NDVmZmRmMjNjODNhNzZjZWRjYzgzMzcwMy4uNWUyMmIxZDhjYzdlODcwZjBmNzkxZWZjZTI3Zjc2
ZmI1ZmI3YWFjZiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDE1LTA3LTMxICBNeWxl
cyBDLiBNYXhmaWVsZCAgPG1tYXhmaWVsZEBhcHBsZS5jb20+CisKKyAgICAgICAgW0NvY29hXSBM
YXRpbiBxdW90ZXMgYXJlIHVzZWQgd2l0aCB0aGUgc3lzdGVtIGZvbnQgb24gQ2hpbmVzZSBkZXZp
Y2VzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNDc1
MDQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBUaGUg
c3lzdGVtIGZvbnQgaGFzIHNvbWUgZmFuY3kgbG9naWMgcmVnYXJkaW5nIGNoYXJhY3RlciBzZWxl
Y3Rpb24gd2hpY2ggcmVxdWlyZXMKKyAgICAgICAgdXNpbmcgQ29yZSBUZXh0IGZvciBnbHlwaCBz
ZWxlY3Rpb24uCisKKyAgICAgICAgTm8gbmV3IHRlc3RzIGJlY2F1c2UgdGVzdHMgY2FuJ3QgY2hh
bmdlIHRoZSBzeXN0ZW0gbGFuZ3VhZ2Ugb2YgdGhlIGRldmljZS4KKworICAgICAgICAqIHBsYXRm
b3JtL2dyYXBoaWNzL21hYy9HbHlwaFBhZ2VNYWMuY3BwOgorICAgICAgICAoV2ViQ29yZTo6c2hv
dWxkVXNlQ29yZVRleHQpOgorCiAyMDE1LTA3LTMxICBKb3NlcGggUGVjb3Jhcm8gIDxwZWNvcmFy
b0BhcHBsZS5jb20+CiAKICAgICAgICAgT25seSBnZW5lcmF0ZSAidGhlbWVXaW4qIiBVc2VyQWdl
bnRTdHlsZVNoZWV0IGNvbnN0YW50cyBmb3IgV2luZG93cyBidWlsZHMKZGlmZiAtLWdpdCBhL1Nv
dXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL21hYy9HbHlwaFBhZ2VNYWMuY3BwIGIvU291
cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvbWFjL0dseXBoUGFnZU1hYy5jcHAKaW5kZXgg
NjAzNmExMjcyOWY3MmVkZmQ3MzcxMGMwZjc4OWNlYjdiOGIxYWI0ZS4uNWE5NTdjNTIyYzZjMWM0
Y2RmZWM3NGM5Y2ZlYWU2YWEyNjI5YjU1OSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxh
dGZvcm0vZ3JhcGhpY3MvbWFjL0dseXBoUGFnZU1hYy5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUv
cGxhdGZvcm0vZ3JhcGhpY3MvbWFjL0dseXBoUGFnZU1hYy5jcHAKQEAgLTQyLDcgKzQyLDcgQEAg
bmFtZXNwYWNlIFdlYkNvcmUgewogCiBzdGF0aWMgYm9vbCBzaG91bGRVc2VDb3JlVGV4dChjb25z
dCBVQ2hhciogYnVmZmVyLCB1bnNpZ25lZCBidWZmZXJMZW5ndGgsIGNvbnN0IEZvbnQqIGZvbnRE
YXRhKQogewotICAgIGlmIChmb250RGF0YS0+cGxhdGZvcm1EYXRhKCkuaXNDb21wb3NpdGVGb250
UmVmZXJlbmNlKCkpCisgICAgaWYgKGZvbnREYXRhLT5wbGF0Zm9ybURhdGEoKS5pc0NvbXBvc2l0
ZUZvbnRSZWZlcmVuY2UoKSB8fCBmb250RGF0YS0+aXNTeXN0ZW1Gb250KCkpCiAgICAgICAgIHJl
dHVybiB0cnVlOwogICAgIGlmIChmb250RGF0YS0+cGxhdGZvcm1EYXRhKCkud2lkdGhWYXJpYW50
KCkgIT0gUmVndWxhcldpZHRoIHx8IGZvbnREYXRhLT5oYXNWZXJ0aWNhbEdseXBocygpKSB7CiAg
ICAgICAgIC8vIElkZW9ncmFwaHMgZG9uJ3QgaGF2ZSBhIHZlcnRpY2FsIHZhcmlhbnQgb3Igd2lk
dGggdmFyaWFudHMuCg==
</data>
<flag name="review"
          id="283140"
          type_id="1"
          status="+"
          setter="dino"
    />
          </attachment>
      

    </bug>

</bugzilla>