[webkit-reviews] review denied: [Bug 66275] "new SharedWorker()" should throw SYNTAX_ERR when an invalid argument is passed : [Attachment 104013] Patch

[bugzilla-daemon at webkit.org]

Adam Barth <abarth at webkit.org> has denied  review:
Bug 66275: "new SharedWorker()" should throw SYNTAX_ERR when an invalid
argument is passed
https://bugs.webkit.org/show_bug.cgi?id=66275

Attachment 104013: Patch
https://bugs.webkit.org/attachment.cgi?id=104013&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
This patch does not appear correct, see:

http://www.w3.org/TR/WebIDL/#es-DOMString

In the null-or-undefined case, we hit step 3, which gives use "null" or
"undefined", which is a valid relative URL.  Therefore, we shouldn't throw a
SYNTAX_ERROR in that case.