<?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>64352</bug_id>
          
          <creation_ts>2011-07-12 04:28:49 -0700</creation_ts>
          <short_desc>[Qt] REGRESSION(r90634): It made http/tests/misc/script-async.html fail with Qt 4.8.x</short_desc>
          <delta_ts>2011-07-13 05:13:24 -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>64169</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Csaba Osztrogonác">ossy</reporter>
          <assigned_to name="Benjamin Poulain">benjamin</assigned_to>
          <cc>benjamin</cc>
    
    <cc>kling</cc>
    
    <cc>markus</cc>
    
    <cc>ossy</cc>
    
    <cc>peter.hartmann</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>435566</commentid>
    <comment_count>0</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2011-07-12 04:28:49 -0700</bug_when>
    <thetext>--- /ramdisk/qt-linux-32-release-qt470/build/layout-test-results/http/tests/misc/script-async-expected.txt	2011-07-12 03:36:30.445722227 -0700
+++ /ramdisk/qt-linux-32-release-qt470/build/layout-test-results/http/tests/misc/script-async-actual.txt	2011-07-12 03:36:30.445722227 -0700
@@ -1,2 +1,2 @@
 This tests for proper execution order of scripts with the async attribute https://bugs.webkit.org/show_bug.cgi?id=20710.
-PASS
+FAIL: Expected one of &apos; async  external  inline  DOMContentLoaded  slowAsync &apos; || &apos; external  async  inline  DOMContentLoaded  slowAsync &apos; || &apos; external  inline  async  DOMContentLoaded  slowAsync &apos; || &apos; external  inline  DOMContentLoaded  async  slowAsync &apos;, Actual=&apos; external  inline  DOMContentLoaded  slowAsync  async &apos;

Marked as P1 critical, because it is a regression.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>435575</commentid>
    <comment_count>1</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2011-07-12 04:54:55 -0700</bug_when>
    <thetext>Peter, Markus, would one of you have time to investigate why http pipelining break the tests?

I am tempted to just revert r90634 since this demonstrate Qt&apos;s Http pipelining is not ready for prime time.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>435591</commentid>
    <comment_count>2</comment_count>
    <who name="Markus Goetz">markus</who>
    <bug_when>2011-07-12 05:32:34 -0700</bug_when>
    <thetext>@Benjamin: HTTP pipelining should not be enabled for Ajax stuff. Everything that blocks the socket (e.g. long polling, COMET, whatever) should not be pipelined.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>435595</commentid>
    <comment_count>3</comment_count>
    <who name="Markus Goetz">markus</who>
    <bug_when>2011-07-12 05:36:04 -0700</bug_when>
    <thetext>@Benjamin: See for example bug 50644</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>436286</commentid>
    <comment_count>4</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2011-07-13 00:07:29 -0700</bug_when>
    <thetext>I skip this test until you fix the bug: http://trac.webkit.org/changeset/90891</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>436364</commentid>
    <comment_count>5</comment_count>
      <attachid>100659</attachid>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2011-07-13 05:08:35 -0700</bug_when>
    <thetext>Created attachment 100659
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>436365</commentid>
    <comment_count>6</comment_count>
      <attachid>100659</attachid>
    <who name="Andreas Kling">kling</who>
    <bug_when>2011-07-13 05:09:23 -0700</bug_when>
    <thetext>Comment on attachment 100659
Patch

Arr plus.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>436366</commentid>
    <comment_count>7</comment_count>
      <attachid>100659</attachid>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2011-07-13 05:13:17 -0700</bug_when>
    <thetext>Comment on attachment 100659
Patch

Clearing flags on attachment: 100659

Committed r90906: &lt;http://trac.webkit.org/changeset/90906&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>436367</commentid>
    <comment_count>8</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2011-07-13 05:13:24 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>100659</attachid>
            <date>2011-07-13 05:08:35 -0700</date>
            <delta_ts>2011-07-13 05:13:17 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-64352-20110713141522.patch</filename>
            <type>text/plain</type>
            <size>1461</size>
            <attacher name="Benjamin Poulain">benjamin</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogOTA5MDQKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5n
ZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwppbmRleCA5MGQ4OWYxYjliYTVlNzhjZTIwY2U0
YzI3M2VkZDQ2YjA0OWE3NzcyLi44NDVjZGRiMTA5NmE2MTk3NDQ5M2I5M2NjZDc3ZDE5ZjQwOGUz
YTNkIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMv
Q2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTEtMDctMTMgIEJlbmphbWluIFBvdWxhaW4g
IDxiZW5qYW1pbkB3ZWJraXQub3JnPgorCisgICAgICAgIFtRdF0gUkVHUkVTU0lPTihyOTA2MzQp
OiBJdCBtYWRlIGh0dHAvdGVzdHMvbWlzYy9zY3JpcHQtYXN5bmMuaHRtbCBmYWlsIHdpdGggUXQg
NC44LngKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTY0
MzUyCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVW5z
a2lwIGh0dHAvdGVzdHMvbWlzYy9zY3JpcHQtYXN5bmMuaHRtbCBzaW5jZSB0aGUgb3JpZ2luYWwg
Y2hhbmdlIGhhcyBiZWVuIHJldmVydGVkLgorCisgICAgICAgICogcGxhdGZvcm0vcXQtNC44L1Nr
aXBwZWQ6CisKIDIwMTEtMDctMTMgIEtyaXN0w7NmIEtvc3p0ecOzICA8S29zenR5by5LcmlzdG9m
QHN0dWQudS1zemVnZWQuaHU+CiAKICAgICAgICAgW1F0XSBBZGQgZXhwZWN0ZWQgcmVzdWx0cyBm
b3IgdGhlIG5ldyBwYXNzaW5nIHRlc3RzIGFuZCB1bnNraXAgdGhlbS4KZGlmZiAtLWdpdCBhL0xh
eW91dFRlc3RzL3BsYXRmb3JtL3F0LTQuOC9Ta2lwcGVkIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0v
cXQtNC44L1NraXBwZWQKaW5kZXggMjM1N2I3ODBhZjljNTlkZWQxMTU2ZmZlNWQxZWM3NDk2YTlh
NWJhZC4uODk5MTM3N2EwZjZiOGQ3NGE2Y2E2YWVlODI0YTZjNjk5NDFjMjczZSAxMDA2NDQKLS0t
IGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQtNC44L1NraXBwZWQKKysrIGIvTGF5b3V0VGVzdHMv
cGxhdGZvcm0vcXQtNC44L1NraXBwZWQKQEAgLTcsNyArNywzIEBAIGh0dHAvdGVzdHMvc2VjdXJp
dHkveHNzLURFTklFRC14c2wtZG9jdW1lbnQueG1sCiAjIGh0dHA6Ly9idWdyZXBvcnRzLnF0Lm5v
a2lhLmNvbS9icm93c2UvUVRCVUctMTk1NTYKICMgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTYwMTc2CiBodHRwL3Rlc3RzL3htbGh0dHByZXF1ZXN0L3htbGh0dHByZXF1
ZXN0LW1pc3NpbmctZmlsZS1leGNlcHRpb24uaHRtbAotCi0jIFtRdF0gUkVHUkVTU0lPTihyOTA2
MzQpOiBJdCBtYWRlIGh0dHAvdGVzdHMvbWlzYy9zY3JpcHQtYXN5bmMuaHRtbCBmYWlsIHdpdGgg
UXQgNC44LngKLSMgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTY0MzUy
Ci1odHRwL3Rlc3RzL21pc2Mvc2NyaXB0LWFzeW5jLmh0bWwK
</data>

          </attachment>
      

    </bug>

</bugzilla>