Remove unnecessary #include SVGResourcesCache.h in SVGDocumentExtensions.h; use forwa...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 Nov 2011 01:43:16 +0000 (01:43 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 Nov 2011 01:43:16 +0000 (01:43 +0000)
https://bugs.webkit.org/show_bug.cgi?id=72335

Patch by Daniel Bates <dbates@rim.com> on 2011-11-14
Reviewed by Eric Seidel.

It's sufficient to forward declare SVGResourcesCache in SVGDocumentExtensions.h and #include SVGResourcesCache.h
in SVGDocumentExtensions.cpp. This will reduce the number of files we need to re-compile after the file
SVGResourcesCache.h has been modified. Currently we #include SVGResourcesCache.h in SVGDocumentExtensions.h.

* rendering/svg/RenderSVGBlock.cpp: Include SVGResourcesCache.h.
* rendering/svg/RenderSVGContainer.cpp: Ditto.
* rendering/svg/RenderSVGForeignObject.cpp: Ditto.
* rendering/svg/RenderSVGImage.cpp: Ditto.
* rendering/svg/RenderSVGInline.cpp: Ditto.
* rendering/svg/RenderSVGModelObject.cpp: Ditto.
* rendering/svg/RenderSVGPath.cpp: Ditto.
* rendering/svg/RenderSVGResource.cpp: Ditto.
* rendering/svg/RenderSVGResourceClipper.cpp: Ditto.
* rendering/svg/RenderSVGResourceContainer.cpp: Ditto.
* rendering/svg/RenderSVGRoot.cpp: Ditto.
* rendering/svg/RenderSVGText.cpp: Ditto.
* rendering/svg/SVGInlineTextBox.cpp: Ditto.
* rendering/svg/SVGRenderSupport.cpp: Ditto.
* svg/SVGDocumentExtensions.cpp: Ditto.
* svg/SVGDocumentExtensions.h: Forward declare SVGResourcesCache.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@100225 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 files changed:
Source/WebCore/ChangeLog
Source/WebCore/rendering/svg/RenderSVGBlock.cpp
Source/WebCore/rendering/svg/RenderSVGContainer.cpp
Source/WebCore/rendering/svg/RenderSVGForeignObject.cpp
Source/WebCore/rendering/svg/RenderSVGImage.cpp
Source/WebCore/rendering/svg/RenderSVGInline.cpp
Source/WebCore/rendering/svg/RenderSVGModelObject.cpp
Source/WebCore/rendering/svg/RenderSVGPath.cpp
Source/WebCore/rendering/svg/RenderSVGResource.cpp
Source/WebCore/rendering/svg/RenderSVGResourceClipper.cpp
Source/WebCore/rendering/svg/RenderSVGResourceContainer.cpp
Source/WebCore/rendering/svg/RenderSVGRoot.cpp
Source/WebCore/rendering/svg/RenderSVGText.cpp
Source/WebCore/rendering/svg/SVGInlineTextBox.cpp
Source/WebCore/rendering/svg/SVGRenderSupport.cpp
Source/WebCore/svg/SVGDocumentExtensions.cpp
Source/WebCore/svg/SVGDocumentExtensions.h

index 4f650488828dd2e816ae3c185cf130dcb0a140e3..5d26f2de9a0de441a642817aa120f7bccedb03f4 100644 (file)
@@ -1,3 +1,31 @@
+2011-11-14  Daniel Bates  <dbates@rim.com>
+
+        Remove unnecessary #include SVGResourcesCache.h in SVGDocumentExtensions.h; use forward declaration
+        https://bugs.webkit.org/show_bug.cgi?id=72335
+
+        Reviewed by Eric Seidel.
+
+        It's sufficient to forward declare SVGResourcesCache in SVGDocumentExtensions.h and #include SVGResourcesCache.h
+        in SVGDocumentExtensions.cpp. This will reduce the number of files we need to re-compile after the file
+        SVGResourcesCache.h has been modified. Currently we #include SVGResourcesCache.h in SVGDocumentExtensions.h.
+
+        * rendering/svg/RenderSVGBlock.cpp: Include SVGResourcesCache.h.
+        * rendering/svg/RenderSVGContainer.cpp: Ditto.
+        * rendering/svg/RenderSVGForeignObject.cpp: Ditto.
+        * rendering/svg/RenderSVGImage.cpp: Ditto.
+        * rendering/svg/RenderSVGInline.cpp: Ditto.
+        * rendering/svg/RenderSVGModelObject.cpp: Ditto.
+        * rendering/svg/RenderSVGPath.cpp: Ditto.
+        * rendering/svg/RenderSVGResource.cpp: Ditto.
+        * rendering/svg/RenderSVGResourceClipper.cpp: Ditto.
+        * rendering/svg/RenderSVGResourceContainer.cpp: Ditto.
+        * rendering/svg/RenderSVGRoot.cpp: Ditto.
+        * rendering/svg/RenderSVGText.cpp: Ditto.
+        * rendering/svg/SVGInlineTextBox.cpp: Ditto.
+        * rendering/svg/SVGRenderSupport.cpp: Ditto.
+        * svg/SVGDocumentExtensions.cpp: Ditto.
+        * svg/SVGDocumentExtensions.h: Forward declare SVGResourcesCache.
+
 2011-11-14  Rafael Weinstein  <rafaelw@chromium.org>
 
         [MutationObservers] Add histogram collection for usage of DOM Mutation Events
