Revert of New PaintController under-invalidation checking

third_party/WebKit/Source/platform/graphics/paint/PaintController.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 "platform/graphics/paint/PaintController.h"
 
 #include "platform/TraceEvent.h"
 #include "platform/graphics/GraphicsLayer.h"
 #include "platform/graphics/paint/DrawingDisplayItem.h"
 #include "third_party/skia/include/core/SkPictureAnalyzer.h"
@@ skipping 23 matching lines @@
 bool PaintController::useCachedDrawingIfPossible(const DisplayItemClient& client, DisplayItem::Type type)
 {
     DCHECK(DisplayItem::isDrawingType(type));
 
     if (displayItemConstructionIsDisabled())
         return false;
 
     if (!clientCacheIsValid(client))
         return false;
 
-#if DCHECK_IS_ON()
-    if (RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled() && isCheckingUnderInvalidation()) {
-        // We are checking under-invalidation of a subsequence enclosing this display item.
-        // Let the client continue to actually paint the display item.
+#if ENABLE(ASSERT)
+    // When under-invalidation checking is enabled, we output CachedDrawing display item
+    // followed by the display item containing forced painting.
+    if (RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled())
         return false;
-    }
 #endif
 
     DisplayItemList::iterator cachedItem = findCachedItem(DisplayItem::Id(client, type));
     if (cachedItem == m_currentPaintArtifact.getDisplayItemList().end()) {
         NOTREACHED();
         return false;
     }
 
     ++m_numCachedNewItems;
     ensureNewDisplayItemListInitialCapacity();
-    if (!DCHECK_IS_ON() || !RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled())
-        processNewItem(m_newDisplayItemList.appendByMoving(*cachedItem));
+    processNewItem(m_newDisplayItemList.appendByMoving(*cachedItem));
 
     m_nextItemToMatch = cachedItem + 1;
     // Items before m_nextItemToMatch have been copied so we don't need to index them.
     if (m_nextItemToMatch - m_nextItemToIndex > 0)
         m_nextItemToIndex = m_nextItemToMatch;
 
-#if DCHECK_IS_ON()
-    if (RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled()) {
-        if (!isCheckingUnderInvalidation()) {
-            m_underInvalidationCheckingBegin = cachedItem;
-            m_underInvalidationCheckingEnd = cachedItem + 1;
-            m_underInvalidationMessagePrefix = "";
-        }
-        // Return false to let the painter actually paint, and we will check if the new painting
-        // is the same as the cached.
-        return false;
-    }
-#endif
-
     return true;
 }
 
 bool PaintController::useCachedSubsequenceIfPossible(const DisplayItemClient& client)
 {
     // TODO(crbug.com/596983): Implement subsequence caching for spv2.
     // The problem is in copyCachedSubsequence() which fails to handle PaintChunkProperties
     // of chunks containing cached display items. We need to find the previous
     // PaintChunkProperties and ensure they are valid in the current paint property tree.
     if (RuntimeEnabledFeatures::slimmingPaintV2Enabled())
         return false;
 
     if (displayItemConstructionIsDisabled() || subsequenceCachingIsDisabled())
         return false;
 
     if (!clientCacheIsValid(client))
         return false;
 
-#if DCHECK_IS_ON()
-    if (RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled() && isCheckingUnderInvalidation()) {
-        // We are checking under-invalidation of an ancestor subsequence enclosing this one.
-        // The ancestor subsequence is supposed to have already "copied", so we should let the
-        // client continue to actually paint the descendant subsequences without "copying".
+#if ENABLE(ASSERT)
+    // When under-invalidation checking is enabled, we output CachedDrawing display item
+    // followed by the display item containing forced painting.
+    if (RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled())
         return false;
-    }
 #endif
 
     DisplayItemList::iterator cachedItem = findCachedItem(DisplayItem::Id(client, DisplayItem::Subsequence));
     if (cachedItem == m_currentPaintArtifact.getDisplayItemList().end()) {
         NOTREACHED();
         return false;
     }
 
     // |cachedItem| will point to the first item after the subsequence or end of the current list.
     ensureNewDisplayItemListInitialCapacity();
     copyCachedSubsequence(cachedItem);
 
     m_nextItemToMatch = cachedItem;
     // Items before |cachedItem| have been copied so we don't need to index them.
     if (cachedItem - m_nextItemToIndex > 0)
         m_nextItemToIndex = cachedItem;
 
-#if DCHECK_IS_ON()
-    if (RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled()) {
-        // Return false to let the painter actually paint, and we will check if the new painting
-        // is the same as the cached.
-        return false;
-    }
-#endif
-
     return true;
 }
 
 bool PaintController::lastDisplayItemIsNoopBegin() const
 {
     if (m_newDisplayItemList.isEmpty())
         return false;
 
     const auto& lastDisplayItem = m_newDisplayItemList.last();
     return lastDisplayItem.isBegin() && !lastDisplayItem.drawsContent();
@@ skipping 62 matching lines @@
     size_t index = findMatchingItemFromIndex(displayItem.getId(), m_newDisplayItemIndicesByClient, m_newDisplayItemList);
     if (index != kNotFound) {
 #ifndef NDEBUG
         showDebugData();
         WTFLogAlways("DisplayItem %s has duplicated id with previous %s (index=%d)\n",
             displayItem.asDebugString().utf8().data(), m_newDisplayItemList[index].asDebugString().utf8().data(), static_cast<int>(index));
 #endif
         NOTREACHED();
     }
     addItemToIndexIfNeeded(displayItem, m_newDisplayItemList.size() - 1, m_newDisplayItemIndicesByClient);
-
-    if (RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled())
-        checkUnderInvalidation();
 #endif // DCHECK_IS_ON()
 
     if (RuntimeEnabledFeatures::slimmingPaintV2Enabled())
         m_newPaintChunks.incrementDisplayItemIndex(behaviorOfItemType(displayItem.getType()));
 }
 
 void PaintController::updateCurrentPaintChunkProperties(const PaintChunkProperties& newProperties)
 {
     m_newPaintChunks.updateCurrentPaintChunkProperties(newProperties);
 }
