Skip to content

Commit

Permalink
build: Remove no-op calls to set_sources_assignment_filter
Browse files Browse the repository at this point in the history
Chromiun no longer use set_sources_assignment_filter() anywhere in the
build, so these are no longer needed.

Bug: chromium:1018739
Change-Id: I7b33612d925563ebca0d93a7d3c9183d7305b7b0
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2456988
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#70405}
  • Loading branch information
sdefresne authored and Commit Bot committed Oct 8, 2020
1 parent 5bf1619 commit 92e6d33
Showing 1 changed file with 4 additions and 14 deletions.
18 changes: 4 additions & 14 deletions testing/gtest/BUILD.gn
Expand Up @@ -4,9 +4,9 @@

import("//build_overrides/gtest.gni")
if (is_ios) {
import("//build/buildflag_header.gni")
import("//build/config/coverage/coverage.gni")
import("//build/config/ios/ios_sdk.gni")
import("//build/buildflag_header.gni")
}

config("gtest_direct_config") {
Expand Down Expand Up @@ -40,9 +40,7 @@ static_library("gtest") {
# Android. https://codereview.chromium.org/2852613002/#ps20001
"empty.cc",
]
public_deps = [
"//third_party/googletest:gtest",
]
public_deps = [ "//third_party/googletest:gtest" ]

public_configs = [ ":gtest_direct_config" ]

Expand All @@ -58,27 +56,21 @@ static_library("gtest") {
}

if ((is_mac || is_ios) && gtest_include_objc_support) {
if (is_ios) {
set_sources_assignment_filter([])
}
sources += [
"../gtest_mac.h",
"../gtest_mac.mm",
]
if (gtest_include_platform_test) {
sources += [ "../platform_test_mac.mm" ]
}
set_sources_assignment_filter(sources_assignment_filter)
}

if (is_ios && gtest_include_ios_coverage) {
sources += [
"../coverage_util_ios.h",
"../coverage_util_ios.mm",
]
deps = [
":ios_enable_coverage",
]
deps = [ ":ios_enable_coverage" ]
}
}

Expand All @@ -87,9 +79,7 @@ static_library("gtest") {
# into //third_party/googletest.
source_set("gtest_main") {
testonly = true
deps = [
"//third_party/googletest:gtest_main",
]
deps = [ "//third_party/googletest:gtest_main" ]
}

if (is_ios) {
Expand Down

0 comments on commit 92e6d33

Please sign in to comment.