Title: [143499] trunk/Source
Revision
143499
Author
[email protected]
Date
2013-02-20 13:45:56 -0800 (Wed, 20 Feb 2013)

Log Message

Remove unnecessary includes for Platform.h from headers
https://bugs.webkit.org/show_bug.cgi?id=110300

Patch by Laszlo Gombos <[email protected]> on 2013-02-20
Reviewed by Alexey Proskuryakov.

Source/WebCore:

wtf/Platform.h only need to be included in a few places directly
as it is included in config.h. Remove the unnecessay duplicate as
it can be confusing.

No new tests, no change in functionality.

* platform/SecureTextInput.h:
* platform/audio/Biquad.h:
* platform/audio/FFTFrame.h:
* platform/graphics/cpu/arm/filters/FEBlendNEON.h:
* platform/graphics/cpu/arm/filters/NEONHelpers.h:
* platform/graphics/filters/FELighting.h:

Source/WebKit2:

wtf/Platform.h only need to be included in a few places directly
as it is included in config.h. Remove the unnecessay duplicate as
it can be confusing.

* Shared/PrintInfo.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (143498 => 143499)


--- trunk/Source/WebCore/ChangeLog	2013-02-20 21:43:29 UTC (rev 143498)
+++ trunk/Source/WebCore/ChangeLog	2013-02-20 21:45:56 UTC (rev 143499)
@@ -1,3 +1,23 @@
+2013-02-20  Laszlo Gombos  <[email protected]>
+
+        Remove unnecessary includes for Platform.h from headers
+        https://bugs.webkit.org/show_bug.cgi?id=110300
+
+        Reviewed by Alexey Proskuryakov.
+
+        wtf/Platform.h only need to be included in a few places directly
+        as it is included in config.h. Remove the unnecessay duplicate as
+        it can be confusing.
+
+        No new tests, no change in functionality.
+
+        * platform/SecureTextInput.h:
+        * platform/audio/Biquad.h:
+        * platform/audio/FFTFrame.h:
+        * platform/graphics/cpu/arm/filters/FEBlendNEON.h:
+        * platform/graphics/cpu/arm/filters/NEONHelpers.h:
+        * platform/graphics/filters/FELighting.h:
+
 2013-02-20  Florin Malita  <[email protected]>
 
         [SVG] Update of element referenced by multiple 'use' nodes is absurdly slow

Modified: trunk/Source/WebCore/platform/SecureTextInput.h (143498 => 143499)


--- trunk/Source/WebCore/platform/SecureTextInput.h	2013-02-20 21:43:29 UTC (rev 143498)
+++ trunk/Source/WebCore/platform/SecureTextInput.h	2013-02-20 21:45:56 UTC (rev 143499)
@@ -31,8 +31,6 @@
 #ifndef SecureTextInput_h
 #define SecureTextInput_h
 
