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

Issue 8526022: Add support for stripping Chrome tests. (Closed)

Created:
9 years, 1 month ago by davidjames
Modified:
9 years, 1 month ago
Reviewers:
rkc, DaveMoore
CC:
chromium-reviews
Visibility:
Public.

Description

Add support for stripping Chrome tests. The chromeos-chrome ebuild strips Chrome tests after the build, but it can run significantly faster if we strip them during the build. This changelist reduces the time it takes to build Chrome by allowing developers to remove unnecessary debug symbols. Performance comparison on z600 LOCAL_SOURCE build: Before: 24m31s (257m04s user, 23m19s sys) After: 20m55s (236m12s user, 22m10s sys) Companion CL for updating Chrome OS to use strip_tests by default is here: https://gerrit.chromium.org/gerrit/11517 BUG=chromium-os:22844 TEST=Build with and without debug symbols. Confirm speed difference. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110948

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -0 lines) Patch
M build/common.gypi View 1 2 chunks +5 lines, -0 lines 0 comments Download
M chrome/chrome_tests.gypi View 1 chunk +8 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
davidjames
9 years, 1 month ago (2011-11-11 01:03:29 UTC) #1
DaveMoore
9 years, 1 month ago (2011-11-11 18:47:45 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698