Changeset 112204 in webkit


Ignore:
Timestamp:
Mar 26, 2012 9:25:16 PM (12 years ago)
Author:
commit-queue@webkit.org
Message:

Fix flaky test fast/eventsource/eventsource-url-attribute.html
​https://bugs.webkit.org/show_bug.cgi?id=82257

Patch by Pablo Flouret <​pablof@motorola.com> on 2012-03-26
Reviewed by Ryosuke Niwa.

The test used ​http://localhost, chosen as a semi-random url, but since
it doesn't return a text/event-stream mime-type sometimes a console
warning would appear. Moved the test to the http folder and changed it
to use a resource with the correct mime-type.

  • http/tests/eventsource/eventsource-url-attribute-expected.txt: Renamed from LayoutTests/fast/eventsource/eventsource-url-attribute-expected.txt.
  • http/tests/eventsource/eventsource-url-attribute.html: Renamed from LayoutTests/fast/eventsource/eventsource-url-attribute.html.
Location:
trunk/LayoutTests
Files:
1 edited
2 moved

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r112202 r112204  
     12012-03-26  Pablo Flouret  <pablof@motorola.com>
     2
     3        Fix flaky test fast/eventsource/eventsource-url-attribute.html
     4        https://bugs.webkit.org/show_bug.cgi?id=82257
     5
     6        Reviewed by Ryosuke Niwa.
     7
     8        The test used http://localhost, chosen as a semi-random url, but since
     9        it doesn't return a text/event-stream mime-type sometimes a console
     10        warning would appear. Moved the test to the http folder and changed it
     11        to use a resource with the correct mime-type.
     12
     13        * http/tests/eventsource/eventsource-url-attribute-expected.txt: Renamed from LayoutTests/fast/eventsource/eventsource-url-attribute-expected.txt.
     14        * http/tests/eventsource/eventsource-url-attribute.html: Renamed from LayoutTests/fast/eventsource/eventsource-url-attribute.html.
     15
    1162012-03-26  David Grogan  <dgrogan@chromium.org>
    217

  • trunk/LayoutTests/http/tests/eventsource/eventsource-url-attribute-expected.txt

    r112203 r112204  
    44
    55
    6 PASS source.URL is "http://localhost/"
    7 PASS source.url is "http://localhost/"
     6PASS source.URL is "http://127.0.0.1:8000/eventsource/resources/event-stream.php"
     7PASS source.url is "http://127.0.0.1:8000/eventsource/resources/event-stream.php"
    88PASS source.URL === source.url is true
    99PASS successfullyParsed is true

  • trunk/LayoutTests/http/tests/eventsource/eventsource-url-attribute.html

    r112203 r112204  
    33<head>
    44<meta charset="utf-8">
    5 <script src="../js/resources/js-test-pre.js"></script>
     5<script src="../../js-test-resources/js-test-pre.js"></script>
    66</head>
    77<body>
    … …  
    1010description("Both .URL and .url should work (for compatibility reasons).");
    1111
    12 var url = "http://localhost/";
     12var url = "http://127.0.0.1:8000/eventsource/resources/event-stream.php";
    1313var source = new EventSource(url);
    1414
    … …  
    1818
    1919</script>
    20 <script src="../js/resources/js-test-post.js"></script>
     20<script src="../../js-test-resources/js-test-post.js"></script>
    2121</body>
    2222</html>
Note: See TracChangeset for help on using the changeset viewer.