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

Issue 7613001: Don't make chrome_browser depend on all of views gyp, since that builds test binaries as well. (Closed)

Created:
9 years, 4 months ago by jam
Modified:
9 years, 4 months ago
Reviewers:
Evan Martin
CC:
chromium-reviews
Visibility:
Public.

Description

Don't make chrome_browser depend on all of views gyp, since that builds test binaries as well. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96258

Patch Set 1 #

Patch Set 2 : '' #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -2 lines) Patch
M chrome/chrome_browser.gypi View 1 1 chunk +4 lines, -2 lines 1 comment Download

Messages

Total messages: 4 (0 generated)
jam
9 years, 4 months ago (2011-08-10 18:59:57 UTC) #1
jam
Rerouting to Evan since Scott is gone.
9 years, 4 months ago (2011-08-10 19:40:11 UTC) #2
Evan Martin
LGTM for now, would be nice if it was cleaner (what is this desktop lib ...
9 years, 4 months ago (2011-08-10 19:52:26 UTC) #3
jam
9 years, 4 months ago (2011-08-10 19:55:59 UTC) #4
On 2011/08/10 19:52:26, Evan Martin wrote:
> LGTM for now, would be nice if it was cleaner (what is this desktop lib
anyway)

No idea. I'm not familiar with views at all, I just did this so we don't build
binaries unnecessarily.

Powered by Google App Engine
This is Rietveld 408576698