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

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

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

Bug: 1018739
Change-Id: I6c3fe6e56e5ad85ffe02efe806f4a087079f31a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2445778Reviewed-by: default avatarSorin Jianu <sorin@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#813658}
parent ac83b0e7
...@@ -7,13 +7,6 @@ import("//build/util/process_version.gni") ...@@ -7,13 +7,6 @@ import("//build/util/process_version.gni")
import("//build/util/version.gni") import("//build/util/version.gni")
import("//chrome/updater/branding.gni") import("//chrome/updater/branding.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)
app_name = "UpdaterTestApp" app_name = "UpdaterTestApp"
app_bundle_id = "org.chromium.updatertestapp" app_bundle_id = "org.chromium.updatertestapp"
...@@ -58,14 +51,8 @@ source_set("app_sources") { ...@@ -58,14 +51,8 @@ source_set("app_sources") {
"//chrome/updater/app/app.h", "//chrome/updater/app/app.h",
"test_app.cc", "test_app.cc",
"test_app.h", "test_app.h",
"test_app_mac.mm",
"test_app_win.cc",
"update_client.cc", "update_client.cc",
"update_client.h", "update_client.h",
"update_client_mac.h",
"update_client_mac.mm",
"update_client_win.cc",
"update_client_win.h",
] ]
deps = [ deps = [
...@@ -83,6 +70,9 @@ source_set("app_sources") { ...@@ -83,6 +70,9 @@ source_set("app_sources") {
"//chrome/updater/app/server/mac/service_protocol.mm", "//chrome/updater/app/server/mac/service_protocol.mm",
"//chrome/updater/app/server/mac/update_service_wrappers.h", "//chrome/updater/app/server/mac/update_service_wrappers.h",
"//chrome/updater/app/server/mac/update_service_wrappers.mm", "//chrome/updater/app/server/mac/update_service_wrappers.mm",
"test_app_mac.mm",
"update_client_mac.h",
"update_client_mac.mm",
] ]
deps += [ "//chrome/updater/mac:xpc_names" ] deps += [ "//chrome/updater/mac:xpc_names" ]
...@@ -91,6 +81,11 @@ source_set("app_sources") { ...@@ -91,6 +81,11 @@ source_set("app_sources") {
} }
if (is_win) { if (is_win) {
sources += [
"test_app_win.cc",
"update_client_win.cc",
"update_client_win.h",
]
deps += [ deps += [
"//chrome/updater/app/server/win:updater_idl_idl", "//chrome/updater/app/server/win:updater_idl_idl",
"//chrome/updater/win:lib", "//chrome/updater/win:lib",
......
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