<?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>59334</bug_id>
          
          <creation_ts>2011-04-25 10:44:26 -0700</creation_ts>
          <short_desc>[Qt] New tests introduced in r84742 fail</short_desc>
          <delta_ts>2013-06-14 08:00:48 -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>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>59241</blocked>
    
    <blocked>87008</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Csaba Osztrogonác">ossy</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>allan.jensen</cc>
    
    <cc>ap</cc>
    
    <cc>cmarcelo</cc>
    
    <cc>kadam</cc>
    
    <cc>ossy</cc>
    
    <cc>robert</cc>
    
    <cc>zarvai</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>391827</commentid>
    <comment_count>0</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2011-04-25 10:44:26 -0700</bug_when>
    <thetext>http/tests/misc/will-send-request-returns-null-on-redirect.html
http/tests/security/XFrameOptions/x-frame-options-deny.html
http/tests/security/XFrameOptions/x-frame-options-parent-same-origin-deny.html

http://build.webkit.org/results/Qt%20Linux%20Release/r84743%20%2831878%29/results.html

Comment #14 From Alexey Proskuryakov 2011-04-23 20:21:44 PST (-) [reply]
Fixing Qt results seems to require some improvement of DRT result printing - not sure what the strategy is, but it seems that some Mac-specific messages are being already synthesized.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>391830</commentid>
    <comment_count>1</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2011-04-25 10:48:14 -0700</bug_when>
    <thetext>Skipped by http://trac.webkit.org/changeset/84786</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>418201</commentid>
    <comment_count>2</comment_count>
    <who name="Robert Hogan">robert</who>
    <bug_when>2011-06-09 12:53:34 -0700</bug_when>
    <thetext>(In reply to comment #0)
&gt; http/tests/security/XFrameOptions/x-frame-options-deny.html
&gt; http/tests/security/XFrameOptions/x-frame-options-parent-same-origin-deny.html

These two are benign, we have platform-specific results for the return codes in the same family of tests.

&gt; http/tests/misc/will-send-request-returns-null-on-redirect.html

This one happens because the Qt API considers a load finished (and DRT dumps it results) when postProgressFinishedNotification() is fired from ProgressTracker::finalProgressComplete() for the main frame. By the time dispatchDidFailLoading() has been called for the iframe, the DRT has dumped.

I think other tests cope with this by putting a notifyDone() into the load event handler of any subframes. That is probably the fix here, or just accept Qt&apos;s platform specific results.

Alexey, does that sound sensible to you or do you suspect Qt is doing something wrong here?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>832892</commentid>
    <comment_count>3</comment_count>
    <who name="Ádám Kallai">kadam</who>
    <bug_when>2013-02-14 02:39:28 -0800</bug_when>
    <thetext>I&apos;m reviewing qt test expectations. What do you think about this test? Should I update it or not? It seems this test result is correct with Wk2. 
Thanks for your help!

 * http/tests/misc/will-send-request-returns-null-on-redirect.html


--- /home/kadam/webkit/WebKit/WebKitBuild/Release/layout-test-results/http/tests/misc/will-send-request-returns-null-on-redirect-expected.txt
+++ /home/kadam/webkit/WebKit/WebKitBuild/Release/layout-test-results/http/tests/misc/will-send-request-returns-null-on-redirect-actual.txt
@@ -2,7 +2,6 @@
 &lt;unknown&gt; - didFinishLoading
 http://127.0.0.1:8000/misc/resources/redirect-to-http-url.php - willSendRequest &lt;NSURLRequest URL http://www.example.com/, main document URL http://127.0.0.1:8000/misc/will-send-request-returns-null-on-redirect.html, http method GET&gt; redirectResponse &lt;NSURLResponse http://127.0.0.1:8000/misc/resources/redirect-to-http-url.php, http status code 302&gt;
 Returning null for this redirect
-http://127.0.0.1:8000/misc/resources/redirect-to-http-url.php - didFailLoadingWithError: &lt;NSError domain NSURLErrorDomain, code -999&gt;
 https://bugs.webkit.org/show_bug.cgi?id=27595
 This test checks to make sure that things behave as expected when the resource load delegate returns null in response to willSendRequest.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>