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/HashMap.h"
 #include "wtf/HashSet.h"
 #include "wtf/PassOwnPtr.h"
 #include "wtf/Vector.h"
 
 namespace blink {
 
 class GraphicsContext;
 
 typedef Vector<OwnPtr<DisplayItem>> PaintList;
 
 class PLATFORM_EXPORT DisplayItemList {
     WTF_MAKE_NONCOPYABLE(DisplayItemList);
     WTF_MAKE_FAST_ALLOCATED;
 public:
     static PassOwnPtr<DisplayItemList> create() { return adoptPtr(new DisplayItemList); }
 
     void endNewPaints() { updatePaintList(); }
 
-    const PaintList& paintList();
+    const PaintList& paintList() const;
     void add(WTF::PassOwnPtr<DisplayItem>);
 
+    const PaintList& newPaintListForTesting() const { return m_newPaints; }

[chrishtr, 2015/02/03 22:30:48]

friend classses would be better for these two. Chromium has FRIEND_TEST to make
this easy..

[Xianzhu, 2015/02/04 00:13:24]

For now we can't include either <gtest/gtest.h> or "base/gtest_prod_util.h" from
non-testing blink code.

Use another way: let the test base classes be friend, and access the private
member in the base classes.

+    const PaintList& oldPaintListForTesting() const { return m_paintList; }
+
     void invalidate(DisplayItemClient);
     void invalidateAll();
     bool clientCacheIsValid(DisplayItemClient) const;
 
     // Plays back the current PaintList() into the given context.
     void replay(GraphicsContext*);
 
 #ifndef NDEBUG
     void showDebugData() const;
 #endif
 
 protected:
     DisplayItemList() { };
 
 private:
-    PaintList::iterator findNextMatchingCachedItem(PaintList::iterator, const DisplayItem&);
-    bool wasInvalidated(const DisplayItem&) const;
-    void updatePaintList();
-
 #ifndef NDEBUG
     WTF::String paintListAsDebugString(const PaintList&) const;
 #endif
 
+    void updatePaintList();
+
+    typedef HashSet<DisplayItemClient> DisplayItemClientSet;
+    typedef HashMap<DisplayItem::Id, size_t> DisplayItemIndexMap;
+
+    size_t findDisplayItemById(const DisplayItem::Id&) const;
+    size_t findMatchingCachedItem(const DisplayItem&) const;
+
+    static void appendDisplayItem(WTF::PassOwnPtr<DisplayItem>, PaintList&, DisplayItemClientSet&, DisplayItemIndexMap&);
+    void copyCachedSubtree(const DisplayItem&, PaintList&, DisplayItemClientSet&, DisplayItemIndexMap&);
+
     PaintList m_paintList;
-    HashSet<DisplayItemClient> m_cachedClients;
+    DisplayItemClientSet m_cachedClients;
+    DisplayItemIndexMap m_displayItemIndexMap;
     PaintList m_newPaints;
+#if ENABLE(ASSERT)
+    HashSet<DisplayItem::Id> m_newPaintIds;
+#endif
 };
 
 } // namespace blink
 
 #endif // DisplayItemList_h