Remove unnecessary ClipDisplayItems in box_.rs

For ScannedTextBox and ImageBox, the ClipDisplayItem's child_list is
currently always empty.

ClipDisplayItem is used to implement overflow hidden and should only be
created for block containers, as per
http://www.w3.org/TR/CSS2/visufx.html#propdef-overflow

Take care of the case when a BlockFlow has no children display items -
like an ImageBox with "display: block".

src/components/main/layout/box_.rs

@@ -967,18 +967,6 @@ impl Box {
         match self.specific {
             UnscannedTextBox(_) => fail!("Shouldn't see unscanned boxes here."),
             ScannedTextBox(ref text_box) => {
-                lists.with_mut(|lists| {
-                    let item = ~ClipDisplayItem {
-                        base: BaseDisplayItem {
-                            bounds: absolute_box_bounds,
-                            extra: ExtraDisplayListData::new(self),
-                        },
-                        child_list: ~[],
-                        need_clip: false
-                    };
-                    lists.lists[index].append_item(ClipDisplayItemClass(item));
-                });
-
                 let text_color = self.style().Color.color.to_gfx_color();
 
                 // Set the various text display item flags.
@@ -1104,18 +1092,6 @@ impl Box {
                 });
             },
             ImageBox(ref image_box) => {
-                lists.with_mut(|lists| {
-                    let item = ~ClipDisplayItem {
-                        base: BaseDisplayItem {
-                            bounds: absolute_box_bounds,
-                            extra: ExtraDisplayListData::new(self),
-                        },
-                        child_list: ~[],
-                        need_clip: false
-                    };
-                    lists.lists[index].append_item(ClipDisplayItemClass(item));
-                });
-
                 let mut image_ref = image_box.image.borrow_mut();
                 let mut bounds = absolute_box_bounds.clone();
                 bounds.origin.x = bounds.origin.x + self.noncontent_left()

src/components/main/layout/flow.rs

@@ -167,6 +167,9 @@ pub trait ImmutableFlowUtils {
     /// Returns the number of children that this flow possesses.
     fn child_count(self) -> uint;
 
+    /// Return true if this flow is a Block Container.
+    fn is_block_container(self) -> bool;
+
     /// Returns true if this flow is a block flow, an inline flow, or a float flow.
     fn starts_block_flow(self) -> bool;
 
@@ -604,6 +607,23 @@ impl<'a> ImmutableFlowUtils for &'a Flow {
         base(self).children.len()
     }
 
+    /// Return true if this flow is a Block Container.
+    ///
+    /// Except for table boxes and replaced elements, block-level boxes (`BlockFlow`) are
+    /// also block container boxes.
+    /// Non-replaced inline blocks and non-replaced table cells are also block
+    /// containers.
+    fn is_block_container(self) -> bool {
+        match self.class() {
+            // TODO: Change this when inline-blocks are supported.
+            InlineFlowClass => false,
+            BlockFlowClass => {
+                // FIXME: Actually check the type of the node
+                self.child_count() != 0
+            }
+        }
+    }
+
     /// Returns true if this flow is a block flow, an inline-block flow, or a float flow.
     fn starts_block_flow(self) -> bool {
         match self.class() {
@@ -720,6 +740,7 @@ impl<'a> MutableFlowUtils for &'a mut Flow {
             return true;
         }
 
+        if self.is_block_container() {
             let mut child_lists = DisplayListCollection::new();
             child_lists.add_list(DisplayList::new());
             let child_lists = RefCell::new(child_lists);
@@ -730,9 +751,6 @@ impl<'a> MutableFlowUtils for &'a mut Flow {
             let mut child_lists = Some(child_lists.unwrap());
             // Find parent ClipDisplayItemClass and push all child display items
             // under it
-        // FIXME: Once we have children for InlineFlow, this might lead to
-        // children display items being pushed under the ClipDisplayItemClass
-        // created by the last box of the InlineFlow. Fix the logic.
             lists.with_mut(|lists| {
                 let mut child_lists = child_lists.take_unwrap();
                 let result = lists.lists[index].list.mut_rev_iter().position(|item| {
@@ -751,6 +769,7 @@ impl<'a> MutableFlowUtils for &'a mut Flow {
 
                 lists.lists.push_all_move(child_lists.lists);
             });
+        }
         true
     }