diff options
Diffstat (limited to 'components/script/dom/window.rs')
-rw-r--r-- | components/script/dom/window.rs | 24 |
1 files changed, 8 insertions, 16 deletions
diff --git a/components/script/dom/window.rs b/components/script/dom/window.rs index d219d051416..1df5d6490c8 100644 --- a/components/script/dom/window.rs +++ b/components/script/dom/window.rs @@ -591,15 +591,11 @@ impl WindowMethods for Window { // https://html.spec.whatwg.org/multipage/#dom-frameelement fn GetFrameElement(&self) -> Option<DomRoot<Element>> { // Steps 1-3. - let window_proxy = match self.window_proxy.get() { - None => return None, - Some(window_proxy) => window_proxy, - }; + let window_proxy = self.window_proxy.get()?; + // Step 4-5. - let container = match window_proxy.frame_element() { - None => return None, - Some(container) => container, - }; + let container = window_proxy.frame_element()?; + // Step 6. let container_doc = document_from_node(container); let current_doc = GlobalScope::current().expect("No current global object").as_window().Document(); @@ -687,10 +683,8 @@ impl WindowMethods for Window { // https://html.spec.whatwg.org/multipage/#dom-parent fn GetParent(&self) -> Option<DomRoot<WindowProxy>> { // Steps 1-3. - let window_proxy = match self.undiscarded_window_proxy() { - Some(window_proxy) => window_proxy, - None => return None, - }; + let window_proxy = self.undiscarded_window_proxy()?; + // Step 4. if let Some(parent) = window_proxy.parent() { return Some(DomRoot::from_ref(parent)); @@ -702,10 +696,8 @@ impl WindowMethods for Window { // https://html.spec.whatwg.org/multipage/#dom-top fn GetTop(&self) -> Option<DomRoot<WindowProxy>> { // Steps 1-3. - let window_proxy = match self.undiscarded_window_proxy() { - Some(window_proxy) => window_proxy, - None => return None, - }; + let window_proxy = self.undiscarded_window_proxy()?; + // Steps 4-5. Some(DomRoot::from_ref(window_proxy.top())) } |