From 6d49d8151cf5893f14d4418dabf606c7daf2e618 Mon Sep 17 00:00:00 2001 From: Hiroyuki Ikezoe Date: Tue, 15 Aug 2017 12:34:44 +0900 Subject: [PATCH] Don't convert zero component value to 'auto' in clone__moz_image_region. Currently Gecko treats 'auto' component value as zero for animation. --- components/style/properties/gecko.mako.rs | 18 ++++-------------- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git a/components/style/properties/gecko.mako.rs b/components/style/properties/gecko.mako.rs index e7f9db30b9ed..92cc18ab3afd 100644 --- a/components/style/properties/gecko.mako.rs +++ b/components/style/properties/gecko.mako.rs @@ -4222,21 +4222,11 @@ fn static_assert() { return Either::Second(Auto); } - let get_clip_rect_component = |value: structs::nscoord| -> Option { - if value == 0 { - None - } else { - Some(Au(value)) - } - }; - Either::First(ClipRect { - top: get_clip_rect_component(self.gecko.mImageRegion.y), - right: get_clip_rect_component(self.gecko.mImageRegion.width).map( - |v| v + Au(self.gecko.mImageRegion.x)), - bottom: get_clip_rect_component(self.gecko.mImageRegion.height).map( - |v| v + Au(self.gecko.mImageRegion.y)), - left: get_clip_rect_component(self.gecko.mImageRegion.x), + top: Some(Au(self.gecko.mImageRegion.y)), + right: Some(Au(self.gecko.mImageRegion.width) + Au(self.gecko.mImageRegion.x)), + bottom: Some(Au(self.gecko.mImageRegion.height) + Au(self.gecko.mImageRegion.y)), + left: Some(Au(self.gecko.mImageRegion.x)), }) }