Title: [125734] trunk/Source/WebCore
Revision
125734
Author
[email protected]
Date
2012-08-15 18:41:37 -0700 (Wed, 15 Aug 2012)

Log Message

Improve Document.h compile time - reduce includes of ScriptCallStack.h
https://bugs.webkit.org/show_bug.cgi?id=94161

Patch by Nikhil Bhargava <[email protected]> on 2012-08-15
Reviewed by Eric Seidel.

ScriptCallStack.h no longer included from ScriptExecutionContext. It's
relatively expensive to compile and gets compiled a ton because
Document.h indirectly includes it.

No new tests. Functionality should remain the same

* Modules/indexeddb/IDBDatabase.cpp:
* Modules/indexeddb/IDBIndex.cpp:
* Modules/indexeddb/IDBObjectStore.cpp:
* bindings/js/ScriptController.cpp:
* dom/ScriptElement.cpp:
* dom/ScriptExecutionContext.h:
(WebCore):
* fileapi/Blob.cpp:
* fileapi/WebKitBlobBuilder.cpp:
* html/HTMLTrackElement.cpp:
* loader/TextTrackLoader.cpp:
* page/DOMSecurityPolicy.cpp:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (125733 => 125734)


--- trunk/Source/WebCore/ChangeLog	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/ChangeLog	2012-08-16 01:41:37 UTC (rev 125734)
@@ -1,3 +1,29 @@
+2012-08-15  Nikhil Bhargava  <[email protected]>
+
+        Improve Document.h compile time - reduce includes of ScriptCallStack.h
+        https://bugs.webkit.org/show_bug.cgi?id=94161
+
+        Reviewed by Eric Seidel.
+
+        ScriptCallStack.h no longer included from ScriptExecutionContext. It's
+        relatively expensive to compile and gets compiled a ton because
+        Document.h indirectly includes it.
+
+        No new tests. Functionality should remain the same
+
+        * Modules/indexeddb/IDBDatabase.cpp:
+        * Modules/indexeddb/IDBIndex.cpp:
+        * Modules/indexeddb/IDBObjectStore.cpp:
+        * bindings/js/ScriptController.cpp:
+        * dom/ScriptElement.cpp:
+        * dom/ScriptExecutionContext.h:
+        (WebCore):
+        * fileapi/Blob.cpp:
+        * fileapi/WebKitBlobBuilder.cpp:
+        * html/HTMLTrackElement.cpp:
+        * loader/TextTrackLoader.cpp:
+        * page/DOMSecurityPolicy.cpp:
+
 2012-08-15  Alec Flett  <[email protected]>
 
         IndexedDB: IDB*::keyPath should return IDBKeyPath, not IDBAny

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBDatabase.cpp (125733 => 125734)


--- trunk/Source/WebCore/Modules/indexeddb/IDBDatabase.cpp	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBDatabase.cpp	2012-08-16 01:41:37 UTC (rev 125734)
@@ -43,6 +43,7 @@
 #include "IDBTransaction.h"
 #include "IDBVersionChangeEvent.h"
 #include "IDBVersionChangeRequest.h"
+#include "ScriptCallStack.h"
 #include "ScriptExecutionContext.h"
 #include <limits>
 

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBIndex.cpp (125733 => 125734)


--- trunk/Source/WebCore/Modules/indexeddb/IDBIndex.cpp	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBIndex.cpp	2012-08-16 01:41:37 UTC (rev 125734)
@@ -37,6 +37,7 @@
 #include "IDBRequest.h"
 #include "IDBTracing.h"
 #include "IDBTransaction.h"
