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

Side by Side Diff: testing/gtest.gyp

Issue 1774123005: Support clang-cl build on Windows. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 4 years, 9 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
OLDNEW
1 # Copyright 2014 PDFium authors. All rights reserved. 1 # Copyright 2014 PDFium authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 # 4 #
5 # Original code from V8, original license was: 5 # Original code from V8, original license was:
6 # Copyright 2014 the V8 project authors. All rights reserved. 6 # Copyright 2014 the V8 project authors. All rights reserved.
7 # Use of this source code is governed by a BSD-style license that can be 7 # Use of this source code is governed by a BSD-style license that can be
8 # found in the LICENSE file. 8 # found in the LICENSE file.
9 # 9 #
10 # This file is used only by the standalone PDFium build. Under a chromium 10 # This file is used only by the standalone PDFium build. Under a chromium
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
135 ['OS=="win"', { 135 ['OS=="win"', {
136 'run_as': { 136 'run_as': {
137 'action????': ['$(TargetPath)', '--gtest_print_time'], 137 'action????': ['$(TargetPath)', '--gtest_print_time'],
138 }, 138 },
139 }], 139 }],
140 ], 140 ],
141 }], 141 }],
142 ], 142 ],
143 'msvs_disabled_warnings': [4800], 143 'msvs_disabled_warnings': [4800],
144 }, 144 },
145 'variables': {
146 'clang_warning_flags': [
147 '-Wno-reorder',
148 '-Wno-unused'
Nico 2016/03/09 02:30:19 We only need the first of these two in chromium's
Oliver Chang 2016/03/09 17:00:17 Added comments. We need both here for some reason
149 ],
150 },
145 }, 151 },
146 { 152 {
147 'target_name': 'gtest_main', 153 'target_name': 'gtest_main',
148 'type': 'static_library', 154 'type': 'static_library',
149 'dependencies': [ 155 'dependencies': [
150 'gtest', 156 'gtest',
151 ], 157 ],
152 'sources': [ 158 'sources': [
153 'gtest/src/gtest_main.cc', 159 'gtest/src/gtest_main.cc',
154 ], 160 ],
155 }, 161 },
156 { 162 {
157 'target_name': 'gtest_prod', 163 'target_name': 'gtest_prod',
158 'toolsets': ['host', 'target'], 164 'toolsets': ['host', 'target'],
159 'type': 'none', 165 'type': 'none',
160 'sources': [ 166 'sources': [
161 'gtest/include/gtest/gtest_prod.h', 167 'gtest/include/gtest/gtest_prod.h',
162 ], 168 ],
163 }, 169 },
164 ], 170 ],
165 } 171 }
OLDNEW
« no previous file with comments | « build/standalone.gypi ('k') | third_party/third_party.gyp » ('j') | third_party/third_party.gyp » ('J')

Powered by Google App Engine
This is Rietveld 408576698