Renaming the color function to just *transparent*,

as suggested on review.
servo · Apr 22, 2015 · 2c7edeb · 2c7edeb
1 parent cf2406d
commit 2c7edeb
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/components/compositing/compositor.rs b/components/compositing/compositor.rs
@@ -548,7 +548,7 @@ impl<Window: WindowMethods> IOCompositor<Window> {
             epoch: Epoch(0),
             id: LayerId::null(),
             rect: Rect::zero(),
-            background_color: color::transparent_black(),
+            background_color: color::transparent(),
             scroll_policy: ScrollPolicy::Scrollable,
         };
 

diff --git a/components/gfx/color.rs b/components/gfx/color.rs
@@ -31,7 +31,7 @@ pub fn black() -> AzColor {
 }
 
 #[inline]
-pub fn transparent_black() -> AzColor {
+pub fn transparent() -> AzColor {
     AzColor { r: 0.0, g: 0.0, b: 0.0, a: 0.0 }
 }
 

diff --git a/components/gfx/paint_context.rs b/components/gfx/paint_context.rs
@@ -169,7 +169,7 @@ impl<'a> PaintContext<'a> {
     }
 
     pub fn clear(&self) {
-        let pattern = ColorPattern::new(color::transparent_black());
+        let pattern = ColorPattern::new(color::transparent());
         let rect = Rect(Point2D(self.page_rect.origin.x as AzFloat,
                                 self.page_rect.origin.y as AzFloat),
                         Size2D(self.screen_rect.size.width as AzFloat,

diff --git a/components/layout/display_list_builder.rs b/components/layout/display_list_builder.rs
@@ -1318,7 +1318,7 @@ impl BlockFlowDisplayListBuilding for BlockFlow {
             ScrollPolicy::Scrollable
         };
 
-        let transparent = color::transparent_black();
+        let transparent = color::transparent();
         let stacking_context =
             self.create_stacking_context(display_list,
                                          Some(Arc::new(PaintLayer::new(self.layer_id(0),

diff --git a/components/layout/layout_task.rs b/components/layout/layout_task.rs
@@ -1226,16 +1226,16 @@ impl FragmentBorderBoxIterator for CollectingFragmentBorderBoxIterator {
 // transparent background color is handled correctly.
 fn get_root_flow_background_color(flow: &mut Flow) -> AzColor {
     if !flow.is_block_like() {
-        return color::transparent_black()
+        return color::transparent()
     }
 
     let block_flow = flow.as_block();
     let kid = match block_flow.base.children.iter_mut().next() {
-        None => return color::transparent_black(),
+        None => return color::transparent(),
         Some(kid) => kid,
     };
     if !kid.is_block_like() {
-        return color::transparent_black()
+        return color::transparent()
     }
 
     let kid_block_flow = kid.as_block();