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

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

Issue 2612253003: Add visibility restrictions to all mojoms that export symbols. (Closed)
Patch Set: Created 3 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 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 # visibility_blink (optional)
91 # The value to use for visibility for the blink variant. If unset,
92 # |visibility| is used.
93 #
90 # use_once_callback (optional) 94 # use_once_callback (optional)
91 # If set to true, generated classes will use base::OnceCallback instead of 95 # If set to true, generated classes will use base::OnceCallback instead of
92 # base::RepeatingCallback. 96 # base::RepeatingCallback.
93 # Default value is false. 97 # Default value is false.
94 # TODO(dcheng): 98 # TODO(dcheng):
95 # - Convert everything to use OnceCallback. 99 # - Convert everything to use OnceCallback.
96 # - Remove support for the old mode. 100 # - Remove support for the old mode.
97 # 101 #
98 # cpp_only (optional) 102 # cpp_only (optional)
99 # If set to true, only the C++ bindings targets will be generated. 103 # If set to true, only the C++ bindings targets will be generated.
(...skipping 328 matching lines...) Expand 10 before | Expand all | Expand 10 after
428 } 432 }
429 foreach(value, typemap.type_mappings) { 433 foreach(value, typemap.type_mappings) {
430 typemap_description += [ "type_mappings=$value" ] 434 typemap_description += [ "type_mappings=$value" ]
431 } 435 }
432 } 436 }
433 args += typemap_description 437 args += typemap_description
434 } 438 }
435 } 439 }
436 440
437 source_set("${target_name}${variant_suffix}") { 441 source_set("${target_name}${variant_suffix}") {
438 if (defined(invoker.visibility)) { 442 if (defined(bindings_configuration.for_blink) &&
443 bindings_configuration.for_blink &&
444 defined(invoker.visibility_blink)) {
445 visibility = invoker.visibility_blink
446 } else if (defined(invoker.visibility)) {
439 visibility = invoker.visibility 447 visibility = invoker.visibility
440 } 448 }
441 if (defined(invoker.testonly)) { 449 if (defined(invoker.testonly)) {
442 testonly = invoker.testonly 450 testonly = invoker.testonly
443 } 451 }
444 if (defined(invoker.sources) && !defined(bindings_configuration.variant)) { 452 if (defined(invoker.sources) && !defined(bindings_configuration.variant)) {
445 data = process_file_template(enabled_sources, generator_js_outputs) 453 data = process_file_template(enabled_sources, generator_js_outputs)
446 } 454 }
447 defines = [] 455 defines = []
448 if (defined(invoker.testonly)) { 456 if (defined(invoker.testonly)) {
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
568 full_name = get_label_info(d, "label_no_toolchain") 576 full_name = get_label_info(d, "label_no_toolchain")
569 deps += [ "${full_name}_java" ] 577 deps += [ "${full_name}_java" ]
570 } 578 }
571 579
572 srcjar_deps = [ ":$java_srcjar_target_name" ] 580 srcjar_deps = [ ":$java_srcjar_target_name" ]
573 run_findbugs_override = false 581 run_findbugs_override = false
574 } 582 }
575 } 583 }
576 } 584 }
577 } 585 }
OLDNEW
« no previous file with comments | « mojo/public/interfaces/bindings/tests/BUILD.gn ('k') | third_party/WebKit/Source/bindings/modules/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698