+#include "ScriptCallStack.h"
 
 namespace WebCore {
 

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBObjectStore.cpp (125733 => 125734)


--- trunk/Source/WebCore/Modules/indexeddb/IDBObjectStore.cpp	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBObjectStore.cpp	2012-08-16 01:41:37 UTC (rev 125734)
@@ -41,6 +41,7 @@
 #include "IDBKeyRange.h"
 #include "IDBTracing.h"
 #include "IDBTransaction.h"
+#include "ScriptCallStack.h"
 #include "SerializedScriptValue.h"
 #include <wtf/UnusedParam.h>
 

Modified: trunk/Source/WebCore/bindings/js/ScriptController.cpp (125733 => 125734)


--- trunk/Source/WebCore/bindings/js/ScriptController.cpp	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/bindings/js/ScriptController.cpp	2012-08-16 01:41:37 UTC (rev 125734)
@@ -36,6 +36,7 @@
 #include "Page.h"
 #include "PageGroup.h"
 #include "PluginView.h"
+#include "ScriptCallStack.h"
 #include "ScriptSourceCode.h"
 #include "ScriptValue.h"
 #include "ScriptableDocumentParser.h"

Modified: trunk/Source/WebCore/dom/ScriptElement.cpp (125733 => 125734)


--- trunk/Source/WebCore/dom/ScriptElement.cpp	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/dom/ScriptElement.cpp	2012-08-16 01:41:37 UTC (rev 125734)
@@ -38,6 +38,7 @@
 #include "IgnoreDestructiveWriteCountIncrementer.h"
 #include "MIMETypeRegistry.h"
 #include "Page.h"
+#include "ScriptCallStack.h"
 #include "ScriptRunner.h"
 #include "ScriptSourceCode.h"
 #include "ScriptValue.h"

Modified: trunk/Source/WebCore/dom/ScriptExecutionContext.h (125733 => 125734)


--- trunk/Source/WebCore/dom/ScriptExecutionContext.h	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/dom/ScriptExecutionContext.h	2012-08-16 01:41:37 UTC (rev 125734)
@@ -31,7 +31,6 @@
 #include "ActiveDOMObject.h"
 #include "ConsoleTypes.h"
 #include "KURL.h"
-#include "ScriptCallStack.h"
 #include "SecurityContext.h"
 #include "Supplementable.h"
 #include <wtf/Forward.h>
@@ -54,6 +53,7 @@
 class EventQueue;
 class EventTarget;
 class MessagePort;
+class ScriptCallStack;
 
 #if ENABLE(BLOB)
 class PublicURLManager;

Modified: trunk/Source/WebCore/fileapi/Blob.cpp (125733 => 125734)


--- trunk/Source/WebCore/fileapi/Blob.cpp	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/fileapi/Blob.cpp	2012-08-16 01:41:37 UTC (rev 125734)
@@ -34,6 +34,7 @@
 #include "BlobURL.h"
 #include "File.h"
 #include "HistogramSupport.h"
+#include "ScriptCallStack.h"
 #include "ScriptExecutionContext.h"
 #include "ThreadableBlobRegistry.h"
 

Modified: trunk/Source/WebCore/fileapi/WebKitBlobBuilder.cpp (125733 => 125734)


--- trunk/Source/WebCore/fileapi/WebKitBlobBuilder.cpp	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/fileapi/WebKitBlobBuilder.cpp	2012-08-16 01:41:37 UTC (rev 125734)
@@ -37,6 +37,7 @@
 #include "File.h"
 #include "HistogramSupport.h"
 #include "LineEnding.h"
+#include "ScriptCallStack.h"
 #include "ScriptExecutionContext.h"
 #include "TextEncoding.h"
 #include <wtf/ArrayBuffer.h>

Modified: trunk/Source/WebCore/html/HTMLTrackElement.cpp (125733 => 125734)


--- trunk/Source/WebCore/html/HTMLTrackElement.cpp	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/html/HTMLTrackElement.cpp	2012-08-16 01:41:37 UTC (rev 125734)
@@ -34,6 +34,7 @@
 #include "HTMLNames.h"
 #include "Logging.h"
 #include "RuntimeEnabledFeatures.h"
+#include "ScriptCallStack.h"
 #include "ScriptEventListener.h"
 
 using namespace std;

Modified: trunk/Source/WebCore/loader/TextTrackLoader.cpp (125733 => 125734)


--- trunk/Source/WebCore/loader/TextTrackLoader.cpp	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/loader/TextTrackLoader.cpp	2012-08-16 01:41:37 UTC (rev 125734)
@@ -35,6 +35,7 @@
 #include "Document.h"
 #include "Logging.h"
 #include "ResourceHandle.h"
+#include "ScriptCallStack.h"
 #include "SecurityOrigin.h"
 #include "SharedBuffer.h"
 #include "WebVTTParser.h"

Modified: trunk/Source/WebCore/page/DOMSecurityPolicy.cpp (125733 => 125734)


--- trunk/Source/WebCore/page/DOMSecurityPolicy.cpp	2012-08-16 01:27:41 UTC (rev 125733)
+++ trunk/Source/WebCore/page/DOMSecurityPolicy.cpp	2012-08-16 01:41:37 UTC (rev 125734)
@@ -30,6 +30,7 @@
 #include "ContextDestructionObserver.h"
 #include "DOMStringList.h"
 #include "Frame.h"
+#include "ScriptCallStack.h"
 #include "ScriptExecutionContext.h"
 #include <wtf/text/TextPosition.h>
 #include <wtf/text/WTFString.h>
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to