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

Issue 264016: Adding a Chrome Frame target to all.gyp to allow a continuous builder to buil...

Created:
11 years, 2 months ago by robertshield
Modified:
11 years, 2 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Adding a Chrome Frame target to all.gyp to allow a continuous builder to build only the subset of projects required by Chrome Frame. BUG=NONE TEST=NONE

Patch Set 1 #

Total comments: 4

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -0 lines) Patch
M build/all.gyp View 1 1 chunk +18 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
robertshield
11 years, 2 months ago (2009-10-08 20:19:34 UTC) #1
robertshield
11 years, 2 months ago (2009-10-08 20:20:08 UTC) #2
slightlyoff
LGTM. Minor nit. http://codereview.chromium.org/264016/diff/1/2 File build/all.gyp (right): http://codereview.chromium.org/264016/diff/1/2#newcode193 Line 193: '../chrome/installer/installer.gyp:installer_util_unittests', it would be nice ...
11 years, 2 months ago (2009-10-08 20:26:17 UTC) #3
Nicolas Sylvain
LGTM http://codereview.chromium.org/264016/diff/1/2 File build/all.gyp (right): http://codereview.chromium.org/264016/diff/1/2#newcode199 Line 199: '../chrome_frame/chrome_frame.gyp:npchrome_tab', I would think that chrome_frame_perftests or ...
11 years, 2 months ago (2009-10-08 20:26:50 UTC) #4
robertshield
11 years, 2 months ago (2009-10-08 20:34:58 UTC) #5
Thanks!

http://codereview.chromium.org/264016/diff/1/2
File build/all.gyp (right):

http://codereview.chromium.org/264016/diff/1/2#newcode193
Line 193: '../chrome/installer/installer.gyp:installer_util_unittests',
On 2009/10/08 20:26:18, slightlyoff wrote:
> it would be nice if we explicitly called out the chrome.gyp:chrome task.

Done.

http://codereview.chromium.org/264016/diff/1/2#newcode199
Line 199: '../chrome_frame/chrome_frame.gyp:npchrome_tab',
On 2009/10/08 20:26:54, Nicolas Sylvain wrote:
> I would think that chrome_frame_perftests or any other chrome frame tests
would
> depend on that. Do we really need to make it explicit?

They should, but currently don't, so for the time being we need to make it
explicit.

Powered by Google App Engine
This is Rietveld 408576698