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

Issue 48021: Add targets for page_cycler_tests, perf_tests and startup_tests. Listed... (Closed)

Created:
11 years, 9 months ago by sgk
Modified:
9 years, 5 months ago
Reviewers:
Mark Mentovai, bradn
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Add targets for page_cycler_tests, perf_tests and startup_tests. Listed minimal dependencies to build on Linux, Windows will need more. Make a 'test_support_base' library containing the .cc files used in common by perf tests in both net and chrome. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=11803

Patch Set 1 #

Total comments: 1

Patch Set 2 : '' #

Total comments: 1

Patch Set 3 : '' #

Patch Set 4 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+102 lines, -4 lines) Patch
M base/base.gyp View 2 3 1 chunk +17 lines, -0 lines 0 comments Download
M chrome/chrome.gyp View 1 2 chunks +84 lines, -2 lines 0 comments Download
M net/net.gyp View 1 chunk +1 line, -2 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
sgk
11 years, 9 months ago (2009-03-16 19:52:38 UTC) #1
Mark Mentovai
http://codereview.chromium.org/48021/diff/1/2 File chrome/chrome.gyp (right): http://codereview.chromium.org/48021/diff/1/2#newcode1810 Line 1810: '../base/perftimer.cc', lgtm as-is if it passes the try ...
11 years, 9 months ago (2009-03-16 20:08:23 UTC) #2
sgk
Update from previous iteration: put these targets in a !=mac condition.
11 years, 9 months ago (2009-03-16 20:50:35 UTC) #3
sgk
Update update: != mac for test_support_base, too.
11 years, 9 months ago (2009-03-16 21:31:59 UTC) #4
Mark Mentovai
☠ needs work ☠ http://codereview.chromium.org/48021/diff/4/1006 File net/net.gyp (right): http://codereview.chromium.org/48021/diff/4/1006#newcode463 Line 463: '../base/base.gyp:test_support_base', This is going ...
11 years, 9 months ago (2009-03-16 21:35:08 UTC) #5
sgk
http://build.chromium.org/buildbot/try-server/builders/mac/builds/5489 Whew!
11 years, 9 months ago (2009-03-16 22:45:42 UTC) #6
Mark Mentovai
11 years, 9 months ago (2009-03-16 23:14:05 UTC) #7
I think we have a winner.

✓ lgtm

Powered by Google App Engine
This is Rietveld 408576698