JSC_OBJC_API_ENABLED should be removed from FeatureDefines.h
authorl.gombos@samsung.com <l.gombos@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 Mar 2013 21:11:33 +0000 (21:11 +0000)
committerl.gombos@samsung.com <l.gombos@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 Mar 2013 21:11:33 +0000 (21:11 +0000)
https://bugs.webkit.org/show_bug.cgi?id=111269

Reviewed by Mark Hahnenberg.

Source/WebKit/mac:

Include JavaScriptCore/JSBase.h for the definition of JSC_OBJC_API_ENABLED.

* WebView/WebDelegateImplementationCaching.h:

Source/WTF:

JSC_OBJC_API_ENABLED flag was added to FeatureDefines.h (r143637)
so that it could be used in WebCore.exp.in to determine whether
or not we should be exporting certain symbols based on whether
the WebKit-related portion of the JSC ObjC API was enabled.

It turned out conditionally including certain symbols didn't
work when building fat binaries, so it's not even necessary any more.

* wtf/FeatureDefines.h: Remove JSC_OBJC_API_ENABLED.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@144662 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WTF/ChangeLog
Source/WTF/wtf/FeatureDefines.h
Source/WebKit/mac/ChangeLog
Source/WebKit/mac/WebView/WebDelegateImplementationCaching.h

index 826f6a1..fbc1a32 100644 (file)
@@ -1,3 +1,20 @@
+2013-03-04  Laszlo Gombos  <l.gombos@samsung.com>
+
+        JSC_OBJC_API_ENABLED should be removed from FeatureDefines.h
+        https://bugs.webkit.org/show_bug.cgi?id=111269
+
+        Reviewed by Mark Hahnenberg.
+
+        JSC_OBJC_API_ENABLED flag was added to FeatureDefines.h (r143637) 
+        so that it could be used in WebCore.exp.in to determine whether 
+        or not we should be exporting certain symbols based on whether 
+        the WebKit-related portion of the JSC ObjC API was enabled.
+        
+        It turned out conditionally including certain symbols didn't
+        work when building fat binaries, so it's not even necessary any more.
+
+        * wtf/FeatureDefines.h: Remove JSC_OBJC_API_ENABLED.
+
 2013-03-04  Andreas Kling  <akling@apple.com>
 
         Deque: Free internal buffer in clear().
index 11b2f38..88fcf42 100644 (file)
 #define ENABLE_JAVASCRIPT_I18N_API 0
 #endif
 
-#if !defined(JSC_OBJC_API_ENABLED)
-#define JSC_OBJC_API_ENABLED (defined(__clang__) && defined(__APPLE__) && defined(__MAC_OS_X_VERSION_MIN_REQUIRED) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 1090 && !defined(__i386__))
-#endif
-
 #if !defined(ENABLE_LEGACY_CSS_VENDOR_PREFIXES)
 #define ENABLE_LEGACY_CSS_VENDOR_PREFIXES 0
 #endif
index 843ff81..5c02902 100644 (file)
@@ -1,3 +1,14 @@
+2013-03-04  Laszlo Gombos  <l.gombos@samsung.com>
+
+        JSC_OBJC_API_ENABLED should be removed from FeatureDefines.h
+        https://bugs.webkit.org/show_bug.cgi?id=111269
+
+        Reviewed by Mark Hahnenberg.
+
+        Include JavaScriptCore/JSBase.h for the definition of JSC_OBJC_API_ENABLED.
+
+        * WebView/WebDelegateImplementationCaching.h:
+
 2013-03-04  Kunihiko Sakamoto  <ksakamoto@chromium.org>
 
         Add build flag for FontLoader
index 3909eaf..620917d 100644 (file)
@@ -29,6 +29,7 @@
 // This header contains WebView declarations that can be used anywhere in WebKit, but are neither SPI nor API.
 
 #import "WebTypesInternal.h"
+#import <JavaScriptCore/JSBase.h>
 
 @class WebView;