Title: [165786] trunk/Source
Revision
165786
Author
[email protected]
Date
2014-03-17 18:35:46 -0700 (Mon, 17 Mar 2014)

Log Message

Rewrite WebHTMLConverter::_getComputedFloat in C++
https://bugs.webkit.org/show_bug.cgi?id=130284

Reviewed by Andreas Kling.

Rewrote _getComputedFloat as HTMLConverterCaches::floatPropertyValueForNode.

* platform/mac/HTMLConverter.h:
* platform/mac/HTMLConverter.mm:
(HTMLConverterCaches::floatPropertyValueForNode):
(-[WebHTMLConverter _getFloat:forNode:property:]): Now that computing the float value is fast, we don't need to
store it in the cache.
(-[WebHTMLConverter dealloc]):
(-[WebHTMLConverter init]):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (165785 => 165786)


--- trunk/Source/WebCore/ChangeLog	2014-03-18 01:26:08 UTC (rev 165785)
+++ trunk/Source/WebCore/ChangeLog	2014-03-18 01:35:46 UTC (rev 165786)
@@ -1,3 +1,20 @@
+2014-03-17  Ryosuke Niwa  <[email protected]>
+
+        Rewrite WebHTMLConverter::_getComputedFloat in C++
+        https://bugs.webkit.org/show_bug.cgi?id=130284
+
+        Reviewed by Andreas Kling.
+
+        Rewrote _getComputedFloat as HTMLConverterCaches::floatPropertyValueForNode.
+
+        * platform/mac/HTMLConverter.h:
+        * platform/mac/HTMLConverter.mm:
+        (HTMLConverterCaches::floatPropertyValueForNode):
+        (-[WebHTMLConverter _getFloat:forNode:property:]): Now that computing the float value is fast, we don't need to
+        store it in the cache.
+        (-[WebHTMLConverter dealloc]):
+        (-[WebHTMLConverter init]):
+
 2014-03-16  Andreas Kling  <[email protected]>
 
         HTMLInputElement::defaultEventHandler() shouldn't force style updates.

Modified: trunk/Source/WebCore/platform/mac/HTMLConverter.h (165785 => 165786)


--- trunk/Source/WebCore/platform/mac/HTMLConverter.h	2014-03-18 01:26:08 UTC (rev 165785)
+++ trunk/Source/WebCore/platform/mac/HTMLConverter.h	2014-03-18 01:35:46 UTC (rev 165786)
@@ -56,7 +56,6 @@
     NSMutableArray *_textTableRows;
     NSMutableArray *_textTableRowArrays;
     NSMutableArray *_textTableRowBackgroundColors;
-    NSMutableDictionary *_floatsForNodes;
     NSMutableDictionary *_colorsForNodes;
     NSMutableDictionary *_attributesForElements;
     NSMutableDictionary *_elementIsBlockLevel;

Modified: trunk/Source/WebCore/platform/mac/HTMLConverter.mm (165785 => 165786)


--- trunk/Source/WebCore/platform/mac/HTMLConverter.mm	2014-03-18 01:26:08 UTC (rev 165785)
+++ trunk/Source/WebCore/platform/mac/HTMLConverter.mm	2014-03-18 01:35:46 UTC (rev 165786)
@@ -402,6 +402,7 @@
 class HTMLConverterCaches {
 public:
     String propertyValueForNode(Node&, const String& propertyName);
+    bool floatPropertyValueForNode(Node&, const String& propertyName, float&);
 
     PassRefPtr<CSSValue> computedStylePropertyForElement(Element&, const String&);
     PassRefPtr<CSSValue> inlineStylePropertyForElement(Element&, const String&);
@@ -763,69 +764,60 @@
     }
 }
 
