From 580dec48471a72b10fe0060683e383c07670238f Mon Sep 17 00:00:00 2001 From: "jrg@chromium.org" Date: Tue, 19 Jan 2010 00:37:39 +0000 Subject: Add browser_tests to coverage (note recent mac fixes). Add platform-specific coverage capability. Add courgette to windows coverage. BUG=none TEST=make sure coverage bots work (continue to have ~36% coverage). Review URL: http://codereview.chromium.org/545107 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36502 0039d316-1c4b-4281-b951-d872f2087c98 --- build/win/chrome_win.croc | 2 +- chrome/chrome_tests.gypi | 37 ++++++++++++++++++++++++++++++++----- 2 files changed, 33 insertions(+), 6 deletions(-) diff --git a/build/win/chrome_win.croc b/build/win/chrome_win.croc index c562f2a..1f12e59 100644 --- a/build/win/chrome_win.croc +++ b/build/win/chrome_win.croc @@ -7,7 +7,7 @@ # Files/paths to include. Specify these before the excludes, since rules # are in order. { - 'regexp' : '^CHROMIUM/(base|media|net|printing|chrome|v8|webkit/glue)/', + 'regexp' : '^CHROMIUM/(base|courgette|media|net|printing|chrome|v8|webkit/glue)/', 'include' : 1, }, # Don't include subversion or mercurial SCM dirs diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi index b98b7e0..b56840a 100755 --- a/chrome/chrome_tests.gypi +++ b/chrome/chrome_tests.gypi @@ -1786,7 +1786,6 @@ }, ]}, # 'targets' ], # OS=="win" - # TODO(jrg): add in Windows code coverage targets. ['coverage!=0', { 'targets': [ { @@ -1795,21 +1794,49 @@ # In gyp, booleans are 0/1 not True/False. 'suppress_wildcard': 1, 'type': 'none', - # If you add new tests here you may need to update the croc configs. - # E.g. build/{linux|mac}/chrome_linux.croc + # Cross platform test bundles. If you add new tests you may + # need to update the croc configs. For example, see the + # first regexp in build/(linux|mac|win)/chrome_*.croc. 'dependencies': [ 'automated_ui_tests', '../app/app.gyp:app_unittests', '../base/base.gyp:base_unittests', + 'browser_tests', '../ipc/ipc.gyp:ipc_tests', '../media/media.gyp:media_unittests', '../net/net.gyp:net_unittests', '../printing/printing.gyp:printing_unittests', - # Mac coverage bot having trouble launching a profiled - # Chromium from within ui_tests; disabling for now. + # ui_tests seem unhappy on both Mac and Win when run under + # coverage (all tests fail, often with a + # "server_->WaitForInitialLoads()"). TODO(jrg): + # investigate why. # 'ui_tests', 'unit_tests', ], + # Platform specific unit test bundles. Unless staging + # a checkin, please add a comment describing why your test is + # in here and is not cross-platform. + 'conditions': [ + ['OS=="win"', { + 'dependencies': [ + # Courgette has not been ported from Windows. + # Note build/win/chrome_win.croc uniquely has the + # courgette source directory in an include path. + '../courgette/courgette.gyp:courgette_unittests', + ], + }], + ['OS=="linux"', { + 'dependencies': [ + # Placeholder; empty for now. + ], + }], + ['OS=="mac"', { + 'dependencies': [ + # Placeholder; empty for now. + ], + }], + ], + 'actions': [ { # 'message' for Linux/scons in particular. Scons -- cgit v1.1