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

Side by Side Diff: ui/base/ui_base_tests.gyp

Issue 588963002: Start the process of renaming ui_unittests to ui_base_unittests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix it? Created 6 years, 3 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 | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium 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 { 5 {
6 'variables': { 6 'variables': {
7 'chromium_code': 1, 7 'chromium_code': 1,
8 }, 8 },
9 'targets': [ 9 'targets': [
10 { 10 {
11 # TODO(tfarina): Rename this target to ui_base_unittests.
12 # In order to do this we will need to keep this target, add a dummy
13 # ui_base_unittests target that just points to ui_unittests, change
14 # buildbot code to reference the new 'ui_base_unittests' target.
15 # After buildbot is updated, we can make the changes in Chromium and
16 # remove 'ui_unittests' target. crbug.com/331829
17 # GN version: //ui/base:unittests 11 # GN version: //ui/base:unittests
18 'target_name': 'ui_unittests', 12 'target_name': 'ui_base_unittests',
19 'type': '<(gtest_target_type)', 13 'type': '<(gtest_target_type)',
20 'dependencies': [ 14 'dependencies': [
21 '../../base/base.gyp:base', 15 '../../base/base.gyp:base',
22 '../../base/base.gyp:test_support_base', 16 '../../base/base.gyp:test_support_base',
23 '../../net/net.gyp:net', 17 '../../net/net.gyp:net',
24 '../../skia/skia.gyp:skia', 18 '../../skia/skia.gyp:skia',
25 '../../testing/gmock.gyp:gmock', 19 '../../testing/gmock.gyp:gmock',
26 '../../testing/gtest.gyp:gtest', 20 '../../testing/gtest.gyp:gtest',
27 '../../third_party/icu/icu.gyp:icui18n', 21 '../../third_party/icu/icu.gyp:icui18n',
28 '../../third_party/icu/icu.gyp:icuuc', 22 '../../third_party/icu/icu.gyp:icuuc',
(...skipping 197 matching lines...) Expand 10 before | Expand all | Expand 10 after
226 'target_conditions': [ 220 'target_conditions': [
227 ['OS == "ios"', { 221 ['OS == "ios"', {
228 'sources/': [ 222 'sources/': [
229 # Pull in specific Mac files for iOS (which have been filtered out 223 # Pull in specific Mac files for iOS (which have been filtered out
230 # by file name rules). 224 # by file name rules).
231 ['include', '^l10n/l10n_util_mac_unittest\\.mm$'], 225 ['include', '^l10n/l10n_util_mac_unittest\\.mm$'],
232 ], 226 ],
233 }], 227 }],
234 ], 228 ],
235 }, 229 },
230 {
231 # TODO(tfarina): Remove this target after all traces of it are updated to
232 # point to ui_base_unittests. That means updating buildbot code and some
233 # references in chromium too. crbug.com/331829
234 'target_name': 'ui_unittests',
235 'type': 'none',
236 'dependencies': [
237 'ui_base_unittests',
238 ],
239 'conditions': [
240 ['OS != "ios"',
241 {
242 'actions': [
243 {
244 'message': 'TEMPORARY: Copying ui_base_unittests to ui_unittests ',
245 'action_name': 'copy_ui_base_unittests',
246 'variables': {
247 'source_file': '<(PRODUCT_DIR)/ui_base_unittests<(EXECUTABLE_S UFFIX)',
cjhopman 2014/09/22 00:07:40 It the only difference is in the variables, why no
tfarina 2014/09/22 00:53:15 Done.
248 'dest_file': '<(PRODUCT_DIR)/ui_unittests<(EXECUTABLE_SUFFIX)' ,
249 },
250 'inputs': [
251 '../../build/cp.py',
252 '<(source_file)',
253 ],
254 'outputs': [
255 '<(dest_file)',
256 ],
257 'action': [
258 'python', '../../build/cp.py', '<(source_file)', '<(dest_file) ',
259 ],
260 },
261 ],
262 },
263 ],
264 ['OS == "ios"',
265 {
266 'actions': [
267 {
268 'message': 'TEMPORARY: Copying ui_base_unittests to ui_unittests ',
269 'action_name': 'copy_ui_base_unittests',
270 'variables': {
271 'source_file': '<(PRODUCT_DIR)/ui_base_unittests.app/ui_base_u nittests',
272 'dest_file': '<(PRODUCT_DIR)/ui_unittests.app/ui_unittests',
273 },
274 'inputs': [
275 '../../build/cp.py',
276 '<(source_file)',
277 ],
278 'outputs': [
279 '<(dest_file)',
280 ],
281 'action': [
282 'python', '../../build/cp.py', '<(source_file)', '<(dest_file) ',
283 ],
284 },
285 ],
286 },
287 ],
288 ],
289 },
236 ], 290 ],
237 'conditions': [ 291 'conditions': [
238 # Mac target to build a test Framework bundle to mock out resource loading. 292 # Mac target to build a test Framework bundle to mock out resource loading.
239 ['OS == "mac"', { 293 ['OS == "mac"', {
240 'targets': [ 294 'targets': [
241 { 295 {
242 'target_name': 'ui_base_tests_bundle', 296 'target_name': 'ui_base_tests_bundle',
243 'type': 'shared_library', 297 'type': 'shared_library',
244 'dependencies': [ 298 'dependencies': [
245 '../resources/ui_resources.gyp:ui_test_pak', 299 '../resources/ui_resources.gyp:ui_test_pak',
246 ], 300 ],
247 'includes': [ 'ui_base_tests_bundle.gypi' ], 301 'includes': [ 'ui_base_tests_bundle.gypi' ],
248 }, 302 },
249 ], 303 ],
250 }], 304 }],
251 ['OS == "android"', { 305 ['OS == "android"', {
252 'targets': [ 306 'targets': [
253 { 307 {
254 'target_name': 'ui_unittests_apk', 308 'target_name': 'ui_unittests_apk',
255 'type': 'none', 309 'type': 'none',
256 'dependencies': [ 310 'dependencies': [
257 'ui_unittests', 311 'ui_unittests',
312 'ui_base_unittests',
258 ], 313 ],
259 'variables': { 314 'variables': {
260 'test_suite_name': 'ui_unittests', 315 'test_suite_name': 'ui_base_unittests',
261 }, 316 },
262 'includes': [ '../../build/apk_test.gypi' ], 317 'includes': [ '../../build/apk_test.gypi' ],
263 }, 318 },
264 ], 319 ],
265 }], 320 }],
266 ], 321 ],
267 } 322 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698