Move application of filter effect boundaries to a helper

third_party/WebKit/Source/platform/graphics/filters/FilterEffect.h

 /*
  * Copyright (C) 2008 Alex Mathews <possessedpenguinbob@gmail.com>
  * 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
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 106 matching lines @@
     void setHasWidth(bool value) { m_hasWidth = value; }
 
     bool hasHeight() const { return m_hasHeight; }
     void setHasHeight(bool value) { m_hasHeight = value; }
 
     FloatRect filterPrimitiveSubregion() const { return m_filterPrimitiveSubregion; }
     void setFilterPrimitiveSubregion(const FloatRect& filterPrimitiveSubregion) { m_filterPrimitiveSubregion = filterPrimitiveSubregion; }
 
     FloatRect effectBoundaries() const { return m_effectBoundaries; }
     void setEffectBoundaries(const FloatRect& effectBoundaries) { m_effectBoundaries = effectBoundaries; }
+    void applyEffectBoundaries(FloatRect&) const;

[Stephen White, 2015/11/19 14:54:34]

I'm not a fan of non-const-refs as arguments, since it's not clear from
the callsite that the argument is modified. Could you make this 
take a const ref and return the new FloatRect? This would also make it 
more in line with mapRect() and friends.

[fs, 2015/11/19 15:48:09]

Done.

 
     Filter* filter() { return m_filter; }
     const Filter* filter() const { return m_filter; }
 
     bool clipsToBounds() const { return m_clipsToBounds; }
     void setClipsToBounds(bool value) { m_clipsToBounds = value; }
 
     ColorSpace operatingColorSpace() const { return m_operatingColorSpace; }
     virtual void setOperatingColorSpace(ColorSpace colorSpace) { m_operatingColorSpace = colorSpace; }
 
@@ skipping 51 matching lines @@
     bool m_clipsToBounds;
 
     ColorSpace m_operatingColorSpace;
 
     RefPtr<SkImageFilter> m_imageFilters[4];
 };
 
 } // namespace blink
 
 #endif // FilterEffect_h