First version of new merge algorithm (not enabled yet)

Source/platform/graphics/paint/DisplayItemList.h

 // 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.
 
 #ifndef DisplayItemList_h
 #define DisplayItemList_h
 
 #include "platform/PlatformExport.h"
 #include "platform/graphics/paint/DisplayItem.h"
-#include "wtf/HashSet.h"
+#include "wtf/HashMap.h"
 #include "wtf/PassOwnPtr.h"
 #include "wtf/Vector.h"
 
 namespace blink {
 
 class GraphicsContext;
 
 typedef Vector<OwnPtr<DisplayItem>> PaintList;
 
 class PLATFORM_EXPORT DisplayItemList {
@@ skipping 15 matching lines @@
     void replay(GraphicsContext*);
 
 #ifndef NDEBUG
     void showDebugData() const;
 #endif
 
 protected:
     DisplayItemList() { };
 
 private:
-    PaintList::iterator findNextMatchingCachedItem(PaintList::iterator, const DisplayItem&);
-    bool wasInvalidated(const DisplayItem&) const;
+    friend class ViewDisplayListTest;
+    friend class DrawingRecorderTest;
+
     void updatePaintList();
 
 #ifndef NDEBUG
     WTF::String paintListAsDebugString(const PaintList&) const;
 #endif
 
+    struct DisplayItemIndices {
+        // Indices into PaintList of all DrawingDisplayItems of a client.

[pdr., 2015/02/05 03:35:57]

Do we need to track these separately or can we just use a single vector here?

[Xianzhu, 2015/02/05 17:57:03]

Thought to reduce the number of items of each vector. Seems not necessary
because the total number will not be big, and seems to sacrifice readibility.

Combined them.

+        Vector<size_t> drawingDisplayItemIndices;
+        // Indices into PaintList of all BeginSubtreeDisplayItems of a client.
+        Vector<size_t> beginSubtreeDisplayItemIndices;
+    };
+    typedef HashMap<DisplayItemClient, DisplayItemIndices> DisplayItemIndicesByClientMap;
+
+    size_t findMatchingCachedItem(const DisplayItem&);
+    static void appendDisplayItem(PaintList&, DisplayItemIndicesByClientMap&, WTF::PassOwnPtr<DisplayItem>);
+    void copyCachedSubtree(const DisplayItem&, PaintList&, DisplayItemIndicesByClientMap&);
+
     PaintList m_paintList;
-    HashSet<DisplayItemClient> m_cachedClients;
+    DisplayItemIndicesByClientMap m_cachedDisplayItemIndicesByClient;
     PaintList m_newPaints;
 };
 
 } // namespace blink
 
 #endif // DisplayItemList_h