[webkit-reviews] review denied: [Bug 15530] XMLHttpRequest should not support certain methods : [Attachment 17429] Patch + test case

[bugzilla-daemon at webkit.org]

Alexey Proskuryakov <ap at webkit.org> has denied Julien Chaffraix
<julien.chaffraix at gmail.com>'s request for review:
Bug 15530: XMLHttpRequest should not support certain methods
http://bugs.webkit.org/show_bug.cgi?id=15530

Attachment 17429: Patch + test case
http://bugs.webkit.org/attachment.cgi?id=17429&action=edit

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
+    layoutTestController.waitUntilDone();

This is not needed.

+    try {
+	 xhr.open(method, "resources/1251.html", true);
+	 xhr.send(null);

This test doesn't differentiate between open() and send() raising an exception,
I think it should.

It may be slightly better to use sync XHR in the test, to maqke it more
explicit that we don't wait for anything to finish.