New display item caching

Source/platform/graphics/paint/DrawingRecorder.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/DrawingRecorder.h"
 
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/graphics/GraphicsContext.h"
 #include "platform/graphics/GraphicsLayer.h"
 #include "platform/graphics/paint/CachedDisplayItem.h"
 #include "platform/graphics/paint/DisplayItemList.h"
 #include "platform/graphics/paint/DrawingDisplayItem.h"
 #include "third_party/skia/include/core/SkPicture.h"
 
 namespace blink {
 
 DrawingRecorder::DrawingRecorder(GraphicsContext* context, const DisplayItemClient displayItemClient, DisplayItem::Type displayItemType, const FloatRect& bounds)
     : m_context(context)
     , m_displayItemClient(displayItemClient)
     , m_displayItemType(displayItemType)
+    , m_canUseCachedDrawing(false)
+#ifndef NDEBUG
     , m_bounds(bounds)
-    , m_canUseCachedDrawing(false)
+#endif
 {
     if (!RuntimeEnabledFeatures::slimmingPaintEnabled())
         return;
 
 #if ENABLE(ASSERT)
     context->setInDrawingRecorder(true);
 #endif
     ASSERT(context->displayItemList());
     m_canUseCachedDrawing = context->displayItemList()->clientCacheIsValid(displayItemClient);
 #if ENABLE(ASSERT)
@@ skipping 14 matching lines @@
     m_context->setInDrawingRecorder(false);
 #endif
 
     OwnPtr<DisplayItem> displayItem;
 
     if (m_canUseCachedDrawing) {
 #if ENABLE(ASSERT)
         RefPtr<const SkPicture> picture = m_context->endRecording();
         ASSERT(!picture || !picture->approximateOpCount());
 #endif
+#ifdef NDEBUG
         displayItem = CachedDisplayItem::create(m_displayItemClient, m_displayItemType);
+#else
+        displayItem = CachedDisplayItem::create(m_displayItemClient, m_displayItemType, m_bounds);
+#endif
     } else {
         RefPtr<const SkPicture> picture = m_context->endRecording();
         if (!picture || !picture->approximateOpCount())
             displayItem = DisplayItem::create(m_displayItemClient, m_displayItemType);
         else
             displayItem = DrawingDisplayItem::create(m_displayItemClient, m_displayItemType, picture);
     }
 
 #ifndef NDEBUG
     displayItem->setClientDebugString(m_clientDebugString);
 #endif
 
     m_context->displayItemList()->add(displayItem.release());
 }
 
 #ifndef NDEBUG
 void DrawingRecorder::setClientDebugString(const WTF::String& clientDebugString)
 {
     m_clientDebugString = clientDebugString;
 }
 #endif
 
 } // namespace blink