Refactor DrawingRecorders to check for cached drawings earlier

Source/platform/graphics/paint/DisplayItemListTest.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "platform/graphics/paint/DisplayItemList.h"
 
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/graphics/GraphicsContext.h"
 #include "platform/graphics/paint/ClipPathRecorder.h"
@@ skipping 66 matching lines @@
         if (expectedSize != actual.size()) \
             break; \
         const TestDisplayItem expected[] = { __VA_ARGS__ }; \
         for (size_t index = 0; index < std::min<size_t>(actual.size(), expectedSize); index++) { \
             TRACE_DISPLAY_ITEMS(index, expected[index], *actual.elementAt(index)); \
             EXPECT_EQ(expected[index].client(), actual.elementAt(index)->client()); \
             EXPECT_EQ(expected[index].type(), actual.elementAt(index)->type()); \
         } \
     } while (false);
 
-void drawRect(GraphicsContext& context, const TestDisplayItemClient& client, DisplayItem::Type type, const FloatRect& bound)
+void drawRect(GraphicsContext& context, const TestDisplayItemClient& client, DisplayItem::Type type, const FloatRect& bounds)
 {
-    DrawingRecorder drawingRecorder(context, client, type, bound);
-    if (drawingRecorder.canUseCachedDrawing())
+    if (DrawingRecorder::useCachedDrawingIfPossible(context, client, type))
         return;
+    DrawingRecorder drawingRecorder(context, client, type, bounds);
     IntRect rect(0, 0, 10, 10);
     context.drawRect(rect);
 }
 
 void drawClippedRect(GraphicsContext& context, const TestDisplayItemClient& client, DisplayItem::Type clipType, DisplayItem::Type drawingType, const FloatRect& bound)
 {
     ClipRecorder clipRecorder(context, client, clipType, LayoutRect(1, 1, 9, 9));
     drawRect(context, client, drawingType, bound);
 }
 
@@ skipping 593 matching lines @@
     drawRect(context, third, backgroundDrawingType, FloatRect(0, 0, 100, 100));
     displayItemList().commitNewDisplayItems();
 
     // Empty clips should have been optimized out.
     EXPECT_DISPLAY_LIST(displayItemList().displayItems(), 2,
         TestDisplayItem(first, backgroundDrawingType),
         TestDisplayItem(third, backgroundDrawingType));
 }
 
 } // namespace blink