-- (BOOL)_getComputedFloat:(CGFloat *)val forNode:(DOMNode *)node property:(NSString *)key
+bool HTMLConverterCaches::floatPropertyValueForNode(Node& node, const String& propertyName, float& result)
 {
-    bool haveResult = false;
-    bool inherit = true;
-    float floatVal = 0;
-    Node* coreNode = core(node);
-    if (coreNode && coreNode->isElementNode()) {
-        Element& element = toElement(*coreNode);
-        String propertyName = key;
-        inherit = false;
-        if (!haveResult) {
-            if (RefPtr<CSSValue> value = _caches->computedStylePropertyForElement(element, propertyName)) {
-                if (value->isPrimitiveValue())
-                    haveResult = floatValueFromPrimitiveValue(toCSSPrimitiveValue(*value), floatVal);
-            }
-        }
-        if (!haveResult) {
-            if (RefPtr<CSSValue> value = _caches->inlineStylePropertyForElement(element, propertyName)) {
-                if (value->isInheritedValue())
-                    inherit = true;
-                else if (value->isPrimitiveValue())
-                    haveResult = floatValueFromPrimitiveValue(toCSSPrimitiveValue(*value), floatVal);
-            }
-        }
-        if (!haveResult) {
-            if ([@"text-indent" isEqualToString:key] || [@"letter-spacing" isEqualToString:key] || [@"word-spacing" isEqualToString:key]
-                || [@"line-height" isEqualToString:key] || [@"widows" isEqualToString:key] || [@"orphans" isEqualToString:key])
-                inherit = true;
-        }
+    if (!node.isElementNode()) {
+        if (ContainerNode* parent = node.parentNode())
+            return floatPropertyValueForNode(*parent, propertyName, result);
+        return false;
     }
-    if (!haveResult && inherit) {
-        DOMNode *parentNode = [node parentNode];
-        if (parentNode)
-            return [self _getFloat:val forNode:parentNode property:key];
+
+    Element& element = toElement(node);
+    if (RefPtr<CSSValue> value = computedStylePropertyForElement(element, propertyName)) {
+        if (value->isPrimitiveValue() && floatValueFromPrimitiveValue(toCSSPrimitiveValue(*value), result))
+            return true;
     }
-    if (haveResult && val)
-        *val = floatVal;
-    return haveResult;
+
+    bool inherit = false;
+    if (RefPtr<CSSValue> value = inlineStylePropertyForElement(element, propertyName)) {
+        if (value->isPrimitiveValue() && floatValueFromPrimitiveValue(toCSSPrimitiveValue(*value), result))
+            return true;
+        if (value->isInheritedValue())
+            inherit = true;
+    }
+
+    switch (cssPropertyID(propertyName)) {
+    case CSSPropertyTextIndent:
+    case CSSPropertyLetterSpacing:
+    case CSSPropertyWordSpacing:
+    case CSSPropertyLineHeight:
+    case CSSPropertyWidows:
+    case CSSPropertyOrphans:
+        inherit = true;
+        break;
+    default:
+        break;
+    }
+
+    if (inherit) {
+        if (ContainerNode* parent = node.parentNode())
+            return floatPropertyValueForNode(*parent, propertyName, result);
+    }
+
+    return false;
 }
 
 - (BOOL)_getFloat:(CGFloat *)val forNode:(DOMNode *)node property:(NSString *)key
 {
-    BOOL result = NO;
-    CGFloat floatVal = 0;
-    NSNumber *floatNumber;
-    RetainPtr<NSMutableDictionary> attributeDictionary = [_floatsForNodes objectForKey:node];
-    if (!attributeDictionary) {
-        attributeDictionary = adoptNS([[NSMutableDictionary alloc] init]);
-        [_floatsForNodes setObject:attributeDictionary.get() forKey:node];
-    }
-    floatNumber = [attributeDictionary objectForKey:key];
-    if (floatNumber) {
-        if (![[NSNull null] isEqual:floatNumber]) {
-            result = YES;
-            floatVal = [floatNumber floatValue];
-        }
-    } else {
-        result = [self _getComputedFloat:&floatVal forNode:node property:key];
-        [attributeDictionary setObject:(result ? (id)[NSNumber numberWithDouble:floatVal] : (id)[NSNull null]) forKey:key];
-    }
-    if (result && val)
-        *val = floatVal;
-    return result;
+    Node* coreNode = core(node);
+    if (!coreNode)
+        return NO;
+    float result;
+    if (!_caches->floatPropertyValueForNode(*coreNode, String(key), result))
+        return NO;
+    if (val)
+        *val = result;
+    return YES;
 }
 
 static NSString *_NSFirstPathForDirectoriesInDomains(NSSearchPathDirectory directory, NSSearchPathDomainMask domainMask, BOOL expandTilde)
@@ -2423,7 +2415,6 @@
     [_textTableRows release];
     [_textTableRowArrays release];
     [_textTableRowBackgroundColors release];
-    [_floatsForNodes release];
     [_colorsForNodes release];
     [_attributesForElements release];
     [_elementIsBlockLevel release];
@@ -2450,7 +2441,6 @@
     _textTableRows = [[NSMutableArray alloc] init];
     _textTableRowArrays = [[NSMutableArray alloc] init];
     _textTableRowBackgroundColors = [[NSMutableArray alloc] init];
-    _floatsForNodes = [[NSMutableDictionary alloc] init];
     _colorsForNodes = [[NSMutableDictionary alloc] init];
     _attributesForElements = [[NSMutableDictionary alloc] init];
     _elementIsBlockLevel = [[NSMutableDictionary alloc] init];

Modified: trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj (165785 => 165786)


--- trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj	2014-03-18 01:26:08 UTC (rev 165785)
+++ trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj	2014-03-18 01:35:46 UTC (rev 165786)
@@ -60,7 +60,6 @@
 		1AAF5FC00EDE3A92008D883D /* WebHostedNetscapePluginView.mm in Sources */ = {isa = PBXBuildFile; fileRef = 1AAF5FBE0EDE3A92008D883D /* WebHostedNetscapePluginView.mm */; };
 		1AB1DAC118BC0232004B6A9F /* WebViewGroup.mm in Sources */ = {isa = PBXBuildFile; fileRef = 1AB1DABF18BC0232004B6A9F /* WebViewGroup.mm */; };
 		1AB1DAC218BC0232004B6A9F /* WebViewGroup.h in Headers */ = {isa = PBXBuildFile; fileRef = 1AB1DAC018BC0232004B6A9F /* WebViewGroup.h */; };
-		1AD7453C18D0D324006F3A1E /* WebKitLegacy.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 1AD7451218D0D24C006F3A1E /* WebKitLegacy.cpp */; };
 		1AEA66D40DC6B1FF003D12BF /* WebNetscapePluginEventHandler.h in Headers */ = {isa = PBXBuildFile; fileRef = 1AEA66D20DC6B1FF003D12BF /* WebNetscapePluginEventHandler.h */; };
 		1AEA66D50DC6B1FF003D12BF /* WebNetscapePluginEventHandler.mm in Sources */ = {isa = PBXBuildFile; fileRef = 1AEA66D30DC6B1FF003D12BF /* WebNetscapePluginEventHandler.mm */; };
 		1AEA66D80DC6B209003D12BF /* WebNetscapePluginEventHandlerCarbon.h in Headers */ = {isa = PBXBuildFile; fileRef = 1AEA66D60DC6B209003D12BF /* WebNetscapePluginEventHandlerCarbon.h */; };
@@ -81,8 +80,6 @@
 		224100F90918190100D2D266 /* WebPluginsPrivate.m in Sources */ = {isa = PBXBuildFile; fileRef = 224100F80918190100D2D266 /* WebPluginsPrivate.m */; };
 		22F219CC08D236730030E078 /* WebBackForwardListPrivate.h in Headers */ = {isa = PBXBuildFile; fileRef = 22F219CB08D236730030E078 /* WebBackForwardListPrivate.h */; settings = {ATTRIBUTES = (Private, ); }; };
 		29AEF960134C76FB00FE5096 /* OutlookQuirksUserScript.js in Resources */ = {isa = PBXBuildFile; fileRef = 29AEF95D134C755400FE5096 /* OutlookQuirksUserScript.js */; };
-		2D25396618CE85C200270222 /* WebSharingServicePickerController.h in Headers */ = {isa = PBXBuildFile; fileRef = 2D25396418CE85C200270222 /* WebSharingServicePickerController.h */; };
-		2D25396718CE85C200270222 /* WebSharingServicePickerController.mm in Sources */ = {isa = PBXBuildFile; fileRef = 2D25396518CE85C200270222 /* WebSharingServicePickerController.mm */; };
 		312E2FE514E48182007CCA18 /* WebNotification.h in Headers */ = {isa = PBXBuildFile; fileRef = 312E2FE314E48182007CCA18 /* WebNotification.h */; settings = {ATTRIBUTES = (Private, ); }; };
 		312E2FE614E48182007CCA18 /* WebNotification.mm in Sources */ = {isa = PBXBuildFile; fileRef = 312E2FE414E48182007CCA18 /* WebNotification.mm */; };
 		312E2FE914E48215007CCA18 /* WebNotificationInternal.h in Headers */ = {isa = PBXBuildFile; fileRef = 312E2FE814E48215007CCA18 /* WebNotificationInternal.h */; };
@@ -453,16 +450,6 @@
 		F834AAD80E64B1C700E2737C /* WebTextIterator.mm in Sources */ = {isa = PBXBuildFile; fileRef = F834AAD60E64B1C700E2737C /* WebTextIterator.mm */; };
 /* End PBXBuildFile section */
 
-/* Begin PBXContainerItemProxy section */
-		1A5B250918D0DBD300913729 /* PBXContainerItemProxy */ = {
-			isa = PBXContainerItemProxy;
-			containerPortal = 0867D690FE84028FC02AAC07 /* Project object */;
-			proxyType = 1;
-			remoteGlobalIDString = 9398100A0824BF01008DF038;
-			remoteInfo = WebKit;
-		};
-/* End PBXContainerItemProxy section */
-
 /* Begin PBXFileReference section */
 		065AD5A10B0C32C7005A2B1D /* WebContextMenuClient.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = WebContextMenuClient.h; sourceTree = "<group>"; };
 		065AD5A20B0C32C7005A2B1D /* WebContextMenuClient.mm */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.objcpp; path = WebContextMenuClient.mm; sourceTree = "<group>"; };
