Drop FilterEffect::m_absolutePaintRect

third_party/WebKit/Source/platform/graphics/filters/FEComposite.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005 Rob Buis <buis@kde.org>
  * Copyright (C) 2005 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 95 matching lines @@
 }
 
 bool FEComposite::setK4(float k4)
 {
     if (m_k4 == k4)
         return false;
     m_k4 = k4;
     return true;
 }
 
-FloatRect FEComposite::determineAbsolutePaintRect(const FloatRect& originalRequestedRect)
+FloatRect FEComposite::determineAbsolutePaintRect(const FloatRect& originalRequestedRect) const
 {
     FloatRect requestedRect = originalRequestedRect;
     if (clipsToBounds())
         requestedRect.intersect(absoluteBounds());
 
-    // We may be called multiple times if result is used more than once. Return
-    // quickly if nothing new is required.
-    if (absolutePaintRect().contains(enclosingIntRect(requestedRect)))
-        return requestedRect;
-
     // No mapPaintRect required for FEComposite.
     FloatRect input1Rect = inputEffect(1)->determineAbsolutePaintRect(requestedRect);
     FloatRect affectedRect;
     switch (m_type) {
     case FECOMPOSITE_OPERATOR_IN:
         // 'in' has output only in the intersection of both inputs.
         affectedRect = intersection(input1Rect, inputEffect(0)->determineAbsolutePaintRect(input1Rect));
         break;
     case FECOMPOSITE_OPERATOR_ATOP:
         // 'atop' has output only in the extents of the second input.
@@ skipping 22 matching lines @@
             break;
         }
         // else fall through to use union
     default:
         // Take the union of both input effects.
         affectedRect = unionRect(input1Rect, inputEffect(0)->determineAbsolutePaintRect(requestedRect));
         break;
     }
 
     affectedRect.intersect(requestedRect);
-    addAbsolutePaintRect(affectedRect);
     return affectedRect;
 }
 
 SkXfermode::Mode toXfermode(CompositeOperationType mode)
 {
     switch (mode) {
     case FECOMPOSITE_OPERATOR_OVER:
         return SkXfermode::kSrcOver_Mode;
     case FECOMPOSITE_OPERATOR_IN:
         return SkXfermode::kSrcIn_Mode;
@@ skipping 73 matching lines @@
     ts << " operation=\"" << m_type << "\"";
     if (m_type == FECOMPOSITE_OPERATOR_ARITHMETIC)
         ts << " k1=\"" << m_k1 << "\" k2=\"" << m_k2 << "\" k3=\"" << m_k3 << "\" k4=\"" << m_k4 << "\"";
     ts << "]\n";
     inputEffect(0)->externalRepresentation(ts, indent + 1);
     inputEffect(1)->externalRepresentation(ts, indent + 1);
     return ts;
 }
 
 } // namespace blink