<?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>55928</bug_id>
          
          <creation_ts>2011-03-08 00:55:46 -0800</creation_ts>
          <short_desc>Add Ajax tests to jQuery test suite in layout tests</short_desc>
          <delta_ts>2011-03-08 00:59:16 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>OS X 10.6</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Mathias Bynens">mathias</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>mathias</cc>
    
    <cc>paulirish</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>363881</commentid>
    <comment_count>0</comment_count>
    <who name="Mathias Bynens">mathias</who>
    <bug_when>2011-03-08 00:55:46 -0800</bug_when>
    <thetext>This is a follow-up to bug 55911. The jQuery test suite has been added to WebKit’s layout tests, but it seems the Ajax tests are missing. Apparently this was intentional, because they don’t seem to run locally — but it sounds like there’s a workaround. IRC log:

[09:40] &lt;matjas&gt; abarth: ping
[09:40] &lt;abarth&gt; hi
[09:41] &lt;abarth&gt; what can i do for you?
[09:41] &lt;matjas&gt; about the jQuery test case getting included in the layout tests (http://trac.webkit.org/changeset/80511)
[09:41] &lt;abarth&gt; yes
[09:42] &lt;matjas&gt; well, paul_irish noticed that it’s missing an ajax.html and ajax-expected.txt
[09:42] &lt;matjas&gt; abarth: is this one of the necessary changes dglazkov mentioned?
[09:42] &lt;abarth&gt; yeah, they don&apos;t seem to run &quot;locally&quot;
[09:42] &lt;matjas&gt; abarth: i see
[09:42] &lt;paul_irish&gt; locally like file:// scheme?
[09:43] &lt;abarth&gt; http://trac.webkit.org/browser/trunk/LayoutTests/jquery/resources/test/unit/ajax.js#L8
[09:43] &lt;abarth&gt; i didn&apos;t look into why isLocal is true
[09:43] &lt;abarth&gt; but it probably keys off the file scheme
[09:44] &lt;paul_irish&gt; yup.
[09:44] &lt;abarth&gt; we&apos;d certainly like to run those tests
[09:44] &lt;abarth&gt; we can run them as http tests
[09:44] &lt;abarth&gt; they&apos;ll just be a bit slower
[09:45] &lt;abarth&gt; data/errorWithText.php
[09:45] &lt;abarth&gt; looks like they use PHP
[09:45] &lt;abarth&gt; we have a PHP interpretor in the http tets
[09:45] &lt;abarth&gt; tests
[09:45] &lt;abarth&gt; so that should be fine
[09:52] &lt;abarth&gt; matjas: the only change I made to the test suite proper was to add a postMessage to top when the tests were over (and to add an XSS hole to let me select which of the test groups to run)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>363884</commentid>
    <comment_count>1</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-03-08 00:59:16 -0800</bug_when>
    <thetext>Patches welcome.  :)

This actually should be pretty straightforward to fix.  We can try moving the whole test suite over to HTTP.  The only downside of that is that it might be slow.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>