@@ -514,10 +501,6 @@
 		1AAF5FBE0EDE3A92008D883D /* WebHostedNetscapePluginView.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WebHostedNetscapePluginView.mm; sourceTree = "<group>"; };
 		1AB1DABF18BC0232004B6A9F /* WebViewGroup.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WebViewGroup.mm; sourceTree = "<group>"; };
 		1AB1DAC018BC0232004B6A9F /* WebViewGroup.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WebViewGroup.h; sourceTree = "<group>"; };
-		1AD7451218D0D24C006F3A1E /* WebKitLegacy.cpp */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.cpp; path = WebKitLegacy.cpp; sourceTree = "<group>"; };
-		1AD7451918D0D26C006F3A1E /* WebKitLegacy.framework */ = {isa = PBXFileReference; explicitFileType = wrapper.framework; includeInIndex = 0; path = WebKitLegacy.framework; sourceTree = BUILT_PRODUCTS_DIR; };
-		1AD7453B18D0D2A6006F3A1E /* WebKitLegacy.xcconfig */ = {isa = PBXFileReference; lastKnownFileType = text.xcconfig; path = WebKitLegacy.xcconfig; sourceTree = "<group>"; };
-		1AD7453D18D0D383006F3A1E /* Info.plist */ = {isa = PBXFileReference; lastKnownFileType = text.plist.xml; path = Info.plist; sourceTree = "<group>"; };
 		1AEA66D20DC6B1FF003D12BF /* WebNetscapePluginEventHandler.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WebNetscapePluginEventHandler.h; sourceTree = "<group>"; };
 		1AEA66D30DC6B1FF003D12BF /* WebNetscapePluginEventHandler.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WebNetscapePluginEventHandler.mm; sourceTree = "<group>"; };
 		1AEA66D60DC6B209003D12BF /* WebNetscapePluginEventHandlerCarbon.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WebNetscapePluginEventHandlerCarbon.h; sourceTree = "<group>"; };
@@ -544,8 +527,6 @@
 		22F219CB08D236730030E078 /* WebBackForwardListPrivate.h */ = {isa = PBXFileReference; fileEncoding = 4; indentWidth = 4; lastKnownFileType = sourcecode.c.h; path = WebBackForwardListPrivate.h; sourceTree = "<group>"; tabWidth = 8; usesTabs = 0; };
 		2568C72C0174912D0ECA149E /* WebKit.h */ = {isa = PBXFileReference; fileEncoding = 4; indentWidth = 4; lastKnownFileType = sourcecode.c.h; path = WebKit.h; sourceTree = "<group>"; tabWidth = 8; usesTabs = 0; };
 		29AEF95D134C755400FE5096 /* OutlookQuirksUserScript.js */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode._javascript_; path = OutlookQuirksUserScript.js; sourceTree = "<group>"; };
