FilterOperation.h should not include FilterEffect.h
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Aug 2015 19:08:42 +0000 (19:08 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Aug 2015 19:08:42 +0000 (19:08 +0000)
https://bugs.webkit.org/show_bug.cgi?id=147970

Reviewed by Daniel Bates.

FilterEffect.h pulls in lots of JSC goop via runtime/Uint8ClampedArray.h,
so move its include to FilterOperation.cpp.

Causes include bloat because FilterOperation.h is pulled in via RenderStyle.h.

Source/WebCore:

* platform/graphics/filters/FilterOperation.cpp:
(WebCore::ReferenceFilterOperation::setFilterEffect):
* platform/graphics/filters/FilterOperation.h:
(WebCore::ReferenceFilterOperation::setFilterEffect): Deleted.

Source/WebKit2:

* UIProcess/ios/WebVideoFullscreenManagerProxy.h:

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

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/filters/FilterOperation.cpp
Source/WebCore/platform/graphics/filters/FilterOperation.h
Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/ios/WebVideoFullscreenManagerProxy.h

index b6f1e3a..f362b49 100644 (file)
@@ -1,5 +1,22 @@
 2015-08-13  Simon Fraser  <simon.fraser@apple.com>
 
+        FilterOperation.h should not include FilterEffect.h
+        https://bugs.webkit.org/show_bug.cgi?id=147970
+
+        Reviewed by Daniel Bates.
+
+        FilterEffect.h pulls in lots of JSC goop via runtime/Uint8ClampedArray.h,
+        so move its include to FilterOperation.cpp.
+        
+        Causes include bloat because FilterOperation.h is pulled in via RenderStyle.h.
+
+        * platform/graphics/filters/FilterOperation.cpp:
+        (WebCore::ReferenceFilterOperation::setFilterEffect):
+        * platform/graphics/filters/FilterOperation.h:
+        (WebCore::ReferenceFilterOperation::setFilterEffect): Deleted.
+
+2015-08-13  Simon Fraser  <simon.fraser@apple.com>
+
         ScriptExecutionContext.h pulls in all the JSC headers
         https://bugs.webkit.org/show_bug.cgi?id=147969
 
index b189e14..1a210c5 100644 (file)
@@ -28,6 +28,7 @@
 
 #include "AnimationUtilities.h"
 #include "CachedSVGDocumentReference.h"
+#include "FilterEffect.h"
 
 namespace WebCore {
     
@@ -65,6 +66,11 @@ CachedSVGDocumentReference* ReferenceFilterOperation::getOrCreateCachedSVGDocume
     return m_cachedSVGDocumentReference.get();
 }
 
+void ReferenceFilterOperation::setFilterEffect(PassRefPtr<FilterEffect> filterEffect)
+{
+    m_filterEffect = filterEffect;
+}
+
 PassRefPtr<FilterOperation> BasicColorMatrixFilterOperation::blend(const FilterOperation* from, double progress, bool blendToPassthrough)
 {
     if (from && !from->isSameType(*this))
index dfb746a..22d840f 100644 (file)
@@ -27,7 +27,6 @@
 #define FilterOperation_h
 
 #include "Color.h"
-#include "FilterEffect.h"
 #include "LayoutSize.h"
 #include "Length.h"
 #include <wtf/RefCounted.h>
@@ -44,6 +43,7 @@ namespace WebCore {
 // CSS Filters
 
 class CachedSVGDocumentReference;
+class FilterEffect;
 
 class FilterOperation : public RefCounted<FilterOperation> {
 public:
@@ -188,7 +188,7 @@ public:
     CachedSVGDocumentReference* getOrCreateCachedSVGDocumentReference();
 
     FilterEffect* filterEffect() const { return m_filterEffect.get(); }
-    void setFilterEffect(PassRefPtr<FilterEffect> filterEffect) { m_filterEffect = filterEffect; }
+    void setFilterEffect(PassRefPtr<FilterEffect>);
 
 private:
     ReferenceFilterOperation(const String& url, const String& fragment);
index e357dad..0ee969a 100644 (file)
@@ -1,3 +1,17 @@
+2015-08-13  Simon Fraser  <simon.fraser@apple.com>
+
+        FilterOperation.h should not include FilterEffect.h
+        https://bugs.webkit.org/show_bug.cgi?id=147970
+
+        Reviewed by Daniel Bates.
+
+        FilterEffect.h pulls in lots of JSC goop via runtime/Uint8ClampedArray.h,
+        so move its include to FilterOperation.cpp.
+        
+        Causes include bloat because FilterOperation.h is pulled in via RenderStyle.h.
+
+        * UIProcess/ios/WebVideoFullscreenManagerProxy.h:
+
 2015-08-12  Anders Carlsson  <andersca@apple.com>
 
         Use WTF::Optional in WindowFeatures
index 4f719fb..7e1e0d9 100644 (file)
@@ -32,6 +32,7 @@
 #include <WebCore/GraphicsLayer.h>
 #include <WebCore/WebVideoFullscreenInterfaceAVKit.h>
 #include <WebCore/WebVideoFullscreenModel.h>
+#include <wtf/HashMap.h>
 #include <wtf/PassRefPtr.h>
 #include <wtf/RefCounted.h>
 #include <wtf/RefPtr.h>