admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor webrtc/modules/video_processing for GN check

Browse Source 
This moves some GN check configurations out of .gn to individual
targets.

The now checked target is:
"//webrtc/modules/video_processing/*"

BUG=webrtc:6828
NOTRY=True

Review-Url: https://codereview.webrtc.org/2595543002
Cr-Commit-Position: refs/heads/master@{#15732}
This commit is contained in:
mbonadei 2016-12-21 02:01:26 -08:00 committed by Commit bot
parent dbb64d8f27
commit 00a810b844
2 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.gn
View File

@ -33,6 +33,7 @@ check_targets = [
"//webrtc/modules/pacing/*",
"//webrtc/modules/video_capture/*",
"//webrtc/modules/video_coding/*",
"//webrtc/modules/video_processing/*",
"//webrtc/stats:rtc_stats",
"//webrtc/voice_engine",
"//webrtc/voice_engine:level_indicator",

10
webrtc/modules/video_processing/BUILD.gn
View File

@ -26,6 +26,7 @@ rtc_static_library("video_processing") {
]
deps = [
"../../base:rtc_base_approved",
"../../common_audio",
"../../common_video",
"../../modules/utility",
@ -46,11 +47,20 @@ rtc_static_library("video_processing") {
if (build_video_processing_sse2) {
rtc_static_library("video_processing_sse2") {
# TODO(mbonadei): Remove (bugs.webrtc.org/6828)
# Errors on cyclic dependency with :video_processing if enabled.
check_includes = false
sources = [
"util/denoiser_filter_sse2.cc",
"util/denoiser_filter_sse2.h",
]
deps = [
"../../base:rtc_base_approved",
"../../system_wrappers",
]
if (!build_with_chromium && is_clang) {
# Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]