Changeset 84770 in webkit


Ignore:
Timestamp:
Apr 25, 2011 2:59:58 AM (13 years ago)
Author:
mjs@apple.com
Message:

2011-04-25 Maciej Stachowiak <mjs@apple.com>

Reviewed by Cameron Zwarich.

Fix tests after URL parsing fix
https://bugs.webkit.org/show_bug.cgi?id=59317

  • fast/dom/HTMLAnchorElement/script-tests/set-href-attribute-host.js: Update expected behavior.
  • fast/dom/HTMLAnchorElement/script-tests/set-href-attribute-protocol.js: Update expected behavior.
  • fast/dom/HTMLAnchorElement/set-href-attribute-host-expected.txt: Update expectations.
  • fast/dom/HTMLAnchorElement/set-href-attribute-protocol-expected.txt: Update expectations.
  • fast/loader/url-parse-1-expected.txt: Update expectations.
  • http/tests/loading/cross-origin-XHR-willLoadRequest.html: Replace single slash in an http: URL with a double slash - the single slash seemed to be erroneous.
Location:
trunk/LayoutTests
Files:
7 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r84766 r84770  
     12011-04-25  Maciej Stachowiak  <mjs@apple.com>
     2
     3        Reviewed by Cameron Zwarich.
     4
     5        Fix tests after URL parsing fix
     6        https://bugs.webkit.org/show_bug.cgi?id=59317
     7
     8        * fast/dom/HTMLAnchorElement/script-tests/set-href-attribute-host.js: Update
     9        expected behavior.
     10        * fast/dom/HTMLAnchorElement/script-tests/set-href-attribute-protocol.js: Update expected
     11        behavior.
     12        * fast/dom/HTMLAnchorElement/set-href-attribute-host-expected.txt: Update expectations.
     13        * fast/dom/HTMLAnchorElement/set-href-attribute-protocol-expected.txt: Update expectations.
     14        * fast/loader/url-parse-1-expected.txt: Update expectations.
     15        * http/tests/loading/cross-origin-XHR-willLoadRequest.html: Replace single slash
     16        in an http: URL with a double slash - the single slash seemed to be erroneous.
     17
    1182011-04-24  Dirk Pranke  <dpranke@chromium.org>
    219
  • trunk/LayoutTests/fast/dom/HTMLAnchorElement/script-tests/set-href-attribute-host.js

    r50784 r84770  
    7878}
    7979
    80 // Firefox 3.5.2 add the missing '/' thus gets a different result than IE8 and Webkit.
     80// WebKit fails to strip the \r in the authority, and therefore treats the URL as invalid
     81// and gets a different result than Firefox or Chrome; we should probably strip it
    8182debug("Set host to a malformed URL");
    8283a.href = "https:/\rww.my@domain.com:8080/path/";
    8384a.host = "www.other!domain.com:15";
    84 shouldBe("a.href", "'https://www.other!domain.com:15/ww.my@domain.com:8080/path/'");
     85shouldBe("a.href", "'https:/\\rww.my@domain.com:8080/path/'");
    8586
    8687// IE8 throws an "Object Error" exception.
  • trunk/LayoutTests/fast/dom/HTMLAnchorElement/script-tests/set-href-attribute-protocol.js

    r50784 r84770  
    6161}
    6262
    63 // Firefox 3.5.2 tries to build a hierarchical URL.
     63// Firefox 4 adds three slashes, unlike Safari and Chrome
    6464debug("Set protocol to http on malformed URL");
    6565a.href = "foo:??bar";
    6666a.protocol = "http";
    67 shouldBe("a.href", "'http:??bar'");
     67shouldBe("a.href", "'http:/??bar'");
    6868
    6969// IE8 keeps the protocol if it is 'c:'.
  • trunk/LayoutTests/fast/dom/HTMLAnchorElement/set-href-attribute-host-expected.txt

    r50784 r84770  
    2727PASS a.href is 'https://www.otherdom/ain.com/path/'
    2828Set host to a malformed URL
    29 PASS a.href is 'https://www.other!domain.com:15/ww.my@domain.com:8080/path/'
     29PASS a.href is 'https:/\rww.my@domain.com:8080/path/'
    3030Set host that starts with ':'
    3131PASS a.href is 'https://domain.com:8080/path/'
  • trunk/LayoutTests/fast/dom/HTMLAnchorElement/set-href-attribute-protocol-expected.txt

    r50784 r84770  
    1919PASS a.href is 'https://www.mydomain.com/path/'
    2020Set protocol to http on malformed URL
    21 PASS a.href is 'http:??bar'
     21PASS a.href is 'http:/??bar'
    2222Set protocol to a URL which points to a local file
    2323PASS a.href is 'f-oo:path'
  • trunk/LayoutTests/fast/loader/url-parse-1-expected.txt

    r50395 r84770  
    2121file://localhost/       file:///                /
    2222file://localhost/test   file:///test            /test
    23 http:   http:           
     23http:   http:/          /
    2424http:/  http:/          /
    2525http:// http:/          /
  • trunk/LayoutTests/http/tests/loading/cross-origin-XHR-willLoadRequest.html

    r75768 r84770  
    11<script>
    22    layoutTestController.dumpAsText();
    3     layoutTestController.addURLToRedirect("http:/localhost:8000/loading/resources/bar.txt", "http://localhost:8000/loading/resources/foo.txt");
     3    layoutTestController.addURLToRedirect("http://localhost:8000/loading/resources/bar.txt", "http://localhost:8000/loading/resources/foo.txt");
    44
    55    var request = new XMLHttpRequest();
    6     request.open("GET", "http:/localhost:8000/loading/resources/bar.txt");
     6    request.open("GET", "http://localhost:8000/loading/resources/bar.txt");
    77    request.send();
    88</script>
Note: See TracChangeset for help on using the changeset viewer.