<?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>89458</bug_id>
          
          <creation_ts>2012-06-19 04:23:58 -0700</creation_ts>
          <short_desc>[EFL] Enable SUBPIXEL_LAYOUT feature flag for EFL</short_desc>
          <delta_ts>2013-01-22 10:05:42 -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>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>WebExposed</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>89572</dependson>
    
    <dependson>90169</dependson>
    
    <dependson>102374</dependson>
    
    <dependson>103121</dependson>
    
    <dependson>103129</dependson>
    
    <dependson>106774</dependson>
    
    <dependson>107548</dependson>
          <blocked>85532</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Dominik Röttsches (drott)">d-r</reporter>
          <assigned_to name="Dominik Röttsches (drott)">d-r</assigned_to>
          <cc>eae</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>leviw</cc>
    
    <cc>lucas.de.marchi</cc>
    
    <cc>ostap73</cc>
    
    <cc>s.choi</cc>
    
    <cc>sw0524.lee</cc>
    
    <cc>tmpsantos</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>652319</commentid>
    <comment_count>0</comment_count>
    <who name="Dominik Röttsches (drott)">d-r</who>
    <bug_when>2012-06-19 04:23:58 -0700</bug_when>
    <thetext>We&apos;d like to follow suit with bug 85532, enabling sub-pixel layout for the EFL port.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>653382</commentid>
    <comment_count>1</comment_count>
    <who name="Dominik Röttsches (drott)">d-r</who>
    <bug_when>2012-06-20 06:10:40 -0700</bug_when>
    <thetext>First obstacle, in a number of cases
setLogicalTop(roundToInt(top + maxAscent - fontMetrics.ascent(baselineType)));
inside
InlineFlowBox::placeBoxesInBlockDirection
actually results in -1.

maxAscent is 840/60 = 14, while fontMetrics.ascent (and fontMetrics.floatAscent fwiw) return 15.

Will investigate - but if you have any ideas, Levi, Emil - let me know.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>653391</commentid>
    <comment_count>2</comment_count>
    <who name="Dominik Röttsches (drott)">d-r</who>
    <bug_when>2012-06-20 06:20:27 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; First obstacle, in a number of cases
&gt; setLogicalTop(roundToInt(top + maxAscent - fontMetrics.ascent(baselineType)));
&gt; inside
&gt; InlineFlowBox::placeBoxesInBlockDirection
&gt; actually results in -1.

To clarify: ...leading to hundreds of test cases failing to show expected results just by this -1 offset - they previously had the logialTop just at 0.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>811446</commentid>
    <comment_count>3</comment_count>
    <who name="Dominik Röttsches (drott)">d-r</who>
    <bug_when>2013-01-19 12:45:26 -0800</bug_when>
    <thetext>Freetype updated and feature enabled in bug 106774, new baselines completely uploaded and the bots are returning back to normal. Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>811486</commentid>
    <comment_count>4</comment_count>
    <who name="Dominik Röttsches (drott)">d-r</who>
    <bug_when>2013-01-19 15:43:43 -0800</bug_when>
    <thetext>Big thanks, Thiago, for helping with the rebaselines!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>813121</commentid>
    <comment_count>5</comment_count>
    <who name="Emil A Eklund">eae</who>
    <bug_when>2013-01-22 10:05:42 -0800</bug_when>
    <thetext>Awesome, thanks for making this happen Dominik!</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>