diff options
author | Anthony Ramine <nox@nox.paris> | 2020-03-31 19:06:25 +0200 |
---|---|---|
committer | Anthony Ramine <nox@nox.paris> | 2020-03-31 19:06:25 +0200 |
commit | a913c6650da2c402863db4e6bf3185a5810d920b (patch) | |
tree | d519faff22144391891d94c1eb612d92c4a5a4c2 /components/script/dom/htmlcanvaselement.rs | |
parent | 5ff931d1717ac0e7a6e5d9d63f0cfa47c1f3cd50 (diff) | |
download | servo-a913c6650da2c402863db4e6bf3185a5810d920b.tar.gz servo-a913c6650da2c402863db4e6bf3185a5810d920b.zip |
Reduce scope of unsafe block in LayoutHTMLCanvasElementHelpers::data
Diffstat (limited to 'components/script/dom/htmlcanvaselement.rs')
-rw-r--r-- | components/script/dom/htmlcanvaselement.rs | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/components/script/dom/htmlcanvaselement.rs b/components/script/dom/htmlcanvaselement.rs index c2958751952..5864321cb13 100644 --- a/components/script/dom/htmlcanvaselement.rs +++ b/components/script/dom/htmlcanvaselement.rs @@ -123,8 +123,8 @@ pub trait LayoutHTMLCanvasElementHelpers { impl LayoutHTMLCanvasElementHelpers for LayoutDom<'_, HTMLCanvasElement> { #[allow(unsafe_code)] fn data(self) -> HTMLCanvasData { - unsafe { - let source = match self.unsafe_get().context.borrow_for_layout().as_ref() { + let source = unsafe { + match self.unsafe_get().context.borrow_for_layout().as_ref() { Some(&CanvasContext::Context2d(ref context)) => { HTMLCanvasDataSource::Image(Some(context.to_layout().get_ipc_renderer())) }, @@ -135,20 +135,20 @@ impl LayoutHTMLCanvasElementHelpers for LayoutDom<'_, HTMLCanvasElement> { context.to_layout().canvas_data_source() }, None => HTMLCanvasDataSource::Image(None), - }; - - let width_attr = self - .upcast::<Element>() - .get_attr_for_layout(&ns!(), &local_name!("width")); - let height_attr = self - .upcast::<Element>() - .get_attr_for_layout(&ns!(), &local_name!("height")); - HTMLCanvasData { - source: source, - width: width_attr.map_or(DEFAULT_WIDTH, |val| val.as_uint()), - height: height_attr.map_or(DEFAULT_HEIGHT, |val| val.as_uint()), - canvas_id: self.get_canvas_id_for_layout(), } + }; + + let width_attr = self + .upcast::<Element>() + .get_attr_for_layout(&ns!(), &local_name!("width")); + let height_attr = self + .upcast::<Element>() + .get_attr_for_layout(&ns!(), &local_name!("height")); + HTMLCanvasData { + source: source, + width: width_attr.map_or(DEFAULT_WIDTH, |val| val.as_uint()), + height: height_attr.map_or(DEFAULT_HEIGHT, |val| val.as_uint()), + canvas_id: self.get_canvas_id_for_layout(), } } |