Bug 132399 - Move the legacy WebKit API into WebKitLegacy.framework and move it inside WebKit.framework
Summary: Move the legacy WebKit API into WebKitLegacy.framework and move it inside Web...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Anders Carlsson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-30 13:22 PDT by Anders Carlsson
Modified: 2014-04-30 15:02 PDT (History)
0 users

See Also:


Attachments
Patch (200.89 KB, patch)
2014-04-30 13:39 PDT, Anders Carlsson
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Anders Carlsson 2014-04-30 13:22:04 PDT
Move the legacy WebKit API into WebKitLegacy.framework and move it inside WebKit.framework
Comment 1 Anders Carlsson 2014-04-30 13:39:19 PDT
Created attachment 230516 [details]
Patch
Comment 2 mitz 2014-04-30 14:33:46 PDT
Comment on attachment 230516 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=230516&action=review

> Source/WebCore/ChangeLog:9
> +        Allow linking against WebKitLegacy.

Allow WebKitLegacy to link against WebCore.

> Source/WebKit/WebKit.xcodeproj/project.pbxproj:2118
> +			shellScript = "if [ \"${ACTION}\" = \"build\" -o \"${ACTION}\" = \"install\" -o \"${ACTION}\" = \"installhdrs\" ]; then\n    if [[ ${PLATFORM_NAME} == \"macosx\" ]]; then\n        mkdir -p \"${TARGET_BUILD_DIR}/${PUBLIC_HEADERS_FOLDER_PATH}\"\n    fi\n    mkdir -p \"${TARGET_BUILD_DIR}/${PRIVATE_HEADERS_FOLDER_PATH}\"\n    make -d -C mac/WebKitLegacy -f \"MigrateHeadersFromWebKitLegacy.make\" -j `/usr/sbin/sysctl -n hw.activecpu`\nfi\n";

Please remove the -d option to make.

> Source/WebKit/ios/DefaultDelegates/WebDefaultFrameLoadDelegate.m:30
> -#import <WebKit/WebFrameLoadDelegatePrivate.h>
> +#import <WebKitLegacy/WebFrameLoadDelegatePrivate.h>

We should not use framework-style imports for WebKitLegacy headers from within WebKit source files.

> Source/WebKit/mac/WebKitLegacy/MigrateHeadersFromWebKitLegacy.make:1
> +# Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.

2014

> Source/WebKit/mac/WebKitLegacy/MigrateHeadersFromWebKitLegacy.make:184
> +    WKBackForwardList.h \
> +    WKBackForwardListItem.h \
> +    WKFoundation.h \
> +    WKFrameInfo.h \
> +    WKNavigation.h \
> +    WKNavigationAction.h \
> +    WKNavigationDelegate.h \
> +    WKNavigationResponse.h \
> +    WKPreferences.h \
> +    WKProcessPool.h \
> +    WKScriptMessage.h \
> +    WKScriptMessageHandler.h \
> +    WKUIDelegate.h \
> +    WKUserContentController.h \
> +    WKWebView.h \
> +    WKWebViewConfiguration.h \

These modern API headers don’t belong here.
Comment 3 Anders Carlsson 2014-04-30 15:02:45 PDT
Committed r168047: <http://trac.webkit.org/changeset/168047>