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

Issue 523027: Removing the last remaining vsprops files.... (Closed)

Created:
10 years, 11 months ago by bradn
Modified:
9 years, 7 months ago
Reviewers:
Mark Mentovai
CC:
chromium-reviews_googlegroups.com, kuchhal, brettw+cc_chromium.org
Base URL:
svn://chrome-svn.corp.google.com/chrome/trunk/src/
Visibility:
Public.

Description

Removing the last remaining vsprops files (that are hooked in). Also pulling in a newer nacl version. Switching to use AdditionalOptions in list form. BUG=None TEST=None TBR=mmentovai Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=35449

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Patch Set 5 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+115 lines, -309 lines) Patch
M DEPS View 1 2 3 4 1 chunk +2 lines, -2 lines 0 comments Download
M base/allocator/allocator.gyp View 1 2 3 4 1 chunk +2 lines, -2 lines 0 comments Download
M build/common.gypi View 1 2 3 4 5 chunks +18 lines, -9 lines 0 comments Download
D build/common.vsprops View 3 4 1 chunk +0 lines, -7 lines 0 comments Download
D build/debug.vsprops View 1 chunk +0 lines, -23 lines 0 comments Download
D build/external_code.vsprops View 1 2 3 4 1 chunk +0 lines, -14 lines 0 comments Download
D build/internal/debug_info.vsprops View 1 2 3 4 1 chunk +0 lines, -15 lines 0 comments Download
D build/internal/debug_infofastbuild.vsprops View 1 2 3 4 1 chunk +0 lines, -15 lines 0 comments Download
D build/internal/essential.vsprops View 1 2 3 4 1 chunk +0 lines, -76 lines 0 comments Download
A build/internal/release_defaults.gypi View 2 3 4 1 chunk +15 lines, -0 lines 0 comments Download
D build/internal/release_defaults.vsprops View 1 2 3 4 1 chunk +0 lines, -25 lines 0 comments Download
A build/internal/release_impl.gypi View 2 3 4 1 chunk +3 lines, -0 lines 0 comments Download
D build/internal/release_impl.vsprops View 1 2 3 4 1 chunk +0 lines, -12 lines 0 comments Download
A build/internal/release_impl_official.gypi View 1 chunk +21 lines, -0 lines 0 comments Download
D build/internal/release_impl_official.vsprops View 1 2 3 4 1 chunk +0 lines, -27 lines 0 comments Download
A build/release.gypi View 1 chunk +19 lines, -0 lines 0 comments Download
D build/release.vsprops View 1 2 3 4 1 chunk +0 lines, -7 lines 0 comments Download
M chrome/installer/mini_installer.gyp View 1 2 3 4 2 chunks +33 lines, -16 lines 0 comments Download
D chrome/installer/mini_installer/mini_installer.vsprops View 3 4 1 chunk +0 lines, -31 lines 0 comments Download
D chrome/installer/mini_installer/mini_installer_debug.vsprops View 3 4 1 chunk +0 lines, -13 lines 0 comments Download
D chrome/installer/mini_installer/mini_installer_release.vsprops View 3 4 1 chunk +0 lines, -13 lines 0 comments Download
M third_party/tcmalloc/tcmalloc.gyp View 1 2 3 4 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
bradn
10 years, 11 months ago (2009-12-30 22:05:09 UTC) #1
Mark Mentovai
10 years, 11 months ago (2010-01-06 21:20:06 UTC) #2
After-holiday rubber-stamp LGTM

Powered by Google App Engine
This is Rietveld 408576698