<?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>94792</bug_id>
          
          <creation_ts>2012-08-23 01:53:51 -0700</creation_ts>
          <short_desc>Enable sub-pixel layout for the GTK port</short_desc>
          <delta_ts>2013-04-27 05:39:16 -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>WebKitGTK</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>Gtk, LayoutTestFailure</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>89572</dependson>
    
    <dependson>115304</dependson>
          <blocked>85532</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Zan Dobersek">zan</reporter>
          <assigned_to name="Zan Dobersek">zan</assigned_to>
          <cc>d-r</cc>
    
    <cc>eae</cc>
    
    <cc>leviw</cc>
    
    <cc>mrobinson</cc>
    
    <cc>pnormand</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>702814</commentid>
    <comment_count>0</comment_count>
    <who name="Zan Dobersek">zan</who>
    <bug_when>2012-08-23 01:53:51 -0700</bug_when>
    <thetext>This bug will cover enabling the sub-pixel layout on the GTK port.

I believe the branch for the next stable release series will be created soon - I&apos;d recommend proceeding with enabling this feature only after that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739936</commentid>
    <comment_count>1</comment_count>
    <who name="Dominik Röttsches (drott)">d-r</who>
    <bug_when>2012-10-11 07:10:24 -0700</bug_when>
    <thetext>So you consider bug 89572 required for enabling sub pixel layout on GTK as well? Do you see the same issue that I observed initially that there&apos;s too much rebaselining required on GTK without that patch?
Maybe together we can make another attempt and working with the Chromium gardeners to get it landed. Last time I tried I couldn&apos;t quite get them to rebase these 6000+ cases - but if we&apos;re two (ports), we might have more luck.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739938</commentid>
    <comment_count>2</comment_count>
    <who name="Zan Dobersek">zan</who>
    <bug_when>2012-10-11 07:14:12 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; So you consider bug 89572 required for enabling sub pixel layout on GTK as well? Do you see the same issue that I observed initially that there&apos;s too much rebaselining required on GTK without that patch?
&gt; Maybe together we can make another attempt and working with the Chromium gardeners to get it landed. Last time I tried I couldn&apos;t quite get them to rebase these 6000+ cases - but if we&apos;re two (ports), we might have more luck.

I haven&apos;t really checked out how this patch affects the number of tests that would need rebaselining, but it wouldn&apos;t come around as smart to enable the subpixel layout, rebaseline gazillion of tests, get that patch landed and rebaseline gazillion of tests again. I&apos;d rather help out the Chromium gardeners do their rebaselining when this patch lands.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>814152</commentid>
    <comment_count>3</comment_count>
    <who name="Dominik Röttsches (drott)">d-r</who>
    <bug_when>2013-01-23 06:19:57 -0800</bug_when>
    <thetext>As you&apos;ve probably seen on some other bugs, we&apos;ve enabled it on EFL, and bug 89572 wasn&apos;t actually the core of the issue. Updating FreeType was. And no rebaselines required on the Chromium side.

However, after my discussions with Werner Lemberg, there will be another change in FreeType that might affect baselines - so, if you plan to enable sub-pixel layout on GTK, I would wait for FreeType 2.4.12 - or in other words any release that includes
http://git.savannah.gnu.org/cgit/freetype/freetype2.git/commit/?id=e0469372be3870a5ad60b2c4586e9c281357bd28</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>876062</commentid>
    <comment_count>4</comment_count>
      <attachid>198186</attachid>
    <who name="Zan Dobersek">zan</who>
    <bug_when>2013-04-15 14:20:14 -0700</bug_when>
    <thetext>Created attachment 198186
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>878784</commentid>
    <comment_count>5</comment_count>
      <attachid>198186</attachid>
    <who name="Zan Dobersek">zan</who>
    <bug_when>2013-04-20 12:20:34 -0700</bug_when>
    <thetext>Comment on attachment 198186
Patch

Clearing flags on attachment: 198186

Committed r148793: &lt;http://trac.webkit.org/changeset/148793&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>878785</commentid>
    <comment_count>6</comment_count>
    <who name="Zan Dobersek">zan</who>
    <bug_when>2013-04-20 12:20:40 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>198186</attachid>
            <date>2013-04-15 14:20:14 -0700</date>
            <delta_ts>2013-04-20 12:20:34 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-94792-20130415231949.patch</filename>
            <type>text/plain</type>
            <size>1062</size>
            <attacher name="Zan Dobersek">zan</attacher>
            
              <data encoding="base64">SW5kZXg6IENoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDE0ODQ2
NykKKysrIENoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEyIEBACisyMDEzLTA0
LTE1ICBaYW4gRG9iZXJzZWsgIDx6ZG9iZXJzZWtAaWdhbGlhLmNvbT4KKworICAgICAgICBFbmFi
bGUgc3ViLXBpeGVsIGxheW91dCBmb3IgdGhlIEdUSyBwb3J0CisgICAgICAgIGh0dHBzOi8vYnVn
cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD05NDc5MgorCisgICAgICAgIFJldmlld2VkIGJ5
IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogU291cmNlL2F1dG90b29scy9TZXR1cFdlYktp
dEZlYXR1cmVzLm00OiBFbmFibGUgdGhlIHN1YnBpeGVsIGxheW91dC4KKwogMjAxMy0wNC0xNSAg
UGF0cmljayBHYW5zdGVyZXIgIDxwYXJvZ2FAd2Via2l0Lm9yZz4KIAogICAgICAgICBbQ01ha2Vd
IEFkZCBXVEZfVVNFXypfVU5JQ09ERSB2YXJpYWJsZXMKSW5kZXg6IFNvdXJjZS9hdXRvdG9vbHMv
U2V0dXBXZWJLaXRGZWF0dXJlcy5tNAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvYXV0b3Rvb2xzL1Nl
dHVwV2ViS2l0RmVhdHVyZXMubTQJKHJldmlzaW9uIDE0ODQ2NikKKysrIFNvdXJjZS9hdXRvdG9v
bHMvU2V0dXBXZWJLaXRGZWF0dXJlcy5tNAkod29ya2luZyBjb3B5KQpAQCAtMTY4LDYgKzE2OCw3
IEBAICRzcmNkaXIvVG9vbHMvZ3RrL2dlbmVyYXRlLWZlYXR1cmUtZGVmaW4KICAgICBFTkFCTEVf
U0hBUkVEX1dPUktFUlM9MSBcCiAgICAgRU5BQkxFX1NNT09USF9TQ1JPTExJTkc9MSBcCiAgICAg
RU5BQkxFX1NRTF9EQVRBQkFTRT0xIFwKKyAgICBFTkFCTEVfU1VCUElYRUxfTEFZT1VUPTEgXAog
ICAgIEVOQUJMRV9TVFlMRV9TQ09QRUQ9MCBcCiAgICAgRU5BQkxFX1RFTVBMQVRFX0VMRU1FTlQ9
MCBcCiAgICAgRU5BQkxFX1RFWFRfQVVUT1NJWklORz0wIFwK
</data>

          </attachment>
      

    </bug>

</bugzilla>