Repaint on interest rect change

third_party/WebKit/Source/core/layout/compositing/CompositedLayerMapping.cpp

 /*
  * Copyright (C) 2009, 2010, 2011 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 2153 matching lines @@
     if (!scrollbar)
         return;
 
     const IntRect& scrollbarRect = scrollbar->frameRect();
     TransformRecorder transformRecorder(context, *scrollbar, AffineTransform::translation(-scrollbarRect.x(), -scrollbarRect.y()));
     IntRect transformedClip = clip;
     transformedClip.moveBy(scrollbarRect.location());
     scrollbar->paint(&context, transformedClip);
 }
 
+// The following should be kept in sync with the code computing potential_new_recorded_viewport in
+// cc::DisplayListRecordingSource::UpdateAndExpandInvalidation() before we keep only one copy of the algorithm.
 static const int kPixelDistanceToRecord = 4000;
 
 IntRect CompositedLayerMapping::computeInterestRect(const GraphicsLayer* graphicsLayer, LayoutObject* owningLayoutObject)
 {
     FloatRect graphicsLayerBounds(FloatPoint(), graphicsLayer->size());
 
     // Start with the bounds of the graphics layer in the space of the owning LayoutObject.
     FloatRect graphicsLayerBoundsInObjectSpace(graphicsLayerBounds);
     graphicsLayerBoundsInObjectSpace.move(graphicsLayer->offsetFromLayoutObject());
 
     // Now map the bounds to its visible content rect in screen space, including applying clips along the way.
     LayoutRect visibleContentRect(graphicsLayerBoundsInObjectSpace);
     LayoutView* rootView = owningLayoutObject->view();
     while (rootView->frame()->ownerLayoutObject())
         rootView = rootView->frame()->ownerLayoutObject()->view();
     owningLayoutObject->mapRectToPaintInvalidationBacking(rootView, visibleContentRect, 0);
     visibleContentRect.intersect(LayoutRect(rootView->frameView()->visibleContentRect()));
 
     // Map the visible content rect from screen space to local graphics layer space.
     IntRect localInterestRect;
     // If the visible content rect is empty, then it makes no sense to map it back since there is nothing to map.
     if (!visibleContentRect.isEmpty()) {
         localInterestRect = owningLayoutObject->absoluteToLocalQuad(FloatRect(visibleContentRect), UseTransforms).enclosingBoundingBox();
         localInterestRect.move(-graphicsLayer->offsetFromLayoutObject());
     }
     // Expand by interest rect padding amount.
-    localInterestRect.expand(IntRectOutsets(kPixelDistanceToRecord, kPixelDistanceToRecord, kPixelDistanceToRecord, kPixelDistanceToRecord));
+    localInterestRect.inflate(kPixelDistanceToRecord);
     localInterestRect.intersect(enclosingIntRect(graphicsLayerBounds));
     return localInterestRect;
 }
 
-void CompositedLayerMapping::paintContentsIfNeeded(const GraphicsLayer* graphicsLayer, GraphicsContext& context, GraphicsLayerPaintingPhase graphicsLayerPaintingPhase) const
+// The following should be kept in sync with cc::DisplayListRecordingSource::ExposesEnoughNewArea()
+// before we keep only one copy of the algorithm.
+static const int kMinimumDistanceBeforeRepaint = 512;
+
+bool CompositedLayerMapping::interestRectChangedEnoughToRepaint(const IntSize& layerSize, const IntRect& previousInterestRect, const IntRect& newInterestRect)
 {
-    ASSERT(RuntimeEnabledFeatures::slimmingPaintSynchronizedPaintingEnabled());
+    if (previousInterestRect.isEmpty() && newInterestRect.isEmpty())
+        return false;
 
-    // TODO(chrishtr): Also paint if the interest rect has changed sufficiently.
-    if (!m_owningLayer.needsRepaint()) {
-        context.paintController().createAndAppend<CachedDisplayItem>(*this, DisplayItem::CachedDisplayItemList);
-        return;
+    // Repaint when going from empty to not-empty, to cover cases where the layer is
+    // painted for the first time, or otherwise becomes visible.
+    if (previousInterestRect.isEmpty())
+        return true;
+
+    // Repaint if the new interest rect includes area outside of a skirt around the existing interest rect.
+    IntRect expandedPreviousInterestRect(previousInterestRect);
+    expandedPreviousInterestRect.inflate(kMinimumDistanceBeforeRepaint);
+    if (!expandedPreviousInterestRect.contains(newInterestRect))
+        return true;
+
+    // Even if the new interest rect doesn't include enough new area to satisfy the condition above,
+    // repaint anyway if it touches a layer edge not touched by the existing interest rect.
+    // Because it's impossible to expose more area in the direction, repainting cannot be deferred
+    // until the exposed new area satisfies the condition above.
+    if (newInterestRect.x() == 0 && previousInterestRect.x() != 0)
+        return true;
+    if (newInterestRect.y() == 0 && previousInterestRect.y() != 0)
+        return true;
+    if (newInterestRect.maxX() == layerSize.width() && previousInterestRect.maxX() != layerSize.width())
+        return true;
+    if (newInterestRect.maxY() == layerSize.height() && previousInterestRect.maxY() != layerSize.height())
+        return true;
+
+    return false;
+}
+
+void CompositedLayerMapping::paintContents(const GraphicsLayer* graphicsLayer, GraphicsContext& context, GraphicsLayerPaintingPhase graphicsLayerPaintingPhase, const IntRect* interestRect) const
+{
+    IntRect defaultInterestRect;
+    if (RuntimeEnabledFeatures::slimmingPaintSynchronizedPaintingEnabled()) {
+        if (!interestRect) {
+            if (graphicsLayer == m_graphicsLayer || graphicsLayer == m_squashingLayer)
+                defaultInterestRect = computeInterestRect(graphicsLayer, m_owningLayer.layoutObject());
+            else
+                defaultInterestRect = enclosingIntRect(FloatRect(FloatPoint(), graphicsLayer->size()));
+            interestRect = &defaultInterestRect;
+        }
+
+        if (!m_owningLayer.needsRepaint()
+            && !context.paintController().cacheIsEmpty()
+            && !interestRectChangedEnoughToRepaint(expandedIntSize(graphicsLayer->size()), m_previousPaintInterestRect, *interestRect)) {
+            context.paintController().createAndAppend<CachedDisplayItem>(*this, DisplayItem::CachedDisplayItemList);
+            return;
+        }
+
+        m_previousPaintInterestRect = *interestRect;
     }
 
-    IntRect interestRect;
-    if (graphicsLayer == m_graphicsLayer || graphicsLayer == m_squashingLayer)
-        interestRect = computeInterestRect(graphicsLayer, m_owningLayer.layoutObject());
-    else
-        interestRect = enclosingIntRect(FloatRect(FloatPoint(), graphicsLayer->size()));
-
-    paintContents(graphicsLayer, context, graphicsLayerPaintingPhase, interestRect);
+    ASSERT(interestRect);
+    paintContentsInternal(graphicsLayer, context, graphicsLayerPaintingPhase, *interestRect);
 }
 
-void CompositedLayerMapping::paintContents(const GraphicsLayer* graphicsLayer, GraphicsContext& context, GraphicsLayerPaintingPhase graphicsLayerPaintingPhase, const IntRect& interestRect) const
+void CompositedLayerMapping::paintContentsInternal(const GraphicsLayer* graphicsLayer, GraphicsContext& context, GraphicsLayerPaintingPhase graphicsLayerPaintingPhase, const IntRect& interestRect) const
 {
     // https://code.google.com/p/chromium/issues/detail?id=343772
     DisableCompositingQueryAsserts disabler;
 #if ENABLE(ASSERT)
     // FIXME: once the state machine is ready, this can be removed and we can refer to that instead.
     if (Page* page = layoutObject()->frame()->page())
         page->setIsPainting(true);
 #endif
 
     TRACE_EVENT1("devtools.timeline", "Paint", "data", InspectorPaintEvent::data(m_owningLayer.layoutObject(), LayoutRect(interestRect), graphicsLayer));
@@ skipping 225 matching lines @@
     } else if (graphicsLayer == m_scrollingBlockSelectionLayer.get()) {
         name = "Scrolling Block Selection Layer";
     } else {
         ASSERT_NOT_REACHED();
     }
 
     return name;
 }
 
 } // namespace blink