-		2D25396418CE85C200270222 /* WebSharingServicePickerController.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WebSharingServicePickerController.h; sourceTree = "<group>"; };
-		2D25396518CE85C200270222 /* WebSharingServicePickerController.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WebSharingServicePickerController.mm; sourceTree = "<group>"; };
 		2D36FD5E03F78F9E00A80166 /* WebFormDelegatePrivate.h */ = {isa = PBXFileReference; fileEncoding = 4; indentWidth = 4; lastKnownFileType = sourcecode.c.h; path = WebFormDelegatePrivate.h; sourceTree = "<group>"; tabWidth = 8; usesTabs = 0; };
 		2D81DAB203EB0B2D00A80166 /* WebFormDelegate.h */ = {isa = PBXFileReference; fileEncoding = 30; indentWidth = 4; lastKnownFileType = sourcecode.c.h; path = WebFormDelegate.h; sourceTree = "<group>"; tabWidth = 8; usesTabs = 0; };
 		2D81DAB303EB0B2D00A80166 /* WebFormDelegate.m */ = {isa = PBXFileReference; fileEncoding = 30; indentWidth = 4; lastKnownFileType = sourcecode.c.objc; path = WebFormDelegate.m; sourceTree = "<group>"; tabWidth = 8; usesTabs = 0; };
