Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-06 14:10:11 +01:00
Code Issues Projects Releases Packages Wiki Activity

Auto merge of #24046 - Eijebong:fix-open-for-spec, r=jdm

Browse source 
Update Window::open and document::open parameters to match the spec

Fixes #24012

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/24046)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2019-08-26 16:19:53 -04:00 • committed by GitHub
commit 2d9e170157
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
7 changed files with 9 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

2
components/script/dom/document.rs
View file

@ -4453,7 +4453,7 @@ impl DocumentMethods for Document {
// https://html.spec.whatwg.org/multipage/#dom-document-open-window
fn Open_(
&self,
url: DOMString,
url: USVString,
target: DOMString,
features: DOMString,
) -> Fallible<DomRoot<WindowProxy>> {

2
components/script/dom/webidls/Document.webidl
View file

@ -122,7 +122,7 @@ partial /*sealed*/ interface Document {
[CEReactions, Throws]
Document open(optional DOMString unused1, optional DOMString unused2);
[CEReactions, Throws]
WindowProxy open(DOMString url, DOMString name, DOMString features);
WindowProxy open(USVString url, DOMString name, DOMString features);
[CEReactions, Throws]
void close();
[CEReactions, Throws]

2
components/script/dom/webidls/Window.webidl
View file

@ -40,7 +40,7 @@
// https://github.com/whatwg/html/issues/2115
[Replaceable] readonly attribute WindowProxy? parent;
readonly attribute Element? frameElement;
WindowProxy? open(optional DOMString url = "about:blank", optional DOMString target = "_blank",
WindowProxy? open(optional USVString url = "", optional DOMString target = "_blank",
optional DOMString features = "");
//getter WindowProxy (unsigned long index);

2
components/script/dom/window.rs
View file

@ -616,7 +616,7 @@ impl WindowMethods for Window {
// https://html.spec.whatwg.org/multipage/#dom-open
fn Open(
&self,
url: DOMString,
url: USVString,
target: DOMString,
features: DOMString,
) -> Option<DomRoot<WindowProxy>> {

4
components/script/dom/windowproxy.rs
View file

@ -10,7 +10,7 @@ use crate::dom::bindings::inheritance::Castable;
use crate::dom::bindings::proxyhandler::fill_property_descriptor;
use crate::dom::bindings::reflector::{DomObject, Reflector};
use crate::dom::bindings::root::{Dom, DomRoot};
use crate::dom::bindings::str::DOMString;
use crate::dom::bindings::str::{DOMString, USVString};
use crate::dom::bindings::trace::JSTraceable;
use crate::dom::bindings::utils::{get_array_index_from_id, AsVoidPtr, WindowProxyHandler};
use crate::dom::dissimilaroriginwindow::DissimilarOriginWindow;
@ -395,7 +395,7 @@ impl WindowProxy {
// https://html.spec.whatwg.org/multipage/#window-open-steps
pub fn open(
&self,
url: DOMString,
url: USVString,
target: DOMString,
features: DOMString,
) -> Option<DomRoot<WindowProxy>> {