@@ skipping 108 matching lines @@
     showDebugData();
     LOG(ERROR) << id.client.debugName() << ":" << DisplayItem::typeAsDebugString(id.type) << " not found in current display item list";
 #endif
     NOTREACHED();
     // We did not find the cached display item. This should be impossible, but may occur if there is a bug
     // in the system, such as under-invalidation, incorrect cache checking or duplicate display ids.
     // In this case, the caller should fall back to repaint the display item.
     return end;
 }
 
-// Copies a cached subsequence from current list to the new list.
 // On return, |it| points to the item after the EndSubsequence item of the subsequence.
-// When paintUnderInvaldiationCheckingEnabled() we'll not actually copy the subsequence,
-// but mark the begin and end of the subsequence for under-invalidation checking.
 void PaintController::copyCachedSubsequence(DisplayItemList::iterator& it)
 {
-#if DCHECK_IS_ON()
     DCHECK(it->getType() == DisplayItem::Subsequence);
-    if (RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled()) {
-        DCHECK(!isCheckingUnderInvalidation());
-        m_underInvalidationCheckingBegin = it;
-#ifndef NDEBUG
-        m_underInvalidationMessagePrefix = "(In CachedSubsequence of " + it->clientDebugString() + ")";
-#else
-        m_underInvalidationMessagePrefix = "(In CachedSubsequence)";
-#endif
-    }
-#endif
-
     DisplayItem::Id endSubsequenceId(it->client(), DisplayItem::EndSubsequence);
-    bool metEndSubsequence = false;
     do {
         // We should always find the EndSubsequence display item.
         DCHECK(it != m_currentPaintArtifact.getDisplayItemList().end());
         DCHECK(it->hasValidClient());
 #if CHECK_DISPLAY_ITEM_CLIENT_ALIVENESS
         CHECK(it->client().isAlive());
 #endif
         ++m_numCachedNewItems;
-        if (it->getId() == endSubsequenceId)
-            metEndSubsequence = true;
-        if (!DCHECK_IS_ON() || !RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled())
-            processNewItem(m_newDisplayItemList.appendByMoving(*it));
+        processNewItem(m_newDisplayItemList.appendByMoving(*it));
         ++it;
-    } while (!metEndSubsequence);
-
-#if DCHECK_IS_ON()
-    if (RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled()) {
-        m_underInvalidationCheckingEnd = it;
-        DCHECK(isCheckingUnderInvalidation());
-    }
-#endif
+    } while (endSubsequenceId != m_newDisplayItemList.last().getId());
 }
 
 static IntRect visualRectForDisplayItem(const DisplayItem& displayItem, const LayoutSize& offsetFromLayoutObject)
 {
     LayoutRect visualRect = displayItem.client().visualRect();
     visualRect.move(-offsetFromLayoutObject);
     return enclosingIntRect(visualRect);
 }
 
 void PaintController::resetCurrentListIterators()
 {
     m_nextItemToMatch = m_currentPaintArtifact.getDisplayItemList().begin();
     m_nextItemToIndex = m_nextItemToMatch;
-#if DCHECK_IS_ON()
-    m_underInvalidationCheckingBegin = m_currentPaintArtifact.getDisplayItemList().end();
-    m_underInvalidationCheckingEnd = m_currentPaintArtifact.getDisplayItemList().begin();
-#endif
 }
 
 void PaintController::commitNewDisplayItems(const LayoutSize& offsetFromLayoutObject)
 {
     TRACE_EVENT2("blink,benchmark", "PaintController::commitNewDisplayItems",
         "current_display_list_size", (int)m_currentPaintArtifact.getDisplayItemList().size(),
         "num_non_cached_new_items", (int)m_newDisplayItemList.size() - m_numCachedNewItems);
     m_numCachedNewItems = 0;
 
     // These data structures are used during painting only.
@@ skipping 66 matching lines @@
 {
     DCHECK(m_newDisplayItemList.isEmpty());
     DrawingDisplayItem& displayItem = m_currentPaintArtifact.getDisplayItemList().allocateAndConstruct<DrawingDisplayItem>(displayItemClient, DisplayItem::DebugDrawing, picture);
     displayItem.setSkippedCache();
     m_currentPaintArtifact.getDisplayItemList().appendVisualRect(visualRectForDisplayItem(displayItem, offsetFromLayoutObject));
 
     // Need to reset the iterators after mutation of the DisplayItemList.
     resetCurrentListIterators();
 }
 
-#if DCHECK_IS_ON()
+#if 0 // DCHECK_IS_ON()
+// TODO(wangxianzhu): Fix under-invalidation checking for the new caching method.
 
-void PaintController::showUnderInvalidationError(const char* reason, const DisplayItem& newItem, const DisplayItem& oldItem) const
+void PaintController::checkUnderInvalidation(DisplayItemList::iterator& newIt, DisplayItemList::iterator& currentIt)
 {
-    LOG(ERROR) << m_underInvalidationMessagePrefix << " " << reason;
+    DCHECK(RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled());
+
+    // When under-invalidation-checking is enabled, the forced painting is following the cached display item.
+    DisplayItem::Type nextItemType = DisplayItem::nonCachedType(newIt->getType());
+    ++newIt;
+    DCHECK(newIt->getType() == nextItemType);
+
+    if (newIt->isDrawing()) {
+        checkCachedDisplayItemIsUnchanged("", *newIt, *currentIt);
+        return;
+    }
+
+    DCHECK(newIt->getType() == DisplayItem::Subsequence);
+
 #ifndef NDEBUG
-    LOG(ERROR) << "New display item: " << newItem.asDebugString();
-    LOG(ERROR) << "Old display item: " << oldItem.asDebugString();
+    CString messagePrefix = String::format("(In CachedSubsequence of %s)", newIt->clientDebugString().utf8().data()).utf8();
 #else
-    LOG(ERROR) << "Run debug build to get more details.";
+    CString messagePrefix = "(In CachedSubsequence)";
 #endif
-    LOG(ERROR) << "See http://crbug.com/619103.";
+
+    DisplayItem::Id endSubsequenceId(newIt->client(), DisplayItem::EndSubsequence);
+    while (true) {
+        DCHECK(newIt != m_newDisplayItemList.end());
+        if (newIt->isCached())
+            checkUnderInvalidation(newIt, currentIt);
+        else
+            checkCachedDisplayItemIsUnchanged(messagePrefix.data(), *newIt, *currentIt);
+
+        if (endSubsequenceId.matches(*newIt))
+            break;
+
+        ++newIt;
+        ++currentIt;
+    }
+}
+
+static void showUnderInvalidationError(const char* messagePrefix, const char* reason, const DisplayItem* newItem, const DisplayItem* oldItem)
+{
+#ifndef NDEBUG
+    WTFLogAlways("%s %s:\nNew display item: %s\nOld display item: %s\nSee http://crbug.com/450725.", messagePrefix, reason,
+        newItem ? newItem->asDebugString().utf8().data() : "None",
+        oldItem ? oldItem->asDebugString().utf8().data() : "None");
+#else
+    WTFLogAlways("%s %s. Run debug build to get more details\nSee http://crbug.com/450725.", messagePrefix, reason);
+#endif // NDEBUG
+}
+
+void PaintController::checkCachedDisplayItemIsUnchanged(const char* messagePrefix, const DisplayItem& newItem, const DisplayItem& oldItem)
+{
+    DCHECK(RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled());
+    DCHECK(!newItem.isCached());
+    DCHECK(!oldItem.isCached());
+
+    if (newItem.skippedCache()) {
+        showUnderInvalidationError(messagePrefix, "ERROR: under-invalidation: skipped-cache in cached subsequence", &newItem, &oldItem);
+        NOTREACHED();
+    }
+
+    if (newItem.isCacheable() && !clientCacheIsValid(newItem.client())) {
+        showUnderInvalidationError(messagePrefix, "ERROR: under-invalidation: invalidated in cached subsequence", &newItem, &oldItem);
+        NOTREACHED();
+    }
+
+    if (newItem.equals(oldItem))
+        return;
+
+    showUnderInvalidationError(messagePrefix, "ERROR: under-invalidation: display item changed", &newItem, &oldItem);
 
 #ifndef NDEBUG
     if (newItem.isDrawing()) {
         RefPtr<const SkPicture> newPicture = static_cast<const DrawingDisplayItem&>(newItem).picture();
         RefPtr<const SkPicture> oldPicture = static_cast<const DrawingDisplayItem&>(oldItem).picture();
-        LOG(INFO) << "new picture:\n" << (newPicture ? pictureAsDebugString(newPicture.get()) : "None");
-        LOG(INFO) << "old picture:\n" << (oldPicture ? pictureAsDebugString(oldPicture.get()) : "None");
+        String oldPictureDebugString = oldPicture ? pictureAsDebugString(oldPicture.get()) : "None";
+        String newPictureDebugString = newPicture ? pictureAsDebugString(newPicture.get()) : "None";
+        WTFLogAlways("old picture:\n%s\n", oldPictureDebugString.utf8().data());
+        WTFLogAlways("new picture:\n%s\n", newPictureDebugString.utf8().data());
     }
 #endif // NDEBUG
-}
 
-void PaintController::checkUnderInvalidation()
-{
-    DCHECK(RuntimeEnabledFeatures::slimmingPaintUnderInvalidationCheckingEnabled());
-
-    if (!isCheckingUnderInvalidation())
-        return;
-
-    const DisplayItem& newItem = m_newDisplayItemList.last();
-    const DisplayItem& oldItem = *m_underInvalidationCheckingBegin++;
-
-    if (newItem.isCacheable() && !clientCacheIsValid(newItem.client())) {
-        showUnderInvalidationError("under-invalidation of PaintLayer: invalidated in cached subsequence", newItem, oldItem);
-        NOTREACHED();
-    }
-    if (!newItem.equals(oldItem)) {
-        showUnderInvalidationError("under-invalidation: display item changed", newItem, oldItem);
-        NOTREACHED();
-    }
+    NOTREACHED();
 }
 
 #endif // DCHECK_IS_ON()
 
 #ifndef NDEBUG
 
 String PaintController::displayItemListAsDebugString(const DisplayItemList& list) const
 {
     StringBuilder stringBuilder;
     size_t i = 0;
     for (auto it = list.begin(); it != list.end(); ++it, ++i) {
         const DisplayItem& displayItem = *it;
         if (i)
             stringBuilder.append(",\n");
         stringBuilder.append(String::format("{index: %d, ", (int)i));
         displayItem.dumpPropertiesAsDebugString(stringBuilder);
         if (displayItem.hasValidClient()) {
             stringBuilder.append(", cacheIsValid: ");
             stringBuilder.append(clientCacheIsValid(displayItem.client()) ? "true" : "false");
         }
-        if (list.hasVisualRect(i)) {
-            IntRect visualRect = list.visualRect(i);
-            stringBuilder.append(String::format(", visualRect: [%d,%d %dx%d]",
-                visualRect.x(), visualRect.y(),
-                visualRect.width(), visualRect.height()));
-        }
+        IntRect visualRect = list.visualRect(i);
+        stringBuilder.append(String::format(", visualRect: [%d,%d %dx%d]",
+            visualRect.x(), visualRect.y(),
+            visualRect.width(), visualRect.height()));
         stringBuilder.append('}');
     }
     return stringBuilder.toString();
 }
 
 void PaintController::showDebugData() const
 {
     WTFLogAlways("current display item list: [%s]\n", displayItemListAsDebugString(m_currentPaintArtifact.getDisplayItemList()).utf8().data());
     WTFLogAlways("new display item list: [%s]\n", displayItemListAsDebugString(m_newDisplayItemList).utf8().data());
 }
 
 #endif // ifndef NDEBUG
 
 } // namespace blink