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

Issue 124343003: Only aura_bench needs cc/ dependency. (Closed)

Created:
6 years, 11 months ago by tfarina
Modified:
6 years, 11 months ago
Reviewers:
sadrul
CC:
chromium-reviews, sadrul, ben+aura_chromium.org, kalyank, tony
Visibility:
Public.

Description

Only aura_bench needs cc/ dependency. The rest of aura targets do not reference cc/, so they don't need to explicit list cc/ as a dependency. BUG=None TEST=aura_bench, aura_demo, aura_unittests, all three still links R=sadrul@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=243719

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M ui/aura/aura.gyp View 2 chunks +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
tfarina
6 years, 11 months ago (2014-01-05 14:08:56 UTC) #1
sadrul
lgtm
6 years, 11 months ago (2014-01-05 22:03:03 UTC) #2
tfarina
6 years, 11 months ago (2014-01-09 01:04:09 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r243719 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698