Skip to content
  • mihaip@chromium.org's avatar
    2011-04-27 Mihai Parparita <mihaip@chromium.org> · 3c16b0c1
    mihaip@chromium.org authored
            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: http://svn.webkit.org/repository/webkit/trunk@85030 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    3c16b0c1