chromium/src
0
Fork 0
Code Activity

Enable -Wexit-time-destructors for remaining Mojo generated C++ code

Browse Source 
Ensure Mojo C++ code generation won't accidentally add numerous exit
time destructors.

Bug: 101600
Change-Id: Ibb64cde704d23fcb63a2b4d5fd7ec5d3245a89aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4655387
Reviewed-by: Ken Rockot <rockot@google.com>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1164272}
This commit is contained in:
Lei Zhang
2023-06-29 19:17:32 +00:00
committed by Chromium LUCI CQ
parent 2933684772
commit d9f69497e8
1 changed files with 2 additions and 0 deletions

2
mojo/public/tools/bindings/mojom.gni

@ -1008,6 +1008,7 @@ template("mojom") {
if (defined(invoker.testonly)) { if (defined(invoker.testonly)) {
testonly = invoker.testonly testonly = invoker.testonly
} }
configs += [ "//build/config/compiler:wexit_time_destructors" ]
deps = [] deps = []
public_deps = [] public_deps = []
if (output_file_base_paths != []) { if (output_file_base_paths != []) {
@ -1892,6 +1893,7 @@ template("mojom") {
"$root_gen_dir/${base_path}-webui-js-bridge-impl.h", "$root_gen_dir/${base_path}-webui-js-bridge-impl.h",
] ]
} }
configs += [ "//build/config/compiler:wexit_time_destructors" ]
deps = [ deps = [
":$cpp_target_name", ":$cpp_target_name",
":$generator_webui_js_bridge_impl_target_name", ":$generator_webui_js_bridge_impl_target_name",