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

Unified Diff: mojo/mojo_public.gyp

Issue 1832703002: Mojo: Simplify typemap usage. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@mojo-bindings-variant-import
Patch Set: add missing mash dependency Created 4 years, 8 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 | « mojo/mojo_edk_tests.gyp ('k') | mojo/mojom_bindings_generator.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/mojo_public.gyp
diff --git a/mojo/mojo_public.gyp b/mojo/mojo_public.gyp
index b3773299ad1c31cccb385072503394783bc53b1c..0e994dd27b9b96d3b85a26df1e5dc0bc2bb58f81 100644
--- a/mojo/mojo_public.gyp
+++ b/mojo/mojo_public.gyp
@@ -27,10 +27,7 @@
'public/interfaces/bindings/tests/serialization_test_structs.mojom',
'public/interfaces/bindings/tests/test_constants.mojom',
'public/interfaces/bindings/tests/test_native_types.mojom',
- 'public/interfaces/bindings/tests/test_structs.mojom',
'public/interfaces/bindings/tests/test_sync_methods.mojom',
- 'public/interfaces/bindings/tests/test_unions.mojom',
- 'public/interfaces/bindings/tests/validation_test_interfaces.mojom',
]
},
'targets': [
@@ -391,7 +388,16 @@
'target_name': 'mojo_public_test_interfaces_mojom',
'type': 'none',
'variables': {
- 'mojom_files': '<(mojo_public_test_interfaces_mojom_files)',
+ 'mojom_files': [
+ 'public/interfaces/bindings/tests/test_structs.mojom',
+ 'public/interfaces/bindings/tests/test_unions.mojom',
+ 'public/interfaces/bindings/tests/validation_test_interfaces.mojom',
+ '<@(mojo_public_test_interfaces_mojom_files)',
+ ],
+ 'mojom_typemaps': [
+ 'public/cpp/bindings/tests/rect_chromium.typemap',
+ 'public/cpp/bindings/tests/test_native_types_chromium.typemap',
+ ],
},
'includes': [ 'mojom_bindings_generator_explicit.gypi' ],
},
@@ -399,12 +405,13 @@
'target_name': 'mojo_public_test_interfaces_struct_traits',
'type': 'static_library',
'variables': {
- 'mojom_extra_generator_args': [
- '--typemap', '<(DEPTH)/mojo/public/interfaces/bindings/tests/struct_with_traits.typemap',
+ 'mojom_typemaps': [
+ 'public/cpp/bindings/tests/struct_with_traits.typemap',
],
},
'sources': [
'public/interfaces/bindings/tests/struct_with_traits.mojom',
+ 'public/cpp/bindings/tests/struct_with_traits_impl_traits.cc',
],
'includes': [ 'mojom_bindings_generator.gypi' ],
},
@@ -412,31 +419,15 @@
'target_name': 'mojo_public_test_interfaces_mojom_blink',
'type': 'none',
'variables': {
- 'mojom_variant': 'blink',
- 'mojom_extra_generator_args': [
- '--typemap', '<(DEPTH)/mojo/public/interfaces/bindings/tests/blink_test.typemap',
- ],
- 'mojom_files': '<(mojo_public_test_interfaces_mojom_files)',
- },
- 'includes': [ 'mojom_bindings_generator_explicit.gypi' ],
- 'dependencies': [
- 'mojo_public_test_interfaces_mojom',
- ],
- },
- {
- 'target_name': 'mojo_public_test_interfaces_mojom_chromium',
- 'type': 'none',
- 'variables': {
- 'mojom_variant': 'chromium',
- 'mojom_extra_generator_args': [
- '--typemap', '<(DEPTH)/mojo/public/interfaces/bindings/tests/chromium_test.typemap',
+ 'mojom_variant': 'wtf',
+ 'for_blink': 'true',
+ 'mojom_typemaps': [
+ 'public/cpp/bindings/tests/rect_blink.typemap',
+ 'public/cpp/bindings/tests/test_native_types_blink.typemap',
],
'mojom_files': '<(mojo_public_test_interfaces_mojom_files)',
},
'includes': [ 'mojom_bindings_generator_explicit.gypi' ],
- 'dependencies': [
- 'mojo_public_test_interfaces_mojom',
- ],
},
{
# GN version: //mojo/public/interfaces/bindings/tests:test_interfaces
@@ -445,6 +436,9 @@
'export_dependent_settings': [
'mojo_cpp_bindings',
],
+ 'sources': [
+ 'public/cpp/bindings/tests/pickled_struct_chromium.cc',
+ ],
'dependencies': [
'mojo_public_test_interfaces_mojom',
'mojo_cpp_bindings',
@@ -455,22 +449,14 @@
'target_name': 'mojo_public_test_interfaces_blink',
'type': 'static_library',
'export_dependent_settings': [
- 'mojo_cpp_bindings',
- ],
- 'dependencies': [
'mojo_public_test_interfaces_mojom_blink',
'mojo_cpp_bindings',
],
- },
- {
- # GN version: //mojo/public/interfaces/bindings/tests:test_interfaces_chromium
- 'target_name': 'mojo_public_test_interfaces_chromium',
- 'type': 'static_library',
- 'export_dependent_settings': [
- 'mojo_cpp_bindings',
+ 'sources': [
+ 'public/cpp/bindings/tests/pickled_struct_blink.cc',
],
'dependencies': [
- 'mojo_public_test_interfaces_mojom_chromium',
+ 'mojo_public_test_interfaces_mojom_blink',
'mojo_cpp_bindings',
],
},
@@ -512,7 +498,7 @@
'target_name': 'mojo_public_test_wtf_types_blink',
'type': 'static_library',
'variables': {
- 'mojom_variant': 'blink',
+ 'mojom_variant': 'wtf',
'for_blink': 'true',
},
'sources': [
@@ -520,18 +506,6 @@
],
'includes': [ 'mojom_bindings_generator.gypi' ],
},
- {
- 'target_name': 'mojo_public_test_variant',
- 'type': 'static_library',
- 'variables': {
- 'mojom_variant': 'test_variant',
- },
- 'sources': [
- 'public/interfaces/bindings/tests/test_variant_import.mojom',
- 'public/interfaces/bindings/tests/test_variant.mojom',
- ],
- 'includes': [ 'mojom_bindings_generator.gypi' ],
- },
],
'conditions': [
['OS == "android"', {
« no previous file with comments | « mojo/mojo_edk_tests.gyp ('k') | mojo/mojom_bindings_generator.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698