Throw in window.open() when an invalid URL is being passed

third_party/WebKit/Source/core/frame/Window.idl

Index: third_party/WebKit/Source/core/frame/Window.idl
diff --git a/third_party/WebKit/Source/core/frame/Window.idl b/third_party/WebKit/Source/core/frame/Window.idl
index 2bbfaa865627a17625dfbc751ada0eb494480289..015c9b4c6371d9cc2bc9b34fe55b0e6c0eb5b9e3 100644
--- a/third_party/WebKit/Source/core/frame/Window.idl
+++ b/third_party/WebKit/Source/core/frame/Window.idl
@@ -63,7 +63,7 @@
     [Replaceable, CrossOrigin] readonly attribute Window? parent;
     [CheckSecurity=ReturnValue, Custom=Getter] readonly attribute Element? frameElement;
     // FIXME: open() should have 4 optional arguments with defaults.
-    [Custom] Window? open(DOMString url, DOMString target, optional DOMString features);
+    [Custom, RaisesException] Window? open(DOMString url, DOMString target, optional DOMString features);

[foolip, 2017/05/05 11:21:29]

FYI, I wondered if returning null is still possible, and it is: "If
tokenizedFeatures contains an entry with the key "noopener", then disown target
browsing context's opener and return null."

[jochen (gone - plz use gerrit), 2017/05/05 11:25:51]

correct, that should return null. Also note that the PR against html didn't
change the idl

[foolip, 2017/05/05 11:28:55]

Yeah, just verifying since sometimes the exception is the only thing missing to
never return null.

 
     // indexed properties
     // https://html.spec.whatwg.org/C/browsers.html#windowproxy-getownproperty