2011-04-27 Mihai Parparita <mihaip@chromium.org>
authormihaip@chromium.org <mihaip@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Apr 2011 07:37:02 +0000 (07:37 +0000)
committermihaip@chromium.org <mihaip@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Apr 2011 07:37:02 +0000 (07:37 +0000)
commit33eed9004324541313b93a6cf440f429579c7de3
tree4e0ab43d5f731a5702ed4a82f4371326fd100922
parenta17dfe402694351fc026eb40aa4f9153c1dcb763
2011-04-27  Mihai Parparita  <mihaip@chromium.org>

        Reviewed by Eric Seidel.

        Frame.h shouldn't include Document.h
        https://bugs.webkit.org/show_bug.cgi?id=59560

        Frame can use a forward-declared Document if we move
        Frame::displayStringModifiedByEncoding's implementation out of the
        header.
                                Size  #includes  Size * #includes
        Before: page/Frame.h:   0.746       653           487.138
                dom/Document.h: 0.220      1306           287.320

        After:  page/Frame.h:   0.646       653           421.838
                dom/Document.h: 0.220      1189           261.580

        * WebCore.exp.in:
        * bindings/ScriptControllerBase.cpp:
        * bindings/generic/BindingSecurityBase.cpp:
        * bindings/v8/V8Proxy.cpp:
        * bindings/v8/custom/V8XMLHttpRequestCustom.cpp:
        * dom/CharacterData.cpp:
        * dom/UserTypingGestureIndicator.cpp:
        * editing/chromium/SelectionControllerChromium.cpp:
        * inspector/InspectorDOMStorageResource.cpp:
        * inspector/InspectorInstrumentation.h:
        * loader/PingLoader.cpp:
        * loader/SubresourceLoader.cpp:
        * loader/appcache/DOMApplicationCache.cpp:
        * loader/cache/CachedResourceRequest.cpp:
        * page/Frame.cpp:
        (WebCore::Frame::displayStringModifiedByEncoding):
        * page/Frame.h:
        * page/Geolocation.cpp:
        * page/Location.cpp:
        * page/Navigator.cpp:
        * page/PageGroupLoadDeferrer.cpp:
        * page/PerformanceTiming.cpp:
        * xml/XSLTProcessorLibxslt.cpp:
2011-04-27  Mihai Parparita  <mihaip@chromium.org>

        Reviewed by Eric Seidel.

        Frame.h shouldn't include Document.h
        https://bugs.webkit.org/show_bug.cgi?id=59560

        Add Document.h #includes for files that were relying on transitive
        #includes via Frame.h.

        * src/LocalFileSystemChromium.cpp:
        * src/StorageEventDispatcherImpl.cpp:
        * src/WebFrameImpl.h:
        * src/WebWorkerBase.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@85030 268f45cc-cd09-0410-ab3c-d52691b4dbfc
28 files changed:
Source/WebCore/ChangeLog
Source/WebCore/WebCore.exp.in
Source/WebCore/bindings/ScriptControllerBase.cpp
Source/WebCore/bindings/generic/BindingSecurityBase.cpp
Source/WebCore/bindings/v8/V8Proxy.cpp
Source/WebCore/bindings/v8/custom/V8XMLHttpRequestCustom.cpp
Source/WebCore/dom/CharacterData.cpp
Source/WebCore/dom/UserTypingGestureIndicator.cpp
Source/WebCore/editing/chromium/SelectionControllerChromium.cpp
Source/WebCore/inspector/InspectorDOMStorageResource.cpp
Source/WebCore/inspector/InspectorInstrumentation.h
Source/WebCore/loader/PingLoader.cpp
Source/WebCore/loader/SubresourceLoader.cpp
Source/WebCore/loader/appcache/DOMApplicationCache.cpp
Source/WebCore/loader/cache/CachedResourceRequest.cpp
Source/WebCore/page/Frame.cpp
Source/WebCore/page/Frame.h
Source/WebCore/page/Geolocation.cpp
Source/WebCore/page/Location.cpp
Source/WebCore/page/Navigator.cpp
Source/WebCore/page/PageGroupLoadDeferrer.cpp
Source/WebCore/page/PerformanceTiming.cpp
Source/WebCore/xml/XSLTProcessorLibxslt.cpp
Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/src/LocalFileSystemChromium.cpp
Source/WebKit/chromium/src/StorageEventDispatcherImpl.cpp
Source/WebKit/chromium/src/WebFrameImpl.h
Source/WebKit/chromium/src/WebWorkerBase.cpp