From 3616c033d63cef6695e765331a1536f744320275 Mon Sep 17 00:00:00 2001 From: Sangeun Kim Date: Fri, 16 Aug 2013 10:52:53 +0900 Subject: [PATCH] Use new_all_same with same value instead of new --- src/components/main/layout/box.rs | 26 ++++++-------------------- 1 file changed, 6 insertions(+), 20 deletions(-) diff --git a/src/components/main/layout/box.rs b/src/components/main/layout/box.rs index c7d683101294..bee12968afd1 100644 --- a/src/components/main/layout/box.rs +++ b/src/components/main/layout/box.rs @@ -628,10 +628,8 @@ impl RenderBox { // should have a real `SERVO_DEBUG` system. debug!("%?", { // Compute the text box bounds and draw a border surrounding them. - let debug_border = SideOffsets2D::new(Au::from_px(1), - Au::from_px(1), - Au::from_px(1), - Au::from_px(1)); + let debug_border = SideOffsets2D::new_all_same(Au::from_px(1)); + do list.with_mut_ref |list| { let border_display_item = ~BorderDisplayItem { base: BaseDisplayItem { @@ -639,10 +637,7 @@ impl RenderBox { extra: ExtraDisplayListData::new(*self), }, border: debug_border, - color: SideOffsets2D::new(rgb(0, 0, 200).to_gfx_color(), - rgb(0, 0, 200).to_gfx_color(), - rgb(0, 0, 200).to_gfx_color(), - rgb(0, 0, 200).to_gfx_color()) + color: SideOffsets2D::new_all_same(rgb(0, 0, 200).to_gfx_color()) }; list.append_item(BorderDisplayItemClass(border_display_item)) @@ -663,10 +658,7 @@ impl RenderBox { extra: ExtraDisplayListData::new(*self), }, border: debug_border, - color: SideOffsets2D::new(rgb(0, 200, 0).to_gfx_color(), - rgb(0, 200, 0).to_gfx_color(), - rgb(0, 200, 0).to_gfx_color(), - rgb(0, 200, 0).to_gfx_color()) + color: SideOffsets2D::new_all_same(rgb(0, 200, 0).to_gfx_color()) }; list.append_item(BorderDisplayItemClass(border_display_item)) @@ -683,10 +675,7 @@ impl RenderBox { // FIXME(pcwalton): This is a bit of an abuse of the logging infrastructure. We // should have a real `SERVO_DEBUG` system. debug!("%?", { - let debug_border = SideOffsets2D::new(Au::from_px(1), - Au::from_px(1), - Au::from_px(1), - Au::from_px(1)); + let debug_border = SideOffsets2D::new_all_same(Au::from_px(1)); do list.with_mut_ref |list| { let border_display_item = ~BorderDisplayItem { @@ -695,10 +684,7 @@ impl RenderBox { extra: ExtraDisplayListData::new(*self), }, border: debug_border, - color: SideOffsets2D::new(rgb(0, 0, 200).to_gfx_color(), - rgb(0, 0, 200).to_gfx_color(), - rgb(0, 0, 200).to_gfx_color(), - rgb(0, 0, 200).to_gfx_color()) + color: SideOffsets2D::new_all_same(rgb(0, 0, 200).to_gfx_color()) }; list.append_item(BorderDisplayItemClass(border_display_item))