<?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>69104</bug_id>
          
          <creation_ts>2011-09-29 14:13:16 -0700</creation_ts>
          <short_desc>[Qt] Fix failing tests after migrating to refactored Qt5</short_desc>
          <delta_ts>2012-05-23 02:26:43 -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>
          <dependson>69105</dependson>
    
    <dependson>69129</dependson>
    
    <dependson>69132</dependson>
    
    <dependson>69134</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Csaba Osztrogonác">ossy</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abecsi</cc>
    
    <cc>kbalazs</cc>
    
    <cc>kling</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>loki</cc>
    
    <cc>ossy</cc>
    
    <cc>vestbo</cc>
    
    <cc>yael</cc>
    
    <cc>zeno</cc>
    
    <cc>zherczeg</cc>
    
    <cc>zoltan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>475670</commentid>
    <comment_count>0</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2011-09-29 14:13:16 -0700</bug_when>
    <thetext>After migrating there are 10605 failing test. (half of the test cases!!!)
You can find all fails here:
http://build.webkit.sed.hu/results/x86-32%20Linux%20Qt%20Release%20WebKit2/r96349%20%2812586%29.zip
Failing, but undocumented fails before migrating: http://build.webkit.sed.hu/results/x86-32%20Linux%20Qt%20Release%20WebKit2/r96328%20%2812574%29.zip

Now all failing tests are in the Skipped list to let developers go ahead.

But I think we should stop for a minute. Disabling half of the tests 
is a brutal regression, which we should fix as soon as possible.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>478441</commentid>
    <comment_count>1</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2011-10-05 09:48:06 -0700</bug_when>
    <thetext>Now we have to check the remainding ~6000 tests manually. :-/
I think most of them pass, but they need a Qt5 specific expected result ...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>478449</commentid>
    <comment_count>2</comment_count>
    <who name="Tor Arne Vestbø">vestbo</who>
    <bug_when>2011-10-05 09:54:40 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; Now we have to check the remainding ~6000 tests manually. :-/
&gt; I think most of them pass, but they need a Qt5 specific expected result ...

We can run them with something similar to the orwt --ignore-metrics, to verify that they at least are metrics-failures only.

Also, we should look at this in relation to switching to Qt 4.8, since some of those 6000 might overlap with 4.8, and 4.8 will be the new platform/qt baseline, and Qt 5 will be then new qt-wk2 baseline.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486799</commentid>
    <comment_count>3</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2011-10-19 08:08:42 -0700</bug_when>
    <thetext>These fails are not WK2 related, but Qt5 related, because they fail with Qt5-WK1 too. I moved them to the qt-5.0/Skipped list: http://trac.webkit.org/changeset/97860

Your idea is good, I ran an old-run-webkit-tests --ignore-metrics test on the new qt-5.0/Skipped list with WK1 and I got the following result:
5878 test cases (84%) succeeded
888 test cases (12%) had incorrect layout
176 test cases (2%) crashed</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>547963</commentid>
    <comment_count>4</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-02-02 10:20:56 -0800</bug_when>
    <thetext>Most of them were unskipped after http://trac.webkit.org/changeset/102776,
so we can close this bug.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>