servo: Merge #8642 - making internal pub enums, functions, and structs private (from jmr0:master); r=jdm

Refactoring to fix #8620

Source-Repo: https://github.com/servo/servo
Source-Revision: 3199eace981899bbaac74abe9362f90b14118d7c

servo/components/layout/query.rs

@@ -134,13 +134,13 @@ impl LayoutRPC for LayoutRPCImpl {
     }
 }
 
-pub struct UnioningFragmentBorderBoxIterator {
-    pub node_address: OpaqueNode,
-    pub rect: Option<Rect<Au>>,
+struct UnioningFragmentBorderBoxIterator {
+    node_address: OpaqueNode,
+    rect: Option<Rect<Au>>,
 }
 
 impl UnioningFragmentBorderBoxIterator {
-    pub fn new(node_address: OpaqueNode) -> UnioningFragmentBorderBoxIterator {
+    fn new(node_address: OpaqueNode) -> UnioningFragmentBorderBoxIterator {
         UnioningFragmentBorderBoxIterator {
             node_address: node_address,
             rect: None
@@ -165,13 +165,13 @@ impl FragmentBorderBoxIterator for UnioningFragmentBorderBoxIterator {
     }
 }
 
-pub struct CollectingFragmentBorderBoxIterator {
-    pub node_address: OpaqueNode,
-    pub rects: Vec<Rect<Au>>,
+struct CollectingFragmentBorderBoxIterator {
+    node_address: OpaqueNode,
+    rects: Vec<Rect<Au>>,
 }
 
 impl CollectingFragmentBorderBoxIterator {
-    pub fn new(node_address: OpaqueNode) -> CollectingFragmentBorderBoxIterator {
+    fn new(node_address: OpaqueNode) -> CollectingFragmentBorderBoxIterator {
         CollectingFragmentBorderBoxIterator {
             node_address: node_address,
             rects: Vec::new(),
@@ -189,19 +189,19 @@ impl FragmentBorderBoxIterator for CollectingFragmentBorderBoxIterator {
     }
 }
 
-pub enum Side {
+enum Side {
     Left,
     Right,
     Bottom,
     Top
 }
 
-pub enum MarginPadding {
+enum MarginPadding {
     Margin,
     Padding
 }
 
-pub enum PositionProperty {
+enum PositionProperty {
     Left,
     Right,
     Top,
@@ -210,15 +210,15 @@ pub enum PositionProperty {
     Height,
 }
 
-pub struct PositionRetrievingFragmentBorderBoxIterator {
+struct PositionRetrievingFragmentBorderBoxIterator {
     node_address: OpaqueNode,
-    pub result: Option<Au>,
+    result: Option<Au>,
     position: Point2D<Au>,
     property: PositionProperty,
 }
 
 impl PositionRetrievingFragmentBorderBoxIterator {
-    pub fn new(node_address: OpaqueNode,
+    fn new(node_address: OpaqueNode,
            property: PositionProperty,
            position: Point2D<Au>) -> PositionRetrievingFragmentBorderBoxIterator {
         PositionRetrievingFragmentBorderBoxIterator {
@@ -252,16 +252,16 @@ impl FragmentBorderBoxIterator for PositionRetrievingFragmentBorderBoxIterator {
     }
 }
 
-pub struct MarginRetrievingFragmentBorderBoxIterator {
+struct MarginRetrievingFragmentBorderBoxIterator {
     node_address: OpaqueNode,
-    pub result: Option<Au>,
+    result: Option<Au>,
     writing_mode: WritingMode,
     margin_padding: MarginPadding,
     side: Side,
 }
 
 impl MarginRetrievingFragmentBorderBoxIterator {
-    pub fn new(node_address: OpaqueNode, side: Side, margin_padding:
+    fn new(node_address: OpaqueNode, side: Side, margin_padding:
     MarginPadding, writing_mode: WritingMode) -> MarginRetrievingFragmentBorderBoxIterator {
         MarginRetrievingFragmentBorderBoxIterator {
             node_address: node_address,