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

Side by Side Diff: mojo/public/tools/bindings/mojom.gni

Issue 2584763002: Mojo C++ bindings: remove support for generating code with mojo::Array/String/Map/WTFArray/WTFMap. (Closed)
Patch Set: Created 4 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
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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 mojom_generator_root = "//mojo/public/tools/bindings" 5 mojom_generator_root = "//mojo/public/tools/bindings"
6 mojom_generator_script = "$mojom_generator_root/mojom_bindings_generator.py" 6 mojom_generator_script = "$mojom_generator_root/mojom_bindings_generator.py"
7 mojom_generator_sources = [ 7 mojom_generator_sources = [
8 "$mojom_generator_root/generators/mojom_cpp_generator.py", 8 "$mojom_generator_root/generators/mojom_cpp_generator.py",
9 "$mojom_generator_root/generators/mojom_js_generator.py", 9 "$mojom_generator_root/generators/mojom_js_generator.py",
10 "$mojom_generator_root/generators/mojom_java_generator.py", 10 "$mojom_generator_root/generators/mojom_java_generator.py",
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 # public_deps (optional) 80 # public_deps (optional)
81 # Note: this can contain only other mojom targets. 81 # Note: this can contain only other mojom targets.
82 # 82 #
83 # import_dirs (optional) 83 # import_dirs (optional)
84 # List of import directories that will get added when processing sources. 84 # List of import directories that will get added when processing sources.
85 # 85 #
86 # testonly (optional) 86 # testonly (optional)
87 # 87 #
88 # visibility (optional) 88 # visibility (optional)
89 # 89 #
90 # use_new_wrapper_types (optional)
91 # If set to true, mojom array/map/string will be mapped to STL (for
92 # chromium variant) or WTF (for blink) types. Otherwise, they will be
93 # mapped to mojo::Array/Map/String/etc.
94 # Default value is true.
95 # TODO(yzshen):
96 # - convert all users to use the new mode;
97 # - remove support for the old mode.
98 #
99 # use_once_callback (optional) 90 # use_once_callback (optional)
100 # If set to true, generated classes will use base::OnceCallback instead of 91 # If set to true, generated classes will use base::OnceCallback instead of
101 # base::RepeatingCallback. 92 # base::RepeatingCallback.
102 # Default value is false. 93 # Default value is false.
103 # TODO(dcheng): 94 # TODO(dcheng):
104 # - Convert everything to use OnceCallback. 95 # - Convert everything to use OnceCallback.
105 # - Remove support for the old mode. 96 # - Remove support for the old mode.
106 # 97 #
107 # cpp_only (optional) 98 # cpp_only (optional)
108 # If set to true, only the C++ bindings targets will be generated. 99 # If set to true, only the C++ bindings targets will be generated.
(...skipping 247 matching lines...) Expand 10 before | Expand all | Expand 10 after
356 if (defined(invoker.export_class_attribute)) { 347 if (defined(invoker.export_class_attribute)) {
357 args += [ 348 args += [
358 "--export_attribute", 349 "--export_attribute",
359 invoker.export_class_attribute, 350 invoker.export_class_attribute,
360 "--export_header", 351 "--export_header",
361 invoker.export_header, 352 invoker.export_header,
362 ] 353 ]
363 } 354 }
364 } 355 }
365 356
366 if (!defined(invoker.use_new_wrapper_types) ||
367 invoker.use_new_wrapper_types) {
368 args += [ "--use_new_wrapper_types" ]
369 }
370
371 if (defined(invoker.use_once_callback) && invoker.use_once_callback) { 357 if (defined(invoker.use_once_callback) && invoker.use_once_callback) {
372 args += [ "--use_once_callback" ] 358 args += [ "--use_once_callback" ]
373 } 359 }
374 } 360 }
375 } 361 }
376 362
377 action(type_mappings_target_name) { 363 action(type_mappings_target_name) {
378 inputs = _bindings_configuration_files 364 inputs = _bindings_configuration_files
379 outputs = [ 365 outputs = [
380 type_mappings_path, 366 type_mappings_path,
(...skipping 187 matching lines...) Expand 10 before | Expand all | Expand 10 after
568 full_name = get_label_info(d, "label_no_toolchain") 554 full_name = get_label_info(d, "label_no_toolchain")
569 deps += [ "${full_name}_java" ] 555 deps += [ "${full_name}_java" ]
570 } 556 }
571 557
572 srcjar_deps = [ ":$java_srcjar_target_name" ] 558 srcjar_deps = [ ":$java_srcjar_target_name" ]
573 run_findbugs_override = false 559 run_findbugs_override = false
574 } 560 }
575 } 561 }
576 } 562 }
577 } 563 }
OLDNEW
« no previous file with comments | « mojo/public/tools/bindings/generators/mojom_cpp_generator.py ('k') | mojo/public/tools/bindings/mojom_bindings_generator.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698