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

Unified Diff: Source/web/web.gyp

Issue 26247009: Remove web/{android, gtk, win} (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Corrected web.gypi! Created 7 years, 2 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
« no previous file with comments | « Source/web/mac/WebInputEventFactory.mm ('k') | Source/web/web.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/web/web.gyp
diff --git a/Source/web/web.gyp b/Source/web/web.gyp
index 1529ee002fdd201c0786a3706b73570bf16b6486..394828e5bf8f3b82a666362ac8d297ac289bbacc 100644
--- a/Source/web/web.gyp
+++ b/Source/web/web.gyp
@@ -174,9 +174,14 @@
'include_dirs': [
'../../public/web/gtk',
],
+ 'sources/': [
+ ['exclude', 'WebInputEventFactoryAndroid.cpp$'],
tfarina 2013/10/17 11:06:20 this shouldn't be necessary, see config.gyp.
r.kasibhatla 2013/10/17 13:00:42 In config.gyp, I see only condition for OS==win, w
+ ['exclude', 'WebInputEventFactoryMac.mm$'],
+ ['exclude', 'WebInputEventFactoryWin.cpp$'],
+ ],
}, { # else: toolkit_uses_gtk != 1
'sources/': [
- ['exclude', 'gtk/'],
+ ['exclude', 'WebInputEventFactoryGtk.cpp$'],
],
}],
['OS=="android"', {
@@ -184,15 +189,25 @@
'../../public/web/android',
'../../public/web/linux', # We need linux/WebFontRendering.h on Android.
],
+ 'sources/': [
+ ['exclude', 'WebInputEventFactoryGtk.cpp$'],
tkent 2013/10/20 23:03:40 These exclude looks unnecessary. For example, If O
r.kasibhatla 2013/10/21 04:05:40 Done.
+ ['exclude', 'WebInputEventFactoryMac.mm$'],
+ ['exclude', 'WebInputEventFactoryWin.cpp$'],
+ ],
}, { # else: OS!="android"
'sources/': [
- ['exclude', 'android/'],
+ ['exclude', 'WebInputEventFactoryAndroid.cpp$'],
],
}],
['OS=="mac"', {
'include_dirs': [
'../../public/web/mac',
],
+ 'sources/': [
+ ['exclude', 'WebInputEventFactoryAndroid.cpp$'],
+ ['exclude', 'WebInputEventFactoryGtk.cpp$'],
+ ['exclude', 'WebInputEventFactoryWin.cpp$'],
+ ],
'link_settings': {
'libraries': [
'$(SDKROOT)/System/Library/Frameworks/Accelerate.framework',
@@ -201,16 +216,21 @@
},
}, { # else: OS!="mac"
'sources/': [
- ['exclude', 'mac/'],
+ ['exclude', 'WebInputEventFactoryMac.mm$'],
],
}],
['OS=="win"', {
'include_dirs': [
'../../public/web/win',
],
+ 'sources/': [
+ ['exclude', 'WebInputEventFactoryAndroid.cpp$'],
+ ['exclude', 'WebInputEventFactoryGtk.cpp$'],
+ ['exclude', 'WebInputEventFactoryMac.mm$'],
+ ],
}, { # else: OS!="win"
'sources/': [
- ['exclude', 'win/']
+ ['exclude', 'WebInputEventFactoryWin.cpp$'],
],
'variables': {
# FIXME: Turn on warnings on Windows.
« no previous file with comments | « Source/web/mac/WebInputEventFactory.mm ('k') | Source/web/web.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698