Commit 07ca63d5 authored by fs's avatar fs Committed by Commit bot

Clean up includes in SVGFE*Element and thereabouts

Drop inclusion of SVGParserUtilities.h in a number of places since it's
no longer used. Push FE*.h inclusion into the implementation where
possible, and don't include FilterEffect.h in those cases.
Also simplify a loop in SVGFEMergeElement::build.

Review-Url: https://codereview.chromium.org/2342353002
Cr-Commit-Position: refs/heads/master@{#419187}
parent ac73140e
...@@ -23,7 +23,6 @@ ...@@ -23,7 +23,6 @@
#include "core/SVGNames.h" #include "core/SVGNames.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FEBlend.h" #include "platform/graphics/filters/FEBlend.h"
#include "platform/graphics/filters/FilterEffect.h"
namespace blink { namespace blink {
......
...@@ -22,7 +22,6 @@ ...@@ -22,7 +22,6 @@
#include "core/SVGNames.h" #include "core/SVGNames.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FilterEffect.h"
namespace blink { namespace blink {
......
...@@ -27,7 +27,7 @@ ...@@ -27,7 +27,7 @@
#include "core/svg/SVGFEFuncGElement.h" #include "core/svg/SVGFEFuncGElement.h"
#include "core/svg/SVGFEFuncRElement.h" #include "core/svg/SVGFEFuncRElement.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FilterEffect.h" #include "platform/graphics/filters/FEComponentTransfer.h"
namespace blink { namespace blink {
......
...@@ -22,7 +22,6 @@ ...@@ -22,7 +22,6 @@
#define SVGFEComponentTransferElement_h #define SVGFEComponentTransferElement_h
#include "core/svg/SVGFilterPrimitiveStandardAttributes.h" #include "core/svg/SVGFilterPrimitiveStandardAttributes.h"
#include "platform/graphics/filters/FEComponentTransfer.h"
#include "platform/heap/Handle.h" #include "platform/heap/Handle.h"
namespace blink { namespace blink {
......
...@@ -22,7 +22,6 @@ ...@@ -22,7 +22,6 @@
#include "core/SVGNames.h" #include "core/SVGNames.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FilterEffect.h"
namespace blink { namespace blink {
......
...@@ -22,10 +22,8 @@ ...@@ -22,10 +22,8 @@
#include "core/SVGNames.h" #include "core/SVGNames.h"
#include "core/dom/Document.h" #include "core/dom/Document.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/geometry/FloatPoint.h"
#include "platform/geometry/IntPoint.h" #include "platform/geometry/IntPoint.h"
#include "platform/geometry/IntSize.h" #include "platform/geometry/IntSize.h"
#include "platform/graphics/filters/FilterEffect.h"
namespace blink { namespace blink {
......
...@@ -21,10 +21,9 @@ ...@@ -21,10 +21,9 @@
#include "core/layout/LayoutObject.h" #include "core/layout/LayoutObject.h"
#include "core/style/ComputedStyle.h" #include "core/style/ComputedStyle.h"
#include "core/svg/SVGParserUtilities.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FEDiffuseLighting.h" #include "platform/graphics/filters/FEDiffuseLighting.h"
#include "platform/graphics/filters/FilterEffect.h" #include "platform/graphics/filters/Filter.h"
namespace blink { namespace blink {
......
...@@ -21,7 +21,6 @@ ...@@ -21,7 +21,6 @@
#include "core/SVGNames.h" #include "core/SVGNames.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FilterEffect.h"
namespace blink { namespace blink {
......
...@@ -23,8 +23,8 @@ ...@@ -23,8 +23,8 @@
#include "core/layout/LayoutObject.h" #include "core/layout/LayoutObject.h"
#include "core/style/ComputedStyle.h" #include "core/style/ComputedStyle.h"
#include "core/style/SVGComputedStyle.h" #include "core/style/SVGComputedStyle.h"
#include "core/svg/SVGParserUtilities.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FEDropShadow.h"
namespace blink { namespace blink {
......
...@@ -23,7 +23,6 @@ ...@@ -23,7 +23,6 @@
#include "core/svg/SVGAnimatedNumber.h" #include "core/svg/SVGAnimatedNumber.h"
#include "core/svg/SVGAnimatedNumberOptionalNumber.h" #include "core/svg/SVGAnimatedNumberOptionalNumber.h"
#include "core/svg/SVGFilterPrimitiveStandardAttributes.h" #include "core/svg/SVGFilterPrimitiveStandardAttributes.h"
#include "platform/graphics/filters/FEDropShadow.h"
#include "platform/heap/Handle.h" #include "platform/heap/Handle.h"
namespace blink { namespace blink {
......
...@@ -24,6 +24,7 @@ ...@@ -24,6 +24,7 @@
#include "core/layout/LayoutObject.h" #include "core/layout/LayoutObject.h"
#include "core/style/ComputedStyle.h" #include "core/style/ComputedStyle.h"
#include "core/style/SVGComputedStyle.h" #include "core/style/SVGComputedStyle.h"
#include "platform/graphics/filters/FEFlood.h"
namespace blink { namespace blink {
......
...@@ -22,7 +22,6 @@ ...@@ -22,7 +22,6 @@
#define SVGFEFloodElement_h #define SVGFEFloodElement_h
#include "core/svg/SVGFilterPrimitiveStandardAttributes.h" #include "core/svg/SVGFilterPrimitiveStandardAttributes.h"
#include "platform/graphics/filters/FEFlood.h"
namespace blink { namespace blink {
......
...@@ -21,9 +21,8 @@ ...@@ -21,9 +21,8 @@
#include "core/svg/SVGFEGaussianBlurElement.h" #include "core/svg/SVGFEGaussianBlurElement.h"
#include "core/SVGNames.h" #include "core/SVGNames.h"
#include "core/svg/SVGParserUtilities.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FilterEffect.h" #include "platform/graphics/filters/FEGaussianBlur.h"
namespace blink { namespace blink {
......
...@@ -23,7 +23,6 @@ ...@@ -23,7 +23,6 @@
#include "core/svg/SVGAnimatedNumberOptionalNumber.h" #include "core/svg/SVGAnimatedNumberOptionalNumber.h"
#include "core/svg/SVGFilterPrimitiveStandardAttributes.h" #include "core/svg/SVGFilterPrimitiveStandardAttributes.h"
#include "platform/graphics/filters/FEGaussianBlur.h"
#include "platform/heap/Handle.h" #include "platform/heap/Handle.h"
namespace blink { namespace blink {
......
...@@ -24,7 +24,7 @@ ...@@ -24,7 +24,7 @@
#include "core/dom/ElementTraversal.h" #include "core/dom/ElementTraversal.h"
#include "core/svg/SVGFEMergeNodeElement.h" #include "core/svg/SVGFEMergeNodeElement.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FilterEffect.h" #include "platform/graphics/filters/FEMerge.h"
namespace blink { namespace blink {
...@@ -39,8 +39,8 @@ FilterEffect* SVGFEMergeElement::build(SVGFilterBuilder* filterBuilder, Filter* ...@@ -39,8 +39,8 @@ FilterEffect* SVGFEMergeElement::build(SVGFilterBuilder* filterBuilder, Filter*
{ {
FilterEffect* effect = FEMerge::create(filter); FilterEffect* effect = FEMerge::create(filter);
FilterEffectVector& mergeInputs = effect->inputEffects(); FilterEffectVector& mergeInputs = effect->inputEffects();
for (SVGFEMergeNodeElement* element = Traversal<SVGFEMergeNodeElement>::firstChild(*this); element; element = Traversal<SVGFEMergeNodeElement>::nextSibling(*element)) { for (SVGFEMergeNodeElement& mergeNode : Traversal<SVGFEMergeNodeElement>::childrenOf(*this)) {
FilterEffect* mergeEffect = filterBuilder->getEffectById(AtomicString(element->in1()->currentValue()->value())); FilterEffect* mergeEffect = filterBuilder->getEffectById(AtomicString(mergeNode.in1()->currentValue()->value()));
ASSERT(mergeEffect); ASSERT(mergeEffect);
mergeInputs.append(mergeEffect); mergeInputs.append(mergeEffect);
} }
......
...@@ -22,7 +22,6 @@ ...@@ -22,7 +22,6 @@
#define SVGFEMergeElement_h #define SVGFEMergeElement_h
#include "core/svg/SVGFilterPrimitiveStandardAttributes.h" #include "core/svg/SVGFilterPrimitiveStandardAttributes.h"
#include "platform/graphics/filters/FEMerge.h"
namespace blink { namespace blink {
......
...@@ -20,9 +20,7 @@ ...@@ -20,9 +20,7 @@
#include "core/svg/SVGFEMorphologyElement.h" #include "core/svg/SVGFEMorphologyElement.h"
#include "core/SVGNames.h" #include "core/SVGNames.h"
#include "core/svg/SVGParserUtilities.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FilterEffect.h"
namespace blink { namespace blink {
......
...@@ -22,7 +22,7 @@ ...@@ -22,7 +22,7 @@
#include "core/SVGNames.h" #include "core/SVGNames.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FilterEffect.h" #include "platform/graphics/filters/FEOffset.h"
namespace blink { namespace blink {
......
...@@ -23,7 +23,6 @@ ...@@ -23,7 +23,6 @@
#include "core/svg/SVGAnimatedNumber.h" #include "core/svg/SVGAnimatedNumber.h"
#include "core/svg/SVGFilterPrimitiveStandardAttributes.h" #include "core/svg/SVGFilterPrimitiveStandardAttributes.h"
#include "platform/graphics/filters/FEOffset.h"
#include "platform/heap/Handle.h" #include "platform/heap/Handle.h"
namespace blink { namespace blink {
......
...@@ -23,9 +23,9 @@ ...@@ -23,9 +23,9 @@
#include "core/layout/LayoutObject.h" #include "core/layout/LayoutObject.h"
#include "core/style/ComputedStyle.h" #include "core/style/ComputedStyle.h"
#include "core/svg/SVGParserUtilities.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FilterEffect.h" #include "platform/graphics/filters/FESpecularLighting.h"
#include "platform/graphics/filters/Filter.h"
namespace blink { namespace blink {
......
...@@ -27,7 +27,6 @@ ...@@ -27,7 +27,6 @@
#include "core/svg/SVGAnimatedNumberOptionalNumber.h" #include "core/svg/SVGAnimatedNumberOptionalNumber.h"
#include "core/svg/SVGFELightElement.h" #include "core/svg/SVGFELightElement.h"
#include "core/svg/SVGFilterPrimitiveStandardAttributes.h" #include "core/svg/SVGFilterPrimitiveStandardAttributes.h"
#include "platform/graphics/filters/FESpecularLighting.h"
#include "platform/heap/Handle.h" #include "platform/heap/Handle.h"
namespace blink { namespace blink {
......
...@@ -22,7 +22,7 @@ ...@@ -22,7 +22,7 @@
#include "core/SVGNames.h" #include "core/SVGNames.h"
#include "core/svg/graphics/filters/SVGFilterBuilder.h" #include "core/svg/graphics/filters/SVGFilterBuilder.h"
#include "platform/graphics/filters/FilterEffect.h" #include "platform/graphics/filters/FETile.h"
namespace blink { namespace blink {
......
...@@ -22,7 +22,6 @@ ...@@ -22,7 +22,6 @@
#define SVGFETileElement_h #define SVGFETileElement_h
#include "core/svg/SVGFilterPrimitiveStandardAttributes.h" #include "core/svg/SVGFilterPrimitiveStandardAttributes.h"
#include "platform/graphics/filters/FETile.h"
#include "platform/heap/Handle.h" #include "platform/heap/Handle.h"
namespace blink { namespace blink {
......
...@@ -21,7 +21,6 @@ ...@@ -21,7 +21,6 @@
#include "core/svg/SVGFETurbulenceElement.h" #include "core/svg/SVGFETurbulenceElement.h"
#include "core/SVGNames.h" #include "core/SVGNames.h"
#include "core/svg/SVGParserUtilities.h"
namespace blink { namespace blink {
......
...@@ -25,7 +25,6 @@ ...@@ -25,7 +25,6 @@
#include "core/frame/UseCounter.h" #include "core/frame/UseCounter.h"
#include "core/layout/svg/LayoutSVGResourceFilter.h" #include "core/layout/svg/LayoutSVGResourceFilter.h"
#include "core/svg/SVGParserUtilities.h"
#include "core/svg/SVGResourceClient.h" #include "core/svg/SVGResourceClient.h"
namespace blink { namespace blink {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment