Bug 206505 – Fix missing header guards and clean up empty files in WebCore, WebKitLegacy, WebKit, Tools

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[David Kilzer (:ddkilzer)]

Fix missing header guards and clean up empty files in WebCore, WebKitLegacy, WebKit, Tools.

See Bug 206480 Comment #6 for WebCore, WebKitLegacy, WebKit.

Here are the Tools directories I checked:

$ ./Tools/Scripts/check-webkit-style --filter=-,+build/header_guard_missing Tools/DumpRenderTree Tools/WebKitTestRunner Tools/TestRunnerShared Tools/TestWebKitAPI
ERROR: Tools/DumpRenderTree/win/resource.h:5:  Missing #pragma once for header guard.  [build/header_guard_missing] [5]
ERROR: Tools/DumpRenderTree/mac/InternalHeaders/WebKit/WebTypesInternal.h:3:  Missing #pragma once for header guard.  [build/header_guard_missing] [5]
ERROR: Tools/DumpRenderTree/TestNetscapePlugIn/TestObject.h:26:  Missing #pragma once for header guard.  [build/header_guard_missing] [5]
ERROR: Tools/DumpRenderTree/TestNetscapePlugIn/ForwardingHeaders/WebKit/npruntime.h:3:  Missing #pragma once for header guard.  [build/header_guard_missing] [5]
ERROR: Tools/DumpRenderTree/TestNetscapePlugIn/ForwardingHeaders/WebKit/npfunctions.h:3:  Missing #pragma once for header guard.  [build/header_guard_missing] [5]
ERROR: Tools/DumpRenderTree/TestNetscapePlugIn/ForwardingHeaders/WebKit/npapi.h:3:  Missing #pragma once for header guard.  [build/header_guard_missing] [5]
ERROR: Tools/TestWebKitAPI/JavaScriptTest.h:26:  Missing #pragma once for header guard.  [build/header_guard_missing] [5]
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/DOMNodeFilterTest.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestDOMDOMWindow.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestWebKitAccessibility.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestWebKitVersion.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestAutocleanups.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/DOMNodeTest.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestContextMenu.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/InspectorTestServer.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestDOMXPathNSResolver.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestDOMNodeFilter.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestInspectorServer.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestInspector.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/DOMClientRectTest.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestOptionMenu.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/AccessibilityTestServer.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestDOMClientRect.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestDOMNode.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestWebViewEditor.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/DOMDOMWindowTest.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/AutocleanupsTest.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/TestPrinting.cpp"
WARNING: File exempt from style guide. Skipping: "Tools/TestWebKitAPI/Tests/WebKitGtk/DOMXPathNSResolverTest.cpp"
ERROR: Tools/TestWebKitAPI/Tests/WebKitGLib/WebProcessTest.h:20:  Missing #pragma once for header guard.  [build/header_guard_missing] [5]
ERROR: Tools/TestWebKitAPI/Tests/WebCore/DFAHelpers.h:26:  Missing #pragma once for header guard.  [build/header_guard_missing] [5]
Total errors found: 9 in 1735 files

[David Kilzer (:ddkilzer)]

Created attachment 388246 [details]
Patch v1

[David Kilzer (:ddkilzer)]

Headers I didn't fix in this patch:

Source/WebKitLegacy/win/resource.h
Tools/DumpRenderTree/win/resource.h
- Headers generated by Microsoft Visual C++.

Tools/DumpRenderTree/TestNetscapePlugIn/ForwardingHeaders/WebKit/npruntime.h
Tools/DumpRenderTree/TestNetscapePlugIn/ForwardingHeaders/WebKit/npfunctions.h
Tools/DumpRenderTree/TestNetscapePlugIn/ForwardingHeaders/WebKit/npapi.h
- Forwarding headers have never had header guards, and are increasingly rarely used.

Source/WebKit/NetworkProcess/glib/WebKitCachedResolver.h
Tools/TestWebKitAPI/Tests/WebKitGLib/WebProcessTest.h
- Didn't touch GTK-related headers.

[WebKit Commit Bot]

Comment on attachment 388246 [details]
Patch v1

Clearing flags on attachment: 388246

Committed r254836: <https://trac.webkit.org/changeset/254836>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/58743410>