Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(229)

Side by Side Diff: webrtc/common_video/BUILD.gn

Issue 2408133008: GN: Exclude suppressions of Chromium Clang warnings for Chromium builds. (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « webrtc/common_audio/BUILD.gn ('k') | webrtc/examples/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved. 1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license 3 # Use of this source code is governed by a BSD-style license
4 # that can be found in the LICENSE file in the root of the source 4 # that can be found in the LICENSE file in the root of the source
5 # tree. An additional intellectual property rights grant can be found 5 # tree. An additional intellectual property rights grant can be found
6 # in the file PATENTS. All contributing project authors may 6 # in the file PATENTS. All contributing project authors may
7 # be found in the AUTHORS file in the root of the source tree. 7 # be found in the AUTHORS file in the root of the source tree.
8 8
9 import("../build/webrtc.gni") 9 import("../build/webrtc.gni")
10 10
(...skipping 27 matching lines...) Expand all
38 "video_frame.cc", 38 "video_frame.cc",
39 "video_frame_buffer.cc", 39 "video_frame_buffer.cc",
40 "video_render_frames.cc", 40 "video_render_frames.cc",
41 "video_render_frames.h", 41 "video_render_frames.h",
42 ] 42 ]
43 43
44 include_dirs = [ "../modules/interface" ] 44 include_dirs = [ "../modules/interface" ]
45 45
46 public_configs = [ ":common_video_config" ] 46 public_configs = [ ":common_video_config" ]
47 47
48 if (is_clang) { 48 if (!build_with_chromium && is_clang) {
49 # Suppress warnings from Chrome's Clang plugins. 49 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
50 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
51 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ] 50 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
52 } 51 }
53 52
54 deps = [ 53 deps = [
55 "..:webrtc_common", 54 "..:webrtc_common",
56 "../base:rtc_task_queue", 55 "../base:rtc_task_queue",
57 "../system_wrappers", 56 "../system_wrappers",
58 ] 57 ]
59 58
60 if (rtc_build_libyuv) { 59 if (rtc_build_libyuv) {
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 "h264/sps_parser_unittest.cc", 97 "h264/sps_parser_unittest.cc",
99 "h264/sps_vui_rewriter_unittest.cc", 98 "h264/sps_vui_rewriter_unittest.cc",
100 "i420_buffer_pool_unittest.cc", 99 "i420_buffer_pool_unittest.cc",
101 "i420_video_frame_unittest.cc", 100 "i420_video_frame_unittest.cc",
102 "libyuv/libyuv_unittest.cc", 101 "libyuv/libyuv_unittest.cc",
103 ] 102 ]
104 103
105 # TODO(jschuh): Bug 1348: fix this warning. 104 # TODO(jschuh): Bug 1348: fix this warning.
106 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ] 105 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
107 106
108 if (is_clang) { 107 if (!build_with_chromium && is_clang) {
109 # Suppress warnings from the Chromium Clang plugin. 108 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
110 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
111 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ] 109 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
112 } 110 }
113 111
114 deps = [ 112 deps = [
115 ":common_video", 113 ":common_video",
116 "../system_wrappers:system_wrappers", 114 "../system_wrappers:system_wrappers",
117 "../test:test_support_main", 115 "../test:test_support_main",
118 "../test:video_test_common", 116 "../test:video_test_common",
119 "//testing/gmock", 117 "//testing/gmock",
120 "//testing/gtest", 118 "//testing/gtest",
121 ] 119 ]
122 120
123 data = common_video_resources 121 data = common_video_resources
124 if (is_android) { 122 if (is_android) {
125 deps += [ "//testing/android/native_test:native_test_support" ] 123 deps += [ "//testing/android/native_test:native_test_support" ]
126 shard_timeout = 900 124 shard_timeout = 900
127 } 125 }
128 126
129 if (is_ios) { 127 if (is_ios) {
130 deps += [ ":common_video_unittests_bundle_data" ] 128 deps += [ ":common_video_unittests_bundle_data" ]
131 } 129 }
132 } 130 }
133 } 131 }
OLDNEW
« no previous file with comments | « webrtc/common_audio/BUILD.gn ('k') | webrtc/examples/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698