<?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>136841</bug_id>
          
          <creation_ts>2014-09-15 15:02:46 -0700</creation_ts>
          <short_desc>Implement cap height in SimpleFontDataWin to enable initial-letter support for Windows</short_desc>
          <delta_ts>2014-09-22 11:35:37 -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>CSS</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</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>
          <dependson>136484</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Roger Fong">roger_fong</reporter>
          <assigned_to name="Brent Fulgham">bfulgham</assigned_to>
          <cc>bfulgham</cc>
    
    <cc>commit-queue</cc>
    
    <cc>hyatt</cc>
    
    <cc>jonlee</cc>
    
    <cc>mmaxfield</cc>
    
    <cc>roger_fong</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1035322</commentid>
    <comment_count>0</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2014-09-15 15:02:46 -0700</bug_when>
    <thetext>In https://bugs.webkit.org/show_bug.cgi?id=136484 initial-letter support was enabled for Mac and iOS but not the other platforms.
In order to enable we have to implement the cap height logic in SimpleFontDataWin.

Until then, skipping all initial-letter tests</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1036257</commentid>
    <comment_count>1</comment_count>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2014-09-19 17:52:59 -0700</bug_when>
    <thetext>I&apos;m testing a patch for this. Easy fix.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1036566</commentid>
    <comment_count>2</comment_count>
      <attachid>238489</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2014-09-22 11:28:17 -0700</bug_when>
    <thetext>Created attachment 238489
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1036567</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2014-09-22 11:29:24 -0700</bug_when>
    <thetext>&lt;rdar://problem/18415046&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1036568</commentid>
    <comment_count>4</comment_count>
      <attachid>238489</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2014-09-22 11:32:47 -0700</bug_when>
    <thetext>Comment on attachment 238489
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1036569</commentid>
    <comment_count>5</comment_count>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2014-09-22 11:35:37 -0700</bug_when>
    <thetext>Committed r173840: &lt;http://trac.webkit.org/changeset/173840&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>238489</attachid>
            <date>2014-09-22 11:28:17 -0700</date>
            <delta_ts>2014-09-22 11:32:47 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-136841-20140922113029.patch</filename>
            <type>text/plain</type>
            <size>1988</size>
            <attacher name="Brent Fulgham">bfulgham</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE3Mzc4MykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEzIEBACisyMDE0LTA5LTE5ICBCcmVudCBG
dWxnaGFtICA8YmZ1bGdoYW1AYXBwbGUuY29tPgorCisgICAgICAgIFtXaW5dIEltcGxlbWVudCBj
YXAgaGVpZ2h0IGluIFNpbXBsZUZvbnREYXRhV2luIHRvIGVuYWJsZSBpbml0aWFsLWxldHRlciBz
dXBwb3J0IGZvciBXaW5kb3dzLgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MTM2ODQxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy93aW4vU2ltcGxlRm9udERhdGFDR1dpbi5j
cHA6CisgICAgICAgIChXZWJDb3JlOjpTaW1wbGVGb250RGF0YTo6cGxhdGZvcm1Jbml0KTogQWRk
IG1pc3NpbmcgY2FsbHMgdG8gc2V0IHVwIG91ciBmb250cy4KKwogMjAxNC0wOS0xOSAgQ2hyaXMg
RHVtZXogIDxjZHVtZXpAYXBwbGUuY29tPgogCiAgICAgICAgIE1pbmltaXplIHZpcnR1YWwgZnVu
Y3Rpb24gY2FsbHMgaW4gTWFya3VwQWNjdW11bGF0b3IKSW5kZXg6IFNvdXJjZS9XZWJDb3JlL3Bs
YXRmb3JtL2dyYXBoaWNzL3dpbi9TaW1wbGVGb250RGF0YUNHV2luLmNwcAo9PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t
LSBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy93aW4vU2ltcGxlRm9udERhdGFDR1dp
bi5jcHAJKHJldmlzaW9uIDE3Mzc3NCkKKysrIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBo
aWNzL3dpbi9TaW1wbGVGb250RGF0YUNHV2luLmNwcAkod29ya2luZyBjb3B5KQpAQCAtNjIsMTAg
KzYyLDEyIEBAIHZvaWQgU2ltcGxlRm9udERhdGE6OnBsYXRmb3JtSW5pdCgpCiAgICAgaW50IGlB
c2NlbnQgPSBDR0ZvbnRHZXRBc2NlbnQoZm9udCk7CiAgICAgaW50IGlEZXNjZW50ID0gQ0dGb250
R2V0RGVzY2VudChmb250KTsKICAgICBpbnQgaUxpbmVHYXAgPSBDR0ZvbnRHZXRMZWFkaW5nKGZv
bnQpOworICAgIGludCBpQ2FwSGVpZ2h0ID0gQ0dGb250R2V0Q2FwSGVpZ2h0KGZvbnQpOwogICAg
IHVuc2lnbmVkIHVuaXRzUGVyRW0gPSBDR0ZvbnRHZXRVbml0c1BlckVtKGZvbnQpOwogICAgIGZs
b2F0IHBvaW50U2l6ZSA9IG1fcGxhdGZvcm1EYXRhLnNpemUoKTsKICAgICBmbG9hdCBmQXNjZW50
ID0gc2NhbGVFbVRvVW5pdHMoaUFzY2VudCwgdW5pdHNQZXJFbSkgKiBwb2ludFNpemU7CiAgICAg
ZmxvYXQgZkRlc2NlbnQgPSAtc2NhbGVFbVRvVW5pdHMoaURlc2NlbnQsIHVuaXRzUGVyRW0pICog
cG9pbnRTaXplOworICAgIGZsb2F0IGZDYXBIZWlnaHQgPSBzY2FsZUVtVG9Vbml0cyhpQ2FwSGVp
Z2h0LCB1bml0c1BlckVtKSAqIHBvaW50U2l6ZTsKICAgICBmbG9hdCBmTGluZUdhcCA9IHNjYWxl
RW1Ub1VuaXRzKGlMaW5lR2FwLCB1bml0c1BlckVtKSAqIHBvaW50U2l6ZTsKIAogICAgIGlmICgh
aXNDdXN0b21Gb250KCkpIHsKQEAgLTgyLDYgKzg0LDcgQEAgdm9pZCBTaW1wbGVGb250RGF0YTo6
cGxhdGZvcm1Jbml0KCkKIAogICAgIG1fZm9udE1ldHJpY3Muc2V0QXNjZW50KGZBc2NlbnQpOwog
ICAgIG1fZm9udE1ldHJpY3Muc2V0RGVzY2VudChmRGVzY2VudCk7CisgICAgbV9mb250TWV0cmlj
cy5zZXRDYXBIZWlnaHQoZkNhcEhlaWdodCk7CiAgICAgbV9mb250TWV0cmljcy5zZXRMaW5lR2Fw
KGZMaW5lR2FwKTsKICAgICBtX2ZvbnRNZXRyaWNzLnNldExpbmVTcGFjaW5nKGxyb3VuZGYoZkFz
Y2VudCkgKyBscm91bmRmKGZEZXNjZW50KSArIGxyb3VuZGYoZkxpbmVHYXApKTsKIAo=
</data>
<flag name="review"
          id="263228"
          type_id="1"
          status="+"
          setter="hyatt"
    />
          </attachment>
      

    </bug>

</bugzilla>