-#include <wtf/Platform.h>
-
 namespace WebCore {
 
 // Once enableSecureTextInput is called, secure text input mode is set until

Modified: trunk/Source/WebCore/platform/audio/Biquad.h (143498 => 143499)


--- trunk/Source/WebCore/platform/audio/Biquad.h	2013-02-20 21:43:29 UTC (rev 143498)
+++ trunk/Source/WebCore/platform/audio/Biquad.h	2013-02-20 21:45:56 UTC (rev 143499)
@@ -32,7 +32,6 @@
 #include "AudioArray.h"
 #include <sys/types.h>
 #include <wtf/Complex.h>
-#include <wtf/Platform.h>
  
 #if USE(WEBAUDIO_IPP)
 #include <ipps.h>

Modified: trunk/Source/WebCore/platform/audio/FFTFrame.h (143498 => 143499)


--- trunk/Source/WebCore/platform/audio/FFTFrame.h	2013-02-20 21:43:29 UTC (rev 143498)
+++ trunk/Source/WebCore/platform/audio/FFTFrame.h	2013-02-20 21:45:56 UTC (rev 143499)
@@ -66,7 +66,6 @@
 
 #include <wtf/Forward.h>
 #include <wtf/PassOwnPtr.h>
-#include <wtf/Platform.h>
 #include <wtf/Threading.h>
 
 namespace WebCore {

Modified: trunk/Source/WebCore/platform/graphics/cpu/arm/filters/FEBlendNEON.h (143498 => 143499)


--- trunk/Source/WebCore/platform/graphics/cpu/arm/filters/FEBlendNEON.h	2013-02-20 21:43:29 UTC (rev 143498)
+++ trunk/Source/WebCore/platform/graphics/cpu/arm/filters/FEBlendNEON.h	2013-02-20 21:45:56 UTC (rev 143499)
@@ -27,8 +27,6 @@
 #ifndef FEBlendNEON_h
 #define FEBlendNEON_h
 
-#include <wtf/Platform.h>
-
 #if ENABLE(FILTERS) && HAVE(ARM_NEON_INTRINSICS)
 
 #include "FEBlend.h"

Modified: trunk/Source/WebCore/platform/graphics/cpu/arm/filters/NEONHelpers.h (143498 => 143499)


--- trunk/Source/WebCore/platform/graphics/cpu/arm/filters/NEONHelpers.h	2013-02-20 21:43:29 UTC (rev 143498)
+++ trunk/Source/WebCore/platform/graphics/cpu/arm/filters/NEONHelpers.h	2013-02-20 21:45:56 UTC (rev 143499)
@@ -27,8 +27,6 @@
 #ifndef NEONHelpers_h
 #define NEONHelpers_h
 
-#include <wtf/Platform.h>
-
 #if ENABLE(FILTERS) && HAVE(ARM_NEON_INTRINSICS)
 
 #include <arm_neon.h>

Modified: trunk/Source/WebCore/platform/graphics/filters/FELighting.h (143498 => 143499)


--- trunk/Source/WebCore/platform/graphics/filters/FELighting.h	2013-02-20 21:43:29 UTC (rev 143498)
+++ trunk/Source/WebCore/platform/graphics/filters/FELighting.h	2013-02-20 21:45:56 UTC (rev 143499)
@@ -34,7 +34,6 @@
 #include "LightSource.h"
 #include "PointLightSource.h"
 #include "SpotLightSource.h"
-#include <wtf/Platform.h>
 #include <wtf/Uint8ClampedArray.h>
 
 // Common base class for FEDiffuseLighting and FESpecularLighting

Modified: trunk/Source/WebKit2/ChangeLog (143498 => 143499)


--- trunk/Source/WebKit2/ChangeLog	2013-02-20 21:43:29 UTC (rev 143498)
+++ trunk/Source/WebKit2/ChangeLog	2013-02-20 21:45:56 UTC (rev 143499)
@@ -1,3 +1,16 @@
+2013-02-20  Laszlo Gombos  <[email protected]>
+
+        Remove unnecessary includes for Platform.h from headers
+        https://bugs.webkit.org/show_bug.cgi?id=110300
+
+        Reviewed by Alexey Proskuryakov.
+
+        wtf/Platform.h only need to be included in a few places directly    
+        as it is included in config.h. Remove the unnecessay duplicate as    
+        it can be confusing.
+
+        * Shared/PrintInfo.h:
+
 2013-02-20  Huang Dongsung  <[email protected]>
 
         [WK2] Remove m_viewportSize in WebPage.

Modified: trunk/Source/WebKit2/Shared/PrintInfo.h (143498 => 143499)


--- trunk/Source/WebKit2/Shared/PrintInfo.h	2013-02-20 21:43:29 UTC (rev 143498)
+++ trunk/Source/WebKit2/Shared/PrintInfo.h	2013-02-20 21:45:56 UTC (rev 143499)
@@ -26,8 +26,6 @@
 #ifndef PrintInfo_h
 #define PrintInfo_h
 
-#include <wtf/Platform.h>
-
 #if PLATFORM(MAC)
 OBJC_CLASS NSPrintInfo;
 #elif PLATFORM(GTK)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to