diff options
author | Matt Brubeck <mbrubeck@limpet.net> | 2017-10-20 08:25:35 -0700 |
---|---|---|
committer | Matt Brubeck <mbrubeck@limpet.net> | 2017-10-20 08:25:35 -0700 |
commit | 2d45e9d2da571e70deef137f9022de87cc1126f3 (patch) | |
tree | d4f91108a958e80e06110c9437fea132785e8d85 /components/script/dom/window.rs | |
parent | fe16c1d5c3c9084da0ccb85af599d6ec0f8ab20b (diff) | |
download | servo-2d45e9d2da571e70deef137f9022de87cc1126f3.tar.gz servo-2d45e9d2da571e70deef137f9022de87cc1126f3.zip |
Use try syntax for Option where appropriate
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 422abd7079e..58d463cc2a3 100644 --- a/components/script/dom/window.rs +++ b/components/script/dom/window.rs @@ -592,15 +592,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(); @@ -688,10 +684,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)); @@ -703,10 +697,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())) } |