diff options
| author | dsinclair <dsinclair@chromium.org> | 2016-07-12 10:41:43 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-07-12 10:41:43 -0700 |
| commit | 7f9c8609999bf3bd0b33b5b350fb9baa008cf967 (patch) | |
| tree | 67b4ce77bc37a0338fa8de212bf818716032d210 | |
| parent | 08fea801054f3afadb95d0a54efe7da6683c5c5d (diff) | |
| download | pdfium-7f9c8609999bf3bd0b33b5b350fb9baa008cf967.tar.xz | |
Create an fxjs static library
This CL moves the fxjs/ code out of the xfa library and into a fxjs library.
Review-Url: https://codereview.chromium.org/2136273002
| -rw-r--r-- | BUILD.gn | 49 | ||||
| -rw-r--r-- | xfa.gyp | 30 |
2 files changed, 52 insertions, 27 deletions
@@ -870,6 +870,35 @@ static_library("formfiller") { } if (pdf_enable_xfa) { + static_library("fxjs") { + sources = [ + "fxjs/cfxjse_arguments.cpp", + "fxjs/cfxjse_class.cpp", + "fxjs/cfxjse_context.cpp", + "fxjs/cfxjse_isolatetracker.cpp", + "fxjs/cfxjse_isolatetracker.h", + "fxjs/cfxjse_runtimedata.cpp", + "fxjs/cfxjse_runtimedata.h", + "fxjs/cfxjse_value.cpp", + "fxjs/include/cfxjse_arguments.h", + "fxjs/include/cfxjse_class.h", + "fxjs/include/cfxjse_context.h", + "fxjs/include/cfxjse_value.h", + "fxjs/include/fxjse.h", + ] + deps = [ + "//v8:v8_libplatform", + ] + configs += [ ":pdfium_core_config" ] + include_dirs = [ + "//v8", + "//v8/include", + ] + public_deps = [ + "//v8", + ] + } + static_library("fpdfxfa") { sources = [ "fpdfsdk/fpdfxfa/fpdfxfa_app.cpp", @@ -889,19 +918,6 @@ if (pdf_enable_xfa) { static_library("xfa") { sources = [ - "fxjs/cfxjse_arguments.cpp", - "fxjs/cfxjse_class.cpp", - "fxjs/cfxjse_context.cpp", - "fxjs/cfxjse_isolatetracker.cpp", - "fxjs/cfxjse_isolatetracker.h", - "fxjs/cfxjse_runtimedata.cpp", - "fxjs/cfxjse_runtimedata.h", - "fxjs/cfxjse_value.cpp", - "fxjs/include/cfxjse_arguments.h", - "fxjs/include/cfxjse_class.h", - "fxjs/include/cfxjse_context.h", - "fxjs/include/cfxjse_value.h", - "fxjs/include/fxjse.h", "xfa/fde/cfde_path.cpp", "xfa/fde/cfde_path.h", "xfa/fde/cfde_txtedtbuf.cpp", @@ -1442,14 +1458,9 @@ if (pdf_enable_xfa) { ] include_dirs = [ ".", - "//v8", - "//v8/include", - ] - public_deps = [ - "//v8", ] deps = [ - "//v8:v8_libplatform", + ":fxjs", ] configs += [ ":pdfium_core_config", @@ -16,27 +16,22 @@ }, "targets":[ { - "target_name":"xfa", - "type":"static_library", + "target_name": "fxjs", + "type": "static_library", 'include_dirs': [ # This is implicit in GN. '<(DEPTH)', '.', - 'third_party/freetype/include', - 'third_party/freetype/include/freetype', '<(DEPTH)/v8', '<(DEPTH)/v8/include', ], - 'defines' : [ - 'FT2_BUILD_LIBRARY', - ], 'dependencies': [ '<(DEPTH)/v8/src/v8.gyp:v8', ], 'export_dependent_settings': [ '<(DEPTH)/v8/src/v8.gyp:v8', ], - "sources":[ + "sources": [ "fxjs/cfxjse_arguments.cpp", "fxjs/cfxjse_class.cpp", "fxjs/cfxjse_context.cpp", @@ -50,6 +45,25 @@ "fxjs/include/cfxjse_context.h", "fxjs/include/cfxjse_value.h", "fxjs/include/fxjse.h", + ] + }, + { + "target_name":"xfa", + "type":"static_library", + 'include_dirs': [ + # This is implicit in GN. + '<(DEPTH)', + '.', + 'third_party/freetype/include', + 'third_party/freetype/include/freetype', + ], + 'defines' : [ + 'FT2_BUILD_LIBRARY', + ], + 'dependencies': [ + ':fxjs', + ], + "sources":[ "xfa/fde/cfde_path.cpp", "xfa/fde/cfde_path.h", "xfa/fde/cfde_txtedtbuf.cpp", |