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

Issue 10272027: Tighten up the DEPS file in aura to prevent inclusion of higher level bits of ui/ (Closed)

Created:
8 years, 7 months ago by Ben Goodger (Google)
Modified:
8 years, 7 months ago
Reviewers:
sky
CC:
chromium-reviews, sadrul, ben+watch_chromium.org
Visibility:
Public.

Description

Tighten up the DEPS file in aura to prevent inclusion of higher level bits of ui/ BUG=none TEST=none Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=134713

Patch Set 1 #

Total comments: 1

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M ui/aura/DEPS View 1 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Ben Goodger (Google)
8 years, 7 months ago (2012-05-01 15:53:27 UTC) #1
sky
http://codereview.chromium.org/10272027/diff/1/ui/aura/DEPS File ui/aura/DEPS (right): http://codereview.chromium.org/10272027/diff/1/ui/aura/DEPS#newcode7 ui/aura/DEPS:7: "+ui/base", nit: you've got ui/base twice.
8 years, 7 months ago (2012-05-01 16:26:48 UTC) #2
Ben Goodger (Google)
updated
8 years, 7 months ago (2012-05-01 16:36:32 UTC) #3
sky
8 years, 7 months ago (2012-05-01 16:37:29 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698