Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(155)

Unified Diff: LayoutTests/http/tests/xmlhttprequest/xmlhttprequest-open-method-allowed.html

Issue 347653003: XMLHttpRequest must build a request using the given method as-is for non-standard types (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: update review comments and rebase Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/http/tests/xmlhttprequest/xmlhttprequest-open-method-allowed.html
diff --git a/LayoutTests/http/tests/xmlhttprequest/xmlhttprequest-open-method-allowed.html b/LayoutTests/http/tests/xmlhttprequest/xmlhttprequest-open-method-allowed.html
new file mode 100644
index 0000000000000000000000000000000000000000..0a6f37b33bfd106b54dae96dcd5b0227c5875c28
--- /dev/null
+++ b/LayoutTests/http/tests/xmlhttprequest/xmlhttprequest-open-method-allowed.html
@@ -0,0 +1,36 @@
+<!DOCTYPE html>
+<html>
+ <head>
+ <title>XMLHttpRequest: open() - allowed case in/sensitive methods test</title>
+ <script src="../resources/testharness.js"></script>
+ <script src="../resources/testharnessreport.js"></script>
+ </head>
+ <body>
+ <p>Valid methods per step-5 of http://xhr.spec.whatwg.org/#the-open()-method.</p>
+ <script type="text/javascript">
+ function testMethod(methodName) {
+ var client = new XMLHttpRequest();
+ client.open(methodName, "resources/echo-request-method.php");
+ client.onreadystatechange = function(event) {
+ if (event.target.readyState == 4)
+ assert_equals(client.getResponseHeader("x-custom-request-method"), methodName.toUpperCase());
+ }
+ client.send(null);
+ }
+
+ test(function() { testMethod("PUT"); });
+ test(function() { testMethod("Put"); });
+ test(function() { testMethod("DELETE"); });
+ test(function() { testMethod("DeLeTe"); });
+ test(function() { testMethod("HEAD"); });
+ test(function() { testMethod("hEAd"); });
+ test(function() { testMethod("OPTIONS"); });
+ test(function() { testMethod("OPtiOns"); });
+ test(function() { testMethod("POST"); });
+ test(function() { testMethod("post"); });
+ test(function() { testMethod("GET"); });
+ test(function() { testMethod("gEt"); });
+ </script>
+ </body>
+</html>
+

Powered by Google App Engine
This is Rietveld 408576698