index 7b16a4d5bfc9cb7bb65e65bb15abc8e2f6ebaea3..58f6631fa199405a650d70951a4d15fe0c3c3726 100644 (file)
@@ -26,6 +26,7 @@
 
 #include "RenderSVGResource.h"
 #include "SVGElement.h"
+#include "SVGResourcesCache.h"
 
 namespace WebCore {
 
index e9985ddc811d3007410353ed532b8b827d29ecfa..13bba54411fade75bd84c34bc5af68a7813ba4ac 100644 (file)
@@ -33,6 +33,7 @@
 #include "RenderView.h"
 #include "SVGRenderSupport.h"
 #include "SVGResources.h"
+#include "SVGResourcesCache.h"
 #include "SVGStyledElement.h"
 
 namespace WebCore {
index 8ea7c0bcc012ac2ebdfb2c585ac5498229f11bc7..062c5d6fbd121b325b54fb2efd2f6fe3e4a09747 100644 (file)
@@ -30,6 +30,7 @@
 #include "RenderView.h"
 #include "SVGForeignObjectElement.h"
 #include "SVGRenderSupport.h"
+#include "SVGResourcesCache.h"
 #include "SVGSVGElement.h"
 #include "TransformState.h"
 
index 05448af959f73f892040fc4d7a0dbc4fd1a48df3..fb2359b62c94ca8d232b7783ddedc0e1790e9eb0 100644 (file)
@@ -43,6 +43,7 @@
 #include "SVGPreserveAspectRatio.h"
 #include "SVGRenderSupport.h"
 #include "SVGResources.h"
+#include "SVGResourcesCache.h"
 
 namespace WebCore {
 
index c22244fc6e10e9bfe0aa13d704df7e28763127c2..b08b152d39369f27a3a010f5f09ad384d02958de 100644 (file)
@@ -27,6 +27,7 @@
 #include "RenderSVGResource.h"
 #include "RenderSVGText.h"
 #include "SVGInlineFlowBox.h"
+#include "SVGResourcesCache.h"
 
 namespace WebCore {
     
index 7086d4ce91e6645c1e1c70d83bc4f49c82fccab6..7e09c76feaca7ca1215581689bf350e9ff6ae805 100644 (file)
@@ -34,6 +34,7 @@
 #include "RenderSVGModelObject.h"
 
 #include "RenderSVGResource.h"
+#include "SVGResourcesCache.h"
 #include "SVGStyledElement.h"
 
 namespace WebCore {
index 2b735f02b2beb91c48602b11414ecf24526186e0..3eaad978ade6e5bec4e34333c08d46d52bd36456 100644 (file)
@@ -40,6 +40,7 @@
 #include "SVGPathData.h"
 #include "SVGRenderSupport.h"
 #include "SVGResources.h"
+#include "SVGResourcesCache.h"
 #include "SVGStyledTransformableElement.h"
 #include "SVGTransformList.h"
 #include "SVGURIReference.h"
index a1394988b8da93ab139403bfa0907afcf1fe6ed9..683f0a25e7979e932fc73e47722bb798f5d788e3 100644 (file)
@@ -28,6 +28,7 @@
 #include "RenderSVGResourceContainer.h"
 #include "RenderSVGResourceSolidColor.h"
 #include "SVGResources.h"
+#include "SVGResourcesCache.h"
 #include "SVGURIReference.h"
 
 namespace WebCore {
index 1be58a54534c1856efa9b90df60ae0b8cb23953c..77aaae5ea340f4be26c8a0c1bff01b12ad2399db 100644 (file)
@@ -41,6 +41,7 @@
 #include "SVGNames.h"
 #include "SVGRenderSupport.h"
 #include "SVGResources.h"
+#include "SVGResourcesCache.h"
 #include "SVGStyledElement.h"
 #include "SVGStyledTransformableElement.h"
 #include "SVGUnitTypes.h"
index 5f50d1c94fce722bf929fe9af9b75f39fdbcc6ae..29d35e9c8f24fad8361ee11256eb8c7eb658c8c9 100644 (file)
@@ -23,6 +23,7 @@
 #include "RenderSVGResourceContainer.h"
 
 #include "RenderSVGShadowTreeRootContainer.h"
+#include "SVGResourcesCache.h"
 #include "SVGStyledTransformableElement.h"
 
 namespace WebCore {
index dc90e88c86e6d01d949dca76b84ac063517cd086..21f69888358c7020dfd6ebdf877c0dfb09e71c88 100644 (file)
@@ -40,6 +40,7 @@
 #include "SVGLength.h"
 #include "SVGRenderSupport.h"
 #include "SVGResources.h"
+#include "SVGResourcesCache.h"
 #include "SVGSVGElement.h"
 #include "SVGStyledElement.h"
 #include "SVGViewSpec.h"
index 312f5aa940c81dd81503c06f28a4a8b51723057e..25a8dcb0f757d4f63f5041ed231ae01772c64957 100644 (file)
@@ -39,6 +39,7 @@
 #include "RenderSVGRoot.h"
 #include "SVGLengthList.h"
 #include "SVGRenderSupport.h"
+#include "SVGResourcesCache.h"
 #include "SVGRootInlineBox.h"
 #include "SVGTextElement.h"
 #include "SVGTextLayoutAttributesBuilder.h"
index c1c5afa2543ae63329b85827ca9777dfa35b6123..b243df978b45ecacbabf7167304d3132eabf1c29 100644 (file)
@@ -34,6 +34,7 @@
 #include "RenderSVGResource.h"
 #include "RenderSVGResourceSolidColor.h"
 #include "SVGImageBufferTools.h"
+#include "SVGResourcesCache.h"
 #include "SVGRootInlineBox.h"
 #include "SVGTextRunRenderingContext.h"
 
index 42455b7cf12d08594a4fc2d2d05016c4d35957b8..57209aafb8add070a9f6918b034d5d49ce72ae90 100644 (file)
@@ -39,6 +39,7 @@
 #include "RenderSVGResourceMasker.h"
 #include "RenderSVGRoot.h"
 #include "SVGResources.h"
+#include "SVGResourcesCache.h"
 #include "SVGStyledElement.h"
 #include "TransformState.h"
 #include <wtf/UnusedParam.h>
index bb073f8b106377390b1fd40fb2d99b5832481515..d747e163a7db3721f9aacd3abca32ecc83eac466 100644 (file)
@@ -33,6 +33,7 @@
 #include "Page.h"
 #include "SMILTimeContainer.h"
 #include "SVGElement.h"
+#include "SVGResourcesCache.h"
 #include "SVGSMILElement.h"
 #include "SVGSVGElement.h"
 #include "ScriptableDocumentParser.h"
index 8ad44f85597313b605cd86cb0cd9b4fa3e958ccb..4ab5f14e04b30c9734e44af70e45735c8263586e 100644 (file)
@@ -22,7 +22,6 @@
 #define SVGDocumentExtensions_h
 
 #if ENABLE(SVG)
-#include "SVGResourcesCache.h"
 #include <wtf/Forward.h>
 #include <wtf/HashMap.h>
 #include <wtf/HashSet.h>
@@ -35,9 +34,10 @@ namespace WebCore {
 class Document;
 class RenderSVGResourceContainer;
 class SVGElement;
-class SVGStyledElement;
+class SVGResourcesCache;
 class SVGSMILElement;
 class SVGSVGElement;
+class SVGStyledElement;
 
 class SVGDocumentExtensions {
     WTF_MAKE_NONCOPYABLE(SVGDocumentExtensions); WTF_MAKE_FAST_ALLOCATED;