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

Unified Diff: webkit/tools/test_shell/test_shell.gypi

Issue 2729004: Move npapi_layout_test_plugin target to webkit_support to use it from... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 6 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 | « webkit/tools/npapi_layout_test_plugin/npapi_layout_test_plugin.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/tools/test_shell/test_shell.gypi
===================================================================
--- webkit/tools/test_shell/test_shell.gypi (revision 49234)
+++ webkit/tools/test_shell/test_shell.gypi (working copy)
@@ -40,10 +40,10 @@
'<(DEPTH)/webkit/support/webkit_support.gyp:appcache',
'<(DEPTH)/webkit/support/webkit_support.gyp:database',
'<(DEPTH)/webkit/support/webkit_support.gyp:glue',
+ '<(DEPTH)/webkit/support/webkit_support.gyp:npapi_layout_test_plugin',
'<(DEPTH)/webkit/support/webkit_support.gyp:webkit_resources',
'<(DEPTH)/webkit/support/webkit_support.gyp:webkit_support',
'<(DEPTH)/webkit/webkit.gyp:inspector_resources',
- 'npapi_layout_test_plugin',
],
'msvs_guid': '77C32787-1B96-CB84-B905-7F170629F0AC',
'sources': [
@@ -487,87 +487,6 @@
}],
],
},
- {
- 'target_name': 'npapi_layout_test_plugin',
- 'type': 'loadable_module',
- 'variables': {
- 'chromium_code': 1,
- },
- 'mac_bundle': 1,
- 'msvs_guid': 'BE6D5659-A8D5-4890-A42C-090DD10EF62C',
- 'sources': [
- '../npapi_layout_test_plugin/PluginObject.cpp',
- '../npapi_layout_test_plugin/TestObject.cpp',
- '../npapi_layout_test_plugin/main.cpp',
- '../npapi_layout_test_plugin/npapi_layout_test_plugin.def',
- '../npapi_layout_test_plugin/npapi_layout_test_plugin.rc',
- ],
- 'include_dirs': [
- '../../..',
- ],
- 'dependencies': [
- '<(DEPTH)/third_party/npapi/npapi.gyp:npapi',
- '<(DEPTH)/third_party/WebKit/JavaScriptCore/JavaScriptCore.gyp/JavaScriptCore.gyp:wtf',
- ],
- 'msvs_disabled_warnings': [ 4996 ],
- 'mac_bundle_resources': [
- '../npapi_layout_test_plugin/Info.r',
- ],
- 'xcode_settings': {
- 'INFOPLIST_FILE': '<(DEPTH)/webkit/tools/npapi_layout_test_plugin/Info.plist',
- },
- 'conditions': [
- ['OS!="win"', {
- 'sources!': [
- '../npapi_layout_test_plugin/npapi_layout_test_plugin.def',
- '../npapi_layout_test_plugin/npapi_layout_test_plugin.rc',
- ],
- # TODO(bradnelson):
- # This copy should really live here, as a post-build step,
- # but it's currently being implemented via
- # AdditionalDependencies, which tries to do the copy before
- # the file is built...
- #
- }, { # OS == "win"
- # # The old VS build would explicitly copy the .dll into the
- # # plugins subdirectory like this. It might be possible to
- # # use the 'product_dir' setting to build directly into
- # # plugins/ (as is done on Linux), but we'd need to verify
- # # that nothing breaks first.
- # 'copies': [
- # {
- # 'destination': '<(PRODUCT_DIR)/plugins',
- # 'files': ['<(PRODUCT_DIR)/npapi_layout_test_plugin.dll'],
- # },
- # ],
- 'variables': {
- # This is not a relative pathname. Avoid pathname relativization
- # by sticking it in a variable that isn't recognized as one
- # containing pathnames, and by using the >(late) form of variable
- # expansion.
- 'winmm_lib': 'winmm.lib',
- },
- 'link_settings': {
- 'libraries': [
- '>(winmm_lib)',
- ],
- },
- }],
- ['OS=="mac"', {
- 'product_name': 'TestNetscapePlugIn',
- 'product_extension': 'plugin',
- 'link_settings': {
- 'libraries': [
- '$(SDKROOT)/System/Library/Frameworks/Carbon.framework',
- ],
- },
- }],
- ['(OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="solaris") and (target_arch=="x64" or target_arch=="arm")', {
- # Shared libraries need -fPIC on x86-64
- 'cflags': ['-fPIC']
- }],
- ],
- },
],
'conditions': [
['target_arch!="x64" and target_arch!="arm"', {
« no previous file with comments | « webkit/tools/npapi_layout_test_plugin/npapi_layout_test_plugin.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698