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

Side by Side Diff: chrome/chrome_browser.gypi

Issue 11615002: Exclude PluginService for builds with enable_plugins==0 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments. Excluded more files. Avoid sending ViewHostMsg_GetPlugins Created 8 years 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/ui/webui/version_handler.cc ('k') | chrome/chrome_browser_ui.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2012 The Chromium Authors. All rights reserved. 1 # Copyright 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'targets': [ 6 'targets': [
7 { 7 {
8 'target_name': 'browser', 8 'target_name': 'browser',
9 'type': 'static_library', 9 'type': 'static_library',
10 'variables': { 'enable_wexit_time_destructors': 1, }, 10 'variables': { 'enable_wexit_time_destructors': 1, },
(...skipping 2352 matching lines...) Expand 10 before | Expand all | Expand 10 after
2363 }], 2363 }],
2364 ['enable_plugins==1', { 2364 ['enable_plugins==1', {
2365 'dependencies': [ 2365 'dependencies': [
2366 '../ppapi/ppapi_internal.gyp:ppapi_ipc', 2366 '../ppapi/ppapi_internal.gyp:ppapi_ipc',
2367 ], 2367 ],
2368 }, { # enable_plugins==0 2368 }, { # enable_plugins==0
2369 'sources/': [ 2369 'sources/': [
2370 ['exclude', '^browser/pepper_'], 2370 ['exclude', '^browser/pepper_'],
2371 ['exclude', '^browser/renderer_host/pepper/'], 2371 ['exclude', '^browser/renderer_host/pepper/'],
2372 ], 2372 ],
2373 'sources!': [
2374 'browser/chrome_plugin_service_filter.cc',
2375 'browser/chrome_plugin_service_filter.h',
2376 'browser/plugins/plugin_observer.cc',
jam 2012/12/19 21:32:31 nit: maybe in a followup change since this is alre
nilesh 2012/12/19 21:47:26 Yes. I can tackle this in a followup CL. I will ke
2377 'browser/plugins/plugin_observer.h',
2378 'browser/plugins/plugin_prefs.cc',
2379 'browser/plugins/plugin_prefs.h',
2380 'browser/plugins/plugin_prefs_factory.cc',
2381 'browser/plugins/plugin_prefs_factory.h',
2382 'browser/renderer_host/plugin_info_message_filter.cc',
2383 'browser/renderer_host/plugin_info_message_filter.h',
2384 ],
2373 }], 2385 }],
2374 ['safe_browsing==1', { 2386 ['safe_browsing==1', {
2375 'defines': [ 2387 'defines': [
2376 'FULL_SAFE_BROWSING', 2388 'FULL_SAFE_BROWSING',
2377 ], 2389 ],
2378 }], 2390 }],
2379 ['safe_browsing==2', { 2391 ['safe_browsing==2', {
2380 'defines': [ 2392 'defines': [
2381 'MOBILE_SAFE_BROWSING', 2393 'MOBILE_SAFE_BROWSING',
2382 ], 2394 ],
(...skipping 733 matching lines...) Expand 10 before | Expand all | Expand 10 after
3116 'proto_in_dir': '../third_party/cros_system_api/dbus', 3128 'proto_in_dir': '../third_party/cros_system_api/dbus',
3117 'proto_out_dir': 'chrome/browser/media_transfer_protocol', 3129 'proto_out_dir': 'chrome/browser/media_transfer_protocol',
3118 }, 3130 },
3119 'includes': ['../build/protoc.gypi'], 3131 'includes': ['../build/protoc.gypi'],
3120 }, 3132 },
3121 ], 3133 ],
3122 }, 3134 },
3123 ], 3135 ],
3124 ], 3136 ],
3125 } 3137 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/webui/version_handler.cc ('k') | chrome/chrome_browser_ui.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698