Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 961540b8e2eeb3788116bea395e2900cea7ebc6d
      
https://github.com/WebKit/WebKit/commit/961540b8e2eeb3788116bea395e2900cea7ebc6d
  Author: Elliott Williams <e...@apple.com>
  Date:   2023-02-22 (Wed, 22 Feb 2023)

  Changed paths:
    R Source/WebKit/Scripts/migrate-headers.py
    M Source/WebKit/WebKit.xcodeproj/project.pbxproj
    A Source/WebKitLegacy/MigratedHeaders-input.xcfilelist
    A Source/WebKitLegacy/MigratedHeaders-output.xcfilelist
    M Source/WebKitLegacy/WebKitLegacy.xcodeproj/project.pbxproj
    M Source/WebKitLegacy/ios/WebCoreSupport/WebInspectorClientIOS.mm
    M Source/WebKitLegacy/mac/Configurations/WebKitLegacy.xcconfig
    M Source/WebKitLegacy/scripts/migrate-header-rule
    A Source/WebKitLegacy/scripts/migrate-headers.py

  Log Message:
  -----------
  [Xcode] Move WebKitLegacy's header migration logic to a script phase
https://bugs.webkit.org/show_bug.cgi?id=252715

Reviewed by Alexey Proskuryakov.

Apply the same refactoring that was done for WebKit in 259916@main.
Move the migrate-headers.py script to WebKitLegacy and use it to copy
headers from WebCore.

Unlike WebKit, which reexports WebKitLegacy, WebKitLegacy doesn't
formally reexport WebCore, so extra work is needed to tell TAPI about
the migrated headers. Use -extra-private-header to pass them to tapi.
This list must be maintained, but since the set of migrated headers is
stable and verified by TAPI, it should be a low maintenance burden. We
can change migrate-headers.py to generate a filelist once
rdar://102197642 is done.

* Source/WebKit/WebKit.xcodeproj/project.pbxproj: Refer to
  migrate-headers.py from its new location in WebKitLegacy.
* Source/WebKitLegacy/MigratedHeaders-input.xcfilelist: Added.
* Source/WebKitLegacy/MigratedHeaders-output.xcfilelist: Added.
* Source/WebKitLegacy/WebKitLegacy.xcodeproj/project.pbxproj:
  - Remove "Migrated Headers" group.
  - Add migrate-headers.py script and the xcfilelists that power it.
  - Replace the migrate-header-rule build rule with a script phase that calls 
migrate-headers.py.
  - Update the "Generate Export Files" script phase to read a single
    `ReexportedFromWebCore.exp` file.
* Source/WebKitLegacy/ios/WebCoreSupport/WebInspectorClientIOS.mm: I'm
  not exactly sure why, but the compiler now complains about some
  missing implementation methods in the stubbed-out
  WebInspectorFrontendClient for iOS. Add them.
(WebInspectorFrontendClient::supportsDiagnosticLogging):
(WebInspectorFrontendClient::logDiagnosticEvent):
(WebInspectorFrontendClient::canSave):
* Source/WebKitLegacy/mac/Configurations/WebKitLegacy.xcconfig: Add
  -extra-private-header flags and leave FIXMEs about when to get rid of
  it.
* Source/WebKitLegacy/scripts/migrate-header-rule:
  - Stop creating forwarding headers. Because the migrated headers
    aren't part of WebKitLegacy's target, they aren't part of its
    headermap, which means we no longer need to support parsing
    unprocessed WebCore headers with other <WebCore/...> includes.
  - Instead of creating one export file per header, write them all to a
    single `ReexportedFromWebCore.exp`.
* Source/WebKitLegacy/scripts/migrate-headers.py: Renamed from 
Source/WebKit/Scripts/migrate-headers.py.

Canonical link: https://commits.webkit.org/260718@main


_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to