<?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>85631</bug_id>
          
          <creation_ts>2012-05-04 09:09:28 -0700</creation_ts>
          <short_desc>[Qt] REGRESSION: 350 tests started to fail with newer Qt5-WK1</short_desc>
          <delta_ts>2012-10-09 01:28:03 -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>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</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>Qt, QtTriaged</keywords>
          <priority>P1</priority>
          <bug_severity>Critical</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>79666</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Csaba Osztrogonác">ossy</reporter>
          <assigned_to name="Pierre Rossi">pierre.rossi</assigned_to>
          <cc>hausmann</cc>
    
    <cc>ossy</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>616276</commentid>
    <comment_count>0</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-05-04 09:09:28 -0700</bug_when>
    <thetext>After this update 360 tests started to fail with Qt5-WK1:
https://lists.webkit.org/pipermail/webkit-qt/2012-May/002715.html

The root of the problem is something near font handling.

a159ca80d73f07d617d6edd9b3e59bda25449a1c in qtbase made almost 
all tests (5000+) fail on Qt5-WK1. (But pass on Qt4-WK1 and Qt5-WK2)

Pierre&apos;s following patch in testfonts fixed the problem more or less,
but after this fix there are still 360 failing tests with Qt-WK1.

commit f552eb4e2ef2773f03a63075683a360b5095c667
Author: Pierre Rossi &lt;pierre.rossi@nokia.com&gt;
Date:   Thu May 3 13:39:56 2012 +0200

    Add one more aliasing rule to not mess with the Qt4 baseline.

    The approach consisting in hard coding font families in
    QFont::defaultFont resulted in interesting results.
    While the default font was always intended to be a serif font,
    it returned &quot;Times&quot; which somehow resolved to &quot;Nimbus Sans L&quot;.
    Since QFont::defaultFont now returns &quot;Nimbus Roman No9 L&quot;
    early on, we need this additional alias so that it later on gets
    the same test font.

diff --git a/fonts-qt5-wk1.conf b/fonts-qt5-wk1.conf
index 07e7f18..332bcdd 100644
--- a/fonts-qt5-wk1.conf
+++ b/fonts-qt5-wk1.conf
@@ -7,6 +7,15 @@
 --&gt;
        &lt;match target=&quot;pattern&quot;&gt;
                &lt;test qual=&quot;any&quot; name=&quot;family&quot;&gt;
+                       &lt;string&gt;Nimbus Roman No9 L&lt;/string&gt;
+               &lt;/test&gt;
+               &lt;edit name=&quot;family&quot; mode=&quot;assign&quot;&gt;
+                       &lt;string&gt;Times&lt;/string&gt;
+               &lt;/edit&gt;
+       &lt;/match&gt;
+
+       &lt;match target=&quot;pattern&quot;&gt;
+               &lt;test qual=&quot;any&quot; name=&quot;family&quot;&gt;
                        &lt;string&gt;serif&lt;/string&gt;
                &lt;/test&gt;
                &lt;edit name=&quot;family&quot; mode=&quot;assign&quot;&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>616304</commentid>
    <comment_count>1</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-05-04 09:33:55 -0700</bug_when>
    <thetext>I skipped them - http://trac.webkit.org/changeset/116109</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>737528</commentid>
    <comment_count>2</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-10-09 01:28:03 -0700</bug_when>
    <thetext>The reamining tests needed only rebase - r130735</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>