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

Unified Diff: chrome/chrome_browser.gypi

Issue 11414180: Add a gyp flag to allow removing dependency on ppapi. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed Jay's comments and rebase Created 8 years, 1 month 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: chrome/chrome_browser.gypi
diff --git a/chrome/chrome_browser.gypi b/chrome/chrome_browser.gypi
index cab7e61d945191ac72730c83773459d977df6889..303156878441051ffa6341c86143f1d0e37c6fca 100644
--- a/chrome/chrome_browser.gypi
+++ b/chrome/chrome_browser.gypi
@@ -2211,7 +2211,6 @@
'../cc/cc.gyp:cc',
'../content/content.gyp:navigation_interception',
'../net/net.gyp:net_with_v8',
- '../ppapi/ppapi_internal.gyp:ppapi_ipc', # For PpapiMsg_LoadPlugin
'../printing/printing.gyp:printing',
'../third_party/adobe/flash/flash_player.gyp:flapper_version_h',
'../third_party/expat/expat.gyp:expat',
@@ -2360,6 +2359,16 @@
['include', 'browser/policy/policy_service_stub.h'],
],
}],
+ ['enable_ppapi==1', {
+ 'dependencies': [
+ '../ppapi/ppapi_internal.gyp:ppapi_ipc', # For PpapiMsg_LoadPlugin
brettw 2012/11/30 23:18:12 Can you just delete the comment here? It's out of
nilesh 2012/12/01 00:41:35 Done.
+ ],
+ }, { # enable_ppapi==0
+ 'sources/': [
+ ['exclude', '^browser/pepper_'],
+ ['exclude', '^browser/renderer_host/pepper/'],
+ ],
+ }],
['safe_browsing==1', {
'defines': [
'FULL_SAFE_BROWSING',

Powered by Google App Engine
This is Rietveld 408576698