@@ -923,13 +904,6 @@
 /* End PBXFileReference section */
 
 /* Begin PBXFrameworksBuildPhase section */
-		1AD7451518D0D26C006F3A1E /* Frameworks */ = {
-			isa = PBXFrameworksBuildPhase;
-			buildActionMask = 2147483647;
-			files = (
-			);
-			runOnlyForDeploymentPostprocessing = 0;
-		};
 		939811270824BF01008DF038 /* Frameworks */ = {
 			isa = PBXFrameworksBuildPhase;
 			buildActionMask = 2147483647;
@@ -949,7 +923,6 @@
 			isa = PBXGroup;
 			children = (
 				939811330824BF01008DF038 /* WebKit.framework */,
-				1AD7451918D0D26C006F3A1E /* WebKitLegacy.framework */,
 			);
 			name = Products;
 			sourceTree = "<group>";
@@ -972,7 +945,6 @@
 				F5EBC45202134BB601CA1520 /* Plugins */,
 				511F3FC30CECC7E200852565 /* Storage */,
 				F5B36B400281DE87018635CB /* WebCoreSupport */,
-				1AD7451118D0D23D006F3A1E /* WebKitLegacy */,
 				9C7CABBB0190A37C0ECA16EA /* WebView */,
 				1C68F63F095B5F9C00C2984E /* WebInspector */,
 				F7EBEE5903F9DB2203CA0DE6 /* Carbon */,
@@ -1054,16 +1026,6 @@
 			name = Events;
 			sourceTree = "<group>";
 		};
