Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(360)

Unified Diff: webkit/features.gypi

Issue 208029: Upstreamings parts of webkit.gyp (step 2) (Closed)
Patch Set: removed hash Created 11 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webkit/features.gypi
diff --git a/webkit/features.gypi b/webkit/features.gypi
index 9cece5ee85cde460df468df8e8ba5ddcf5b48dc7..3f93b332d9ae4cffd1289553f8382b8db32a5202 100644
--- a/webkit/features.gypi
+++ b/webkit/features.gypi
@@ -3,6 +3,10 @@
# found in the LICENSE file.
{
+ 'includes': {
+ # TODO(yaar) Include upstream (webkit.org) features.gypi here, so
+ # that this file inherits upstream feature_defines.
+ },
'variables': {
'feature_defines': [
'ENABLE_CHANNEL_MESSAGING=1',
@@ -27,62 +31,5 @@
'ENABLE_VIDEO=1',
'ENABLE_WORKERS=1',
],
- 'non_feature_defines': [
- 'BUILDING_CHROMIUM__=1',
- 'USE_GOOGLE_URL_LIBRARY=1',
- 'USE_SYSTEM_MALLOC=1',
- 'WEBCORE_NAVIGATOR_VENDOR="Google Inc."',
- ],
- 'conditions': [
- ['OS=="linux"', {
- 'non_feature_defines': [
- # Mozilla on Linux effectively uses uname -sm, but when running
- # 32-bit x86 code on an x86_64 processor, it uses
- # "Linux i686 (x86_64)". Matching that would require making a
- # run-time determination.
- 'WEBCORE_NAVIGATOR_PLATFORM="Linux i686"',
- ],
- }],
- ['OS=="mac"', {
- 'non_feature_defines': [
- # Ensure that only Leopard features are used when doing the Mac build.
- 'BUILDING_ON_LEOPARD',
- # Match Safari and Mozilla on Mac x86.
- 'WEBCORE_NAVIGATOR_PLATFORM="MacIntel"',
-
- # Chromium's version of WebCore includes the following Objective-C
- # classes. The system-provided WebCore framework may also provide
- # these classes. Because of the nature of Objective-C binding
- # (dynamically at runtime), it's possible for the Chromium-provided
- # versions to interfere with the system-provided versions. This may
- # happen when a system framework attempts to use WebCore.framework,
- # such as when converting an HTML-flavored string to an
- # NSAttributedString. The solution is to force Objective-C class
- # names that would conflict to use alternate names.
-
- # TODO(mark) This list will hopefully shrink but may also grow.
- # Periodically run:
- # nm libwebcore.a | grep -E '[atsATS] ([+-]\[|\.objc_class_name)'
- # and make sure that everything listed there has the alternate
- # ChromiumWebCoreObjC name, and that nothing extraneous is listed
- # here. If all Objective-C can be eliminated from Chromium's WebCore
- # library, these defines should be removed entirely.
- # TODO(yaar) move these out of command line defines.
- 'ScrollbarPrefsObserver=ChromiumWebCoreObjCScrollbarPrefsObserver',
- 'WebCoreRenderThemeNotificationObserver=ChromiumWebCoreObjCWebCoreRenderThemeNotificationObserver',
- 'WebFontCache=ChromiumWebCoreObjCWebFontCache',
-
- # Use USE_NEW_THEME on Mac.
- 'WTF_USE_NEW_THEME=1',
- ],
- }],
- ['OS=="win"', {
- 'non_feature_defines': [
- 'CRASH=__debugbreak',
- # Match Safari and Mozilla on Windows.
- 'WEBCORE_NAVIGATOR_PLATFORM="Win32"',
- ],
- }],
- ],
- }, # variables
+ },
}
« no previous file with comments | « webkit/config.gyp ('k') | webkit/javascriptcore.gyp » ('j') | webkit/webcore.gyp » ('J')

Powered by Google App Engine
This is Rietveld 408576698