- http://bugzilla.opendarwin.org/show_bug.cgi?id=8770
XMLHttpRequest should strip CR/LF characters from the URL
Test: fast/loader/url-strip-cr-lf-tab.html
* platform/KURL.cpp:
(appendEscapingBadChars): Strip CR, LF and TAB, as Firefox and IE do.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@14320
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2006-05-11 Alexey Proskuryakov <ap@nypop.com>
+
+ Reviewed by Darin.
+
+ - test for http://bugzilla.opendarwin.org/show_bug.cgi?id=8770
+ XMLHttpRequest should strip CR/LF characters from the URL
+
+ * fast/loader/url-strip-cr-lf-tab-expected.txt: Added.
+ * fast/loader/url-strip-cr-lf-tab.html: Added.
+
2006-05-11 Justin Garcia <justin.garcia@apple.com>
Reviewed by thatcher, levi
--- /dev/null
+Test for bug 8770: XMLHttpRequest should strip CR/LF characters from the URL (not just XMLHttpRequest, really, and TAB is also stripped).
+Success
+
+
--- /dev/null
+<body>
+Test for <a href="http://bugzilla.opendarwin.org/show_bug.cgi?id=8770">bug 8770</a>:
+XMLHttpRequest should strip CR/LF characters from the URL (not just XMLHttpRequest, really, and TAB is also stripped).
+
+<script>
+if (window.layoutTestController) {
+ layoutTestController.dumpAsText();
+ layoutTestController.waitUntilDone();
+}
+
+if (document.URL.indexOf('?') == -1) {
+
+ window.location.href = "url-strip-cr-lf-tab\x09\x0a\x0d.html?a=b\x09\x0a\x0d#\x09\x0a\x0dc";
+
+} else {
+
+ if (document.URL.substring(document.URL.indexOf('?')+1, document.URL.length) == "a=b#c")
+ document.write("<p>Success</p>");
+ else
+ document.write("<p>Failure</p>");
+
+ if (window.layoutTestController)
+ layoutTestController.notifyDone();
+}
+</script>
+</body>
+2006-05-11 Alexey Proskuryakov <ap@nypop.com>
+
+ Reviewed by Darin.
+
+ - http://bugzilla.opendarwin.org/show_bug.cgi?id=8770
+ XMLHttpRequest should strip CR/LF characters from the URL
+
+ Test: fast/loader/url-strip-cr-lf-tab.html
+
+ * platform/KURL.cpp:
+ (appendEscapingBadChars): Strip CR, LF and TAB, as Firefox and IE do.
+
2006-05-11 David Hyatt <hyatt@apple.com>
Remove the misspelling drawing code from Font and FontData. Implement it
Reviewed by Adele.
- - test for http://bugzilla.opendarwin.org/show_bug.cgi?id=8743
+ - http://bugzilla.opendarwin.org/show_bug.cgi?id=8743
REGRESSION: focus() on input field selects all text within it
(was: cannot select cities on British Rail reservation site)
if (isBadChar(c)) {
if (c == '%' || c == '?') {
*p++ = c;
- } else {
+ } else if (c != 0x09 && c != 0x0a && c != 0x0d) {
*p++ = '%';
*p++ = hexDigits[c >> 4];
*p++ = hexDigits[c & 0xF];