-		1AD7451118D0D23D006F3A1E /* WebKitLegacy */ = {
-			isa = PBXGroup;
-			children = (
-				1AD7453D18D0D383006F3A1E /* Info.plist */,
-				1AD7451218D0D24C006F3A1E /* WebKitLegacy.cpp */,
-			);
-			name = WebKitLegacy;
-			path = mac/WebKitLegacy;
-			sourceTree = "<group>";
-		};
 		1C68F63F095B5F9C00C2984E /* WebInspector */ = {
 			isa = PBXGroup;
 			children = (
@@ -1094,7 +1056,6 @@
 				FEE7D6910D99B06A005351F6 /* iOS.xcconfig */,
 				1C904FD30BA9DD0F0081E9D0 /* Version.xcconfig */,
 				1C904FD20BA9DD0F0081E9D0 /* WebKit.xcconfig */,
-				1AD7453B18D0D2A6006F3A1E /* WebKitLegacy.xcconfig */,
 			);
 			name = Configurations;
 			path = mac/Configurations;
@@ -1173,8 +1134,6 @@
 				9345DDAE0365FB27008635CE /* WebNSWindowExtras.h */,
 				9345DDAF0365FB27008635CE /* WebNSWindowExtras.m */,
 				A57E2F22120749E600048DF3 /* WebQuotaManager.h */,
-				2D25396418CE85C200270222 /* WebSharingServicePickerController.h */,
-				2D25396518CE85C200270222 /* WebSharingServicePickerController.mm */,
 				F59668C802AD2923018635CA /* WebStringTruncator.h */,
 				F59668C902AD2923018635CA /* WebStringTruncator.mm */,
 				DD7CDEE60A23BA9E00069928 /* WebTypesInternal.h */,
@@ -1895,7 +1854,6 @@
 				939810240824BF01008DF038 /* WebNSViewExtras.h in Headers */,
 				939810250824BF01008DF038 /* WebNSWindowExtras.h in Headers */,
 				A58A5799143E727000125F50 /* WebOpenPanelResultListener.h in Headers */,
-				2D25396618CE85C200270222 /* WebSharingServicePickerController.h in Headers */,
 				9398102A0824BF01008DF038 /* WebPanelAuthenticationHandler.h in Headers */,
 				37B6FB4E1063530C000FDB3B /* WebPDFDocumentExtras.h in Headers */,
 				939810A50824BF01008DF038 /* WebPDFRepresentation.h in Headers */,
@@ -1965,24 +1923,6 @@
 /* End PBXHeadersBuildPhase section */
 
 /* Begin PBXNativeTarget section */
-		1AD7451818D0D26C006F3A1E /* WebKitLegacy */ = {
-			isa = PBXNativeTarget;
-			buildConfigurationList = 1AD7453318D0D26C006F3A1E /* Build configuration list for PBXNativeTarget "WebKitLegacy" */;
-			buildPhases = (
-				1A5B250818D0D90C00913729 /* Migrate Headers */,
-				1AD7451418D0D26C006F3A1E /* Sources */,
-				1AD7451518D0D26C006F3A1E /* Frameworks */,
-			);
-			buildRules = (
-			);
-			dependencies = (
-				1A5B250A18D0DBD300913729 /* PBXTargetDependency */,
-			);
-			name = WebKitLegacy;
-			productName = WebKitLegacy;
-			productReference = 1AD7451918D0D26C006F3A1E /* WebKitLegacy.framework */;
-			productType = "com.apple.product-type.framework";
-		};
 		9398100A0824BF01008DF038 /* WebKit */ = {
 			isa = PBXNativeTarget;
 			buildConfigurationList = 149C282D08902B0F008A9EFC /* Build configuration list for PBXNativeTarget "WebKit" */;
@@ -2038,7 +1978,6 @@
 			projectDirPath = "";
 			projectRoot = "";
 			targets = (
-				1AD7451818D0D26C006F3A1E /* WebKitLegacy */,
 				9398100A0824BF01008DF038 /* WebKit */,
 			);
 		};
@@ -2060,20 +1999,6 @@
 /* End PBXResourcesBuildPhase section */
 
 /* Begin PBXShellScriptBuildPhase section */
-		1A5B250818D0D90C00913729 /* Migrate Headers */ = {
-			isa = PBXShellScriptBuildPhase;
-			buildActionMask = 2147483647;
-			files = (
-			);
-			inputPaths = (
-			);
-			name = "Migrate Headers";
-			outputPaths = (
-			);
-			runOnlyForDeploymentPostprocessing = 0;
-			shellPath = /bin/sh;
-			shellScript = "if [ \"${ACTION}\" = \"build\" -o \"${ACTION}\" = \"install\" -o \"${ACTION}\" = \"installhdrs\" ]; then\n    mkdir -p \"${TARGET_BUILD_DIR}/${PRIVATE_HEADERS_FOLDER_PATH}\"\n    mkdir -p \"${TARGET_BUILD_DIR}/${PUBLIC_HEADERS_FOLDER_PATH}\"\n    make -C mac/WebKitLegacy -f \"MigrateHeadersToLegacy.make\" -j `/usr/sbin/sysctl -n hw.activecpu`\nfi\n";
-		};
 		1C395DE20C6BE8E0000D1E52 /* Generate Export Files */ = {
 			isa = PBXShellScriptBuildPhase;
 			buildActionMask = 2147483647;
@@ -2106,7 +2031,7 @@
 			);
 			runOnlyForDeploymentPostprocessing = 0;
 			shellPath = /bin/sh;
-			shellScript = "exec \"${SRCROOT}/mac/migrate-headers.sh\"";
+			shellScript = "mkdir -p \"${TARGET_BUILD_DIR}/${PRIVATE_HEADERS_FOLDER_PATH}\"\nmkdir -p \"${TARGET_BUILD_DIR}/${PUBLIC_HEADERS_FOLDER_PATH}\"\nmkdir -p \"${BUILT_PRODUCTS_DIR}/DerivedSources/WebKit\"\n\n# If we didn't build WebCore, use the production copy of the headers\nif [ ! -d \"${WEBCORE_PRIVATE_HEADERS_DIR}\" ]; then\n    export WEBCORE_PRIVATE_HEADERS_DIR=\"`eval 'echo ${WEBCORE_PRIVATE_HEADERS_DIR_'${PLATFORM_NAME}'_Production}'`\"\nfi\n\nif [ \"${ACTION}\" = \"build\" -o \"${ACTION}\" = \"install\" -o \"${ACTION}\" = \"installhdrs\" ]; then\n    make -C mac -f \"MigrateHeaders.make\" -j `/usr/sbin/sysctl -n hw.activecpu`\nfi\n";
 		};
 		3713F018142905B70036387F /* Check For Inappropriate Objective-C Class Names */ = {
 			isa = PBXShellScriptBuildPhase;
@@ -2232,19 +2157,10 @@
 /* End PBXShellScriptBuildPhase section */
 
 /* Begin PBXSourcesBuildPhase section */
-		1AD7451418D0D26C006F3A1E /* Sources */ = {
-			isa = PBXSourcesBuildPhase;
-			buildActionMask = 2147483647;
-			files = (
-				1AD7453C18D0D324006F3A1E /* WebKitLegacy.cpp in Sources */,
-			);
-			runOnlyForDeploymentPostprocessing = 0;
-		};
 		939810BB0824BF01008DF038 /* Sources */ = {
 			isa = PBXSourcesBuildPhase;
 			buildActionMask = 2147483647;
 			files = (
-				2D25396718CE85C200270222 /* WebSharingServicePickerController.mm in Sources */,
 				1A60519117502A5D00BC62F5 /* BinaryPropertyList.cpp in Sources */,
 				939811010824BF01008DF038 /* CarbonUtils.m in Sources */,
 				939810FD0824BF01008DF038 /* CarbonWindowAdapter.mm in Sources */,
@@ -2428,14 +2344,6 @@
 		};
 /* End PBXSourcesBuildPhase section */
 
-/* Begin PBXTargetDependency section */
-		1A5B250A18D0DBD300913729 /* PBXTargetDependency */ = {
-			isa = PBXTargetDependency;
-			target = 9398100A0824BF01008DF038 /* WebKit */;
-			targetProxy = 1A5B250918D0DBD300913729 /* PBXContainerItemProxy */;
-		};
-/* End PBXTargetDependency section */
-
 /* Begin PBXVariantGroup section */
 		5DE83A740D0F7F9400CAD12A /* WebJavaScriptTextInputPanel.nib */ = {
 			isa = PBXVariantGroup;
@@ -2521,27 +2429,6 @@
 			};
 			name = Production;
 		};
-		1AD7453418D0D26C006F3A1E /* Debug */ = {
-			isa = XCBuildConfiguration;
-			baseConfigurationReference = 1AD7453B18D0D2A6006F3A1E /* WebKitLegacy.xcconfig */;
-			buildSettings = {
-			};
-			name = Debug;
-		};
-		1AD7453518D0D26C006F3A1E /* Release */ = {
-			isa = XCBuildConfiguration;
-			baseConfigurationReference = 1AD7453B18D0D2A6006F3A1E /* WebKitLegacy.xcconfig */;
-			buildSettings = {
-			};
-			name = Release;
-		};
-		1AD7453618D0D26C006F3A1E /* Production */ = {
-			isa = XCBuildConfiguration;
-			baseConfigurationReference = 1AD7453B18D0D2A6006F3A1E /* WebKitLegacy.xcconfig */;
-			buildSettings = {
-			};
-			name = Production;
-		};
 /* End XCBuildConfiguration section */
 
 /* Begin XCConfigurationList section */
@@ -2565,16 +2452,6 @@
 			defaultConfigurationIsVisible = 0;
 			defaultConfigurationName = Production;
 		};
-		1AD7453318D0D26C006F3A1E /* Build configuration list for PBXNativeTarget "WebKitLegacy" */ = {
-			isa = XCConfigurationList;
-			buildConfigurations = (
-				1AD7453418D0D26C006F3A1E /* Debug */,
-				1AD7453518D0D26C006F3A1E /* Release */,
-				1AD7453618D0D26C006F3A1E /* Production */,
-			);
-			defaultConfigurationIsVisible = 0;
-			defaultConfigurationName = Production;
-		};
 /* End XCConfigurationList section */
 	};
 	rootObject = 0867D690FE84028FC02AAC07 /* Project object */;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to