Removed a bunch of unneeded paint phase checks.

sky/engine/core/rendering/RenderLayer.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  *
  * Portions are Copyright (C) 1998 Netscape Communications Corporation.
  *
  * Other contributors:
  *   Robert O'Callahan <roc+@cs.cmu.edu>
  *   David Baron <dbaron@fas.harvard.edu>
  *   Christian Biesinger <cbiesinger@web.de>
  *   Randall Jesup <rjesup@wgate.com>
@@ skipping 998 matching lines @@
 
     // Begin transparency if we have something to paint.
     if (haveTransparency && !foregroundRectIsEmpty)
         beginTransparencyLayers(transparencyLayerContext, localPaintingInfo.rootLayer, transparencyPaintDirtyRect, localPaintingInfo.subPixelAccumulation);
 
     // Optimize clipping for the single fragment case.
     bool shouldClip = localPaintingInfo.clipToDirtyRect && !foregroundRectIsEmpty;
     if (shouldClip)
         clipToRect(localPaintingInfo, context, layerForegroundRect);
 
+    // TODO(ojan): We probably should early return at the beginning of this function
+    // if the rect is empty.
     if (!foregroundRectIsEmpty) {
-        // We have to loop through every fragment multiple times, since we have to issue paint invalidations in each specific phase in order for
-        // interleaving of the fragments to work properly.
-        // FIXME(sky): Do we still need this for anything now that we don't have fragments?
         paintForegroundWithPhase(PaintPhaseForeground,
             context, localPaintingInfo,
             layerLocation, layerForegroundRect);
     }
 
     if (shouldClip)
         restoreClip(context, localPaintingInfo.paintDirtyRect, layerForegroundRect);
 }
 
 void RenderLayer::paintForegroundWithPhase(PaintPhase phase, GraphicsContext* context,
@@ skipping 666 matching lines @@
     }
 }
 
 void showLayerTree(const blink::RenderObject* renderer)
 {
     if (!renderer)
         return;
     showLayerTree(renderer->enclosingLayer());
 }
 #endif