Commit 35a3eb33 authored by Sylvain Defresne's avatar Sylvain Defresne Committed by Commit Bot

Clean up deprecated sources assignment filter in //chrome/services

Remove deprecated_default_sources_assignment_filter and replace
it with explicit is_win, is_mac, etc. sections.

Bug: 1018739
Change-Id: Ib4a41b401603cd96852136f038cbb358cf4605a0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2445616Reviewed-by: default avatarKen Rockot <rockot@google.com>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#813667}
parent 44668ef6
......@@ -5,26 +5,23 @@
import("//build/config/features.gni")
import("//components/safe_browsing/buildflags.gni")
# This file depends on the legacy global sources assignment filter. It should
# be converted to check target platform before assigning source files to the
# sources variable. Remove this import and set_sources_assignment_filter call
# when the file has been converted. See https://crbug.com/1018739 for details.
import("//build/config/deprecated_default_sources_assignment_filter.gni")
set_sources_assignment_filter(deprecated_default_sources_assignment_filter)
source_set("cpp") {
deps = [ "//chrome/common/safe_browsing:archive_analyzer_results" ]
public_deps = [ "//chrome/services/file_util/public/mojom" ]
if (safe_browsing_mode == 1) {
sources = [
"sandboxed_dmg_analyzer_mac.cc",
"sandboxed_dmg_analyzer_mac.h",
"sandboxed_rar_analyzer.cc",
"sandboxed_rar_analyzer.h",
"sandboxed_zip_analyzer.cc",
"sandboxed_zip_analyzer.h",
]
if (is_mac) {
sources += [
"sandboxed_dmg_analyzer_mac.cc",
"sandboxed_dmg_analyzer_mac.h",
]
}
if (is_chromeos) {
sources += [
"zip_file_creator.cc",
......@@ -46,10 +43,12 @@ if (safe_browsing_mode == 1) {
testonly = true
sources = [
"sandboxed_dmg_analyzer_mac_unittest.cc",
"sandboxed_rar_analyzer_unittest.cc",
"sandboxed_zip_analyzer_unittest.cc",
]
if (is_mac) {
sources += [ "sandboxed_dmg_analyzer_mac_unittest.cc" ]
}
deps = [
":cpp",
......
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