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

Side by Side Diff: sync/sync.gyp

Issue 9691067: Add a new 'run_all_unittests' target in base (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync to head Created 8 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 | Annotate | Revision Log
« no previous file with comments | « dbus/dbus.gyp ('k') | third_party/cacheinvalidation/cacheinvalidation.gyp » ('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) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 # The core sync library. 10 # The core sync library.
(...skipping 306 matching lines...) Expand 10 before | Expand all | Expand 10 after
317 }, 317 },
318 318
319 # The unit test executable for sync tests. Currently this isn't 319 # The unit test executable for sync tests. Currently this isn't
320 # automatically run, as there is already a sync_unit_tests 320 # automatically run, as there is already a sync_unit_tests
321 # executable in chrome.gyp; this is just to make sure that all the 321 # executable in chrome.gyp; this is just to make sure that all the
322 # link-time dependencies for the files in the targets above 322 # link-time dependencies for the files in the targets above
323 # resolve. 323 # resolve.
324 # 324 #
325 # TODO(akalin): Rename this to sync_unit_tests once we've moved 325 # TODO(akalin): Rename this to sync_unit_tests once we've moved
326 # everything from chrome.gyp. 326 # everything from chrome.gyp.
327 #
328 # TODO(akalin): Make base.gyp have a test_main target that
329 # includes run_all_unittests.cc and the possible tcmalloc
330 # dependency and use that everywhere.
331 { 327 {
332 'target_name': 'sync_unit_tests_canary', 328 'target_name': 'sync_unit_tests_canary',
333 'type': 'executable', 329 'type': 'executable',
334 'sources': [
335 '../base/test/run_all_unittests.cc',
336 ],
337 'dependencies': [ 330 'dependencies': [
331 '../base/base.gyp:run_all_unittests',
338 'sync_tests', 332 'sync_tests',
339 ], 333 ],
340 334
341 # TODO(akalin): This is needed because histogram.cc uses 335 # TODO(akalin): This is needed because histogram.cc uses
342 # leak_annotations.h, which pulls this in. Make 'base' 336 # leak_annotations.h, which pulls this in. Make 'base'
343 # propagate this dependency. 337 # propagate this dependency.
344 'conditions': [ 338 'conditions': [
345 ['OS=="linux" and linux_use_tcmalloc==1', { 339 ['OS=="linux" and linux_use_tcmalloc==1', {
346 'dependencies': [ 340 'dependencies': [
347 '../base/allocator/allocator.gyp:allocator', 341 '../base/allocator/allocator.gyp:allocator',
348 ], 342 ],
349 }], 343 }],
350 ], 344 ],
351 }, 345 },
352 ], 346 ],
353 } 347 }
OLDNEW
« no previous file with comments | « dbus/dbus.gyp ('k') | third_party/cacheinvalidation/cacheinvalidation.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698