Move width and height from Box to Position.

This helps with style struct alignment between Servo and Gecko.

7 files changed, 25 insertions, 25 deletions

diff --git a/components/layout/block.rs b/components/layout/block.rs
index 987a3038e9b..e61a7e8c22f 100644
--- a/components/layout/block.rs
+++ b/components/layout/block.rs
@@ -1654,7 +1654,7 @@ impl Flow for BlockFlow {
     fn bubble_inline_sizes(&mut self) {
         // If this block has a fixed width, just use that for the minimum and preferred width,
         // rather than bubbling up children inline width.
-        let consult_children = match self.fragment.style().get_box().width {
+        let consult_children = match self.fragment.style().get_position().width {
             LengthOrPercentageOrAuto::Length(_) => false,
             _ => true,
         };
diff --git a/components/layout/flex.rs b/components/layout/flex.rs
index eed7ec8aae3..789018e4121 100644
--- a/components/layout/flex.rs
+++ b/components/layout/flex.rs
@@ -125,7 +125,7 @@ impl FlexFlow {
     // Currently, this is the core of BlockFlow::bubble_inline_sizes() with all float logic
     // stripped out, and max replaced with union_nonbreaking_inline.
     fn inline_mode_bubble_inline_sizes(&mut self) {
-        let fixed_width = match self.block_flow.fragment.style().get_box().width {
+        let fixed_width = match self.block_flow.fragment.style().get_position().width {
             LengthOrPercentageOrAuto::Length(_) => true,
             _ => false,
         };
@@ -151,7 +151,7 @@ impl FlexFlow {
     // Currently, this is the core of BlockFlow::bubble_inline_sizes() with all float logic
     // stripped out.
     fn block_mode_bubble_inline_sizes(&mut self) {
-        let fixed_width = match self.block_flow.fragment.style().get_box().width {
+        let fixed_width = match self.block_flow.fragment.style().get_position().width {
             LengthOrPercentageOrAuto::Length(_) => true,
             _ => false,
         };
@@ -398,9 +398,9 @@ impl Flow for FlexFlow {
         let (available_block_size, available_inline_size) = {
             let style = &self.block_flow.fragment.style;
             let (specified_block_size, specified_inline_size) = if style.writing_mode.is_vertical() {
-                (style.get_box().width, style.get_box().height)
+                (style.get_position().width, style.get_position().height)
             } else {
-                (style.get_box().height, style.get_box().width)
+                (style.get_position().height, style.get_position().width)
             };
 
             let available_inline_size = AxisSize::new(specified_inline_size,
diff --git a/components/layout/incremental.rs b/components/layout/incremental.rs
index a743afd1555..e8cf8e4c500 100644
--- a/components/layout/incremental.rs
+++ b/components/layout/incremental.rs
@@ -211,7 +211,7 @@ pub fn compute_damage(old: Option<&Arc<ServoComputedValues>>, new: &ServoCompute
         get_margin.margin_bottom, get_margin.margin_left,
         get_padding.padding_top, get_padding.padding_right,
         get_padding.padding_bottom, get_padding.padding_left,
-        get_box.width, get_box.height,
+        get_position.width, get_position.height,
         get_inheritedtext.line_height,
         get_inheritedtext.text_align, get_inheritedtext.text_indent,
         get_table.table_layout,
diff --git a/components/style/animation.rs b/components/style/animation.rs
index 9c0242aa971..08ae32baf0c 100644
--- a/components/style/animation.rs
+++ b/components/style/animation.rs
@@ -158,7 +158,7 @@ impl PropertyAnimation {
             [Color; get_color; color],
             [FontSize; get_font; font_size],
             [FontWeight; get_font; font_weight],
-            [Height; get_box; height],
+            [Height; get_position; height],
             [Left; get_position; left],
             [LineHeight; get_inheritedtext; line_height],
             [MarginBottom; get_margin; margin_bottom],
@@ -181,7 +181,7 @@ impl PropertyAnimation {
             [Top; get_position; top],
             [VerticalAlign; get_box; vertical_align],
             [Visibility; get_inheritedbox; visibility],
-            [Width; get_box; width],
+            [Width; get_position; width],
             [ZIndex; get_position; z_index]);
 
         let property_animation = PropertyAnimation {
@@ -256,7 +256,7 @@ impl PropertyAnimation {
             [Color; mutate_color; color],
             [FontSize; mutate_font; font_size],
             [FontWeight; mutate_font; font_weight],
-            [Height; mutate_box; height],
+            [Height; mutate_position; height],
             [Left; mutate_position; left],
             [LineHeight; mutate_inheritedtext; line_height],
             [MarginBottom; mutate_margin; margin_bottom],
@@ -281,7 +281,7 @@ impl PropertyAnimation {
             [Transform; mutate_effects; transform],
             [VerticalAlign; mutate_box; vertical_align],
             [Visibility; mutate_inheritedbox; visibility],
-            [Width; mutate_box; width],
+            [Width; mutate_position; width],
             [ZIndex; mutate_position; z_index]);
     }
 
diff --git a/components/style/properties/longhand/box.mako.rs b/components/style/properties/longhand/box.mako.rs
index f1e3cfb5be5..a88db8bf03e 100644
--- a/components/style/properties/longhand/box.mako.rs
+++ b/components/style/properties/longhand/box.mako.rs
@@ -122,16 +122,6 @@ ${helpers.single_keyword("clear", "none left right both", gecko_ffi_name="mBreak
 
 </%helpers:longhand>
 
-${helpers.predefined_type("width",
-                          "LengthOrPercentageOrAuto",
-                          "computed::LengthOrPercentageOrAuto::Auto",
-                          "parse_non_negative")}
-
-${helpers.predefined_type("height",
-                          "LengthOrPercentageOrAuto",
-                          "computed::LengthOrPercentageOrAuto::Auto",
-                          "parse_non_negative")}
-
 <%helpers:longhand name="vertical-align">
   use cssparser::ToCss;
   use std::fmt;
diff --git a/components/style/properties/longhand/position.mako.rs b/components/style/properties/longhand/position.mako.rs
index 41dc76ee3b6..c4ea596758d 100644
--- a/components/style/properties/longhand/position.mako.rs
+++ b/components/style/properties/longhand/position.mako.rs
@@ -96,6 +96,16 @@ ${helpers.predefined_type("flex-basis",
 
 ${helpers.single_keyword("flex-wrap", "nowrap wrap wrap-reverse", products="gecko")}
 
+${helpers.predefined_type("width",
+                          "LengthOrPercentageOrAuto",
+                          "computed::LengthOrPercentageOrAuto::Auto",
+                          "parse_non_negative")}
+
+${helpers.predefined_type("height",
+                          "LengthOrPercentageOrAuto",
+                          "computed::LengthOrPercentageOrAuto::Auto",
+                          "parse_non_negative")}
+
 ${helpers.predefined_type("min-width",
                           "LengthOrPercentage",
                           "computed::LengthOrPercentage::Length(Au(0))",
diff --git a/components/style/properties/properties.mako.rs b/components/style/properties/properties.mako.rs
index 6582d5a6020..0299b3a4276 100644
--- a/components/style/properties/properties.mako.rs
+++ b/components/style/properties/properties.mako.rs
@@ -972,18 +972,18 @@ impl ServoComputedValues {
 
     #[inline]
     pub fn content_inline_size(&self) -> computed::LengthOrPercentageOrAuto {
-        let box_style = self.get_box();
+        let position_style = self.get_position();
         if self.writing_mode.is_vertical() {
-            box_style.height
+            position_style.height
         } else {
-            box_style.width
+            position_style.width
         }
     }
 
     #[inline]
     pub fn content_block_size(&self) -> computed::LengthOrPercentageOrAuto {
-        let box_style = self.get_box();
-        if self.writing_mode.is_vertical() { box_style.width } else { box_style.height }
+        let position_style = self.get_position();
+        if self.writing_mode.is_vertical() { position_style.width } else { position_style.height }
     }
 
     #[inline]