Bug 206505 - Fix missing header guards and clean up empty files in WebCore, WebKitLegacy, WebKit, Tools
Summary: Fix missing header guards and clean up empty files in WebCore, WebKitLegacy, ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-01-20 09:27 PST by David Kilzer (:ddkilzer)
Modified: 2020-01-20 13:22 PST (History)
14 users (show)

See Also:


Attachments
Patch v1 (35.86 KB, patch)
2020-01-20 09:41 PST, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2020-01-20 09:27:41 PST
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
Comment 1 David Kilzer (:ddkilzer) 2020-01-20 09:41:45 PST
Created attachment 388246 [details]
Patch v1
Comment 2 David Kilzer (:ddkilzer) 2020-01-20 09:45:15 PST
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.
Comment 3 WebKit Commit Bot 2020-01-20 13:21:23 PST
Comment on attachment 388246 [details]
Patch v1

Clearing flags on attachment: 388246

Committed r254836: <https://trac.webkit.org/changeset/254836>
Comment 4 WebKit Commit Bot 2020-01-20 13:21:25 PST
All reviewed patches have been landed.  Closing bug.
Comment 5 Radar WebKit Bug Importer 2020-01-20 13:22:16 PST
<rdar://problem/58743410>