Commit ef910780 authored by Sylvain Defresne's avatar Sylvain Defresne Committed by Commit Bot

Use filter_{exclude,include} to filter list

Instead of abusing set_sources_assignement_filter, use the two
gn function filter_{exclude,include} to partition the list in
two.

Expected to be a no-op change.

Bug: 1018739
Change-Id: If6d5e2d8c300358bb31b8773d629cdb95f09421b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2440583
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Yuki Shiino <yukishiino@chromium.org>
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: default avatarYuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#812166}
parent 65a6d971
......@@ -92,17 +92,12 @@ group("bindings_modules_v8_generated") {
idl_compiler("generate_bindings_modules_v8_interfaces") {
if (use_blink_v8_binding_new_idl_interface) {
# modules_idl_files = interfaces + callback interfaces
# modules_non_callback_idl_files = interfaces only
set_sources_assignment_filter([ "*_callback.idl" ])
sources = modules_idl_files
modules_interface_idl_files = sources
if (use_blink_v8_binding_new_idl_callback_interface) {
modules_callback_interface_idl_files =
modules_idl_files - modules_interface_idl_files
}
set_sources_assignment_filter([])
sources = []
modules_interface_idl_files =
filter_exclude(modules_idl_files, [ "*_callback.idl" ])
}
if (use_blink_v8_binding_new_idl_callback_interface) {
modules_callback_interface_idl_files =
filter_include(modules_idl_files, [ "*_callback.idl" ])
}
sources = modules_definition_idl_files
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment