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

Unified Diff: pdf/pdf.gyp

Issue 799643004: Combine PDF plugin into the Chromium binary. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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 | « pdf/pdf.def ('k') | pdf/pdf.rc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pdf/pdf.gyp
diff --git a/pdf/pdf.gyp b/pdf/pdf.gyp
index 022592153f048f8c38e101102d434f1adf5c9e45..d49a0f1c62618e66ae59304583578b1f31e471ac 100644
--- a/pdf/pdf.gyp
+++ b/pdf/pdf.gyp
@@ -3,28 +3,18 @@
'chromium_code': 1,
'pdf_engine%': 0, # 0 PDFium
},
- 'target_defaults': {
- 'cflags': [
- '-fPIC',
- ],
- },
'targets': [
{
'target_name': 'pdf',
- 'type': 'loadable_module',
- 'msvs_guid': '647863C0-C7A3-469A-B1ED-AD7283C34BED',
+ 'type': 'static_library',
'dependencies': [
'../base/base.gyp:base',
+ '../components/components.gyp:ui_zoom',
+ '../content/content.gyp:content_common',
'../net/net.gyp:net',
- '../ppapi/ppapi.gyp:ppapi_cpp',
jochen (gone - plz use gerrit) 2015/02/11 08:30:01 this breaks compilation with disable_nacl=1, becau
+ '../ppapi/ppapi.gyp:ppapi_internal_module',
'../third_party/pdfium/pdfium.gyp:pdfium',
],
- 'xcode_settings': {
- 'INFOPLIST_FILE': 'Info.plist',
- },
- 'mac_framework_dirs': [
- '$(SDKROOT)/System/Library/Frameworks/ApplicationServices.framework/Frameworks',
- ],
'ldflags': [ '-L<(PRODUCT_DIR)',],
'sources': [
'button.h',
@@ -55,7 +45,6 @@
'paint_manager.h',
'pdf.cc',
'pdf.h',
- 'pdf.rc',
'progress_control.cc',
'progress_control.h',
'pdf_engine.h',
@@ -65,8 +54,6 @@
'resource_consts.h',
'thumbnail_control.cc',
'thumbnail_control.h',
- '../components/ui/zoom/page_zoom_constants.cc',
- '../content/common/page_zoom.cc',
],
'conditions': [
['pdf_engine==0', {
@@ -86,117 +73,11 @@
'pdfium/pdfium_range.h',
],
}],
- ['OS!="win"', {
- 'sources!': [
- 'pdf.rc',
- ],
- }],
- ['OS=="mac"', {
- 'mac_bundle': 1,
- 'product_name': 'PDF',
- 'product_extension': 'plugin',
- # Strip the shipping binary of symbols so "Foxit" doesn't appear in
- # the binary. Symbols are stored in a separate .dSYM.
- 'variables': {
- 'mac_real_dsym': 1,
- },
- 'sources+': [
- 'Info.plist'
- ],
- }],
['OS=="win"', {
- 'defines': [
- 'COMPILE_CONTENT_STATICALLY',
- ],
# TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
'msvs_disabled_warnings': [ 4267, ],
}],
- ['OS=="linux"', {
- 'configurations': {
- 'Release_Base': {
- #'cflags': [ '-fno-weak',], # get rid of symbols that strip doesn't remove.
- # Don't do this for now since official builder will take care of it. That
- # way symbols can still be uploaded to the crash server.
- #'ldflags': [ '-s',], # strip local symbols from binary.
- },
- },
- }],
],
},
],
- 'conditions': [
- # CrOS has a separate step to do this.
- ['OS=="linux" and chromeos==0',
- { 'targets': [
- {
- 'target_name': 'pdf_linux_symbols',
- 'type': 'none',
- 'conditions': [
- ['linux_dump_symbols==1', {
- 'actions': [
- {
- 'action_name': 'dump_symbols',
- 'inputs': [
- '<(DEPTH)/build/linux/dump_app_syms',
- '<(PRODUCT_DIR)/dump_syms',
- '<(PRODUCT_DIR)/libpdf.so',
- ],
- 'outputs': [
- '<(PRODUCT_DIR)/libpdf.so.breakpad.<(target_arch)',
- ],
- 'action': ['<(DEPTH)/build/linux/dump_app_syms',
- '<(PRODUCT_DIR)/dump_syms',
- '<(linux_strip_binary)',
- '<(PRODUCT_DIR)/libpdf.so',
- '<@(_outputs)'],
- 'message': 'Dumping breakpad symbols to <(_outputs)',
- 'process_outputs_as_sources': 1,
- },
- ],
- 'dependencies': [
- 'pdf',
- '../breakpad/breakpad.gyp:dump_syms',
- ],
- }],
- ],
- },
- ],
- },], # OS=="linux" and chromeos==0
- ['OS=="win" and fastbuild==0 and target_arch=="ia32" and syzyasan==1', {
- 'variables': {
- 'dest_dir': '<(PRODUCT_DIR)/syzygy',
- },
- 'targets': [
- {
- 'target_name': 'pdf_syzyasan',
- 'type': 'none',
- 'sources' : [],
- 'dependencies': [
- 'pdf',
- ],
- # Instrument PDFium with SyzyAsan.
- 'actions': [
- {
- 'action_name': 'Instrument PDFium with SyzyAsan',
- 'inputs': [
- '<(PRODUCT_DIR)/pdf.dll',
- ],
- 'outputs': [
- '<(dest_dir)/pdf.dll',
- '<(dest_dir)/pdf.dll.pdb',
- ],
- 'action': [
- 'python',
- '<(DEPTH)/chrome/tools/build/win/syzygy/instrument.py',
- '--mode', 'asan',
- '--input_executable', '<(PRODUCT_DIR)/pdf.dll',
- '--input_symbol', '<(PRODUCT_DIR)/pdf.dll.pdb',
- '--destination_dir', '<(dest_dir)',
- ],
- },
- ],
- },
- ],
- }], # OS=="win" and fastbuild==0 and target_arch=="ia32" and syzyasan==1
- ],
}
« no previous file with comments | « pdf/pdf.def ('k') | pdf/pdf.rc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698