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

Issue 8510077: set noparent on resources/ntp4/ OWNERS file (Closed)

Created:
9 years, 1 month ago by Evan Stade
Modified:
9 years, 1 month ago
CC:
chromium-reviews, estade+watch_chromium.org, arv (Not doing code reviews)
Visibility:
Public.

Description

set noparent on resources/ntp4/OWNERS and resources/OWNERS also add self to resources/ OWNERS file BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111218

Patch Set 1 #

Total comments: 1

Patch Set 2 : another noparent #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M chrome/browser/resources/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download
M chrome/browser/resources/ntp4/OWNERS View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Evan Stade
9 years, 1 month ago (2011-11-15 04:07:16 UTC) #1
Evan Stade
ping
9 years, 1 month ago (2011-11-21 22:29:03 UTC) #2
arv (Not doing code reviews)
http://codereview.chromium.org/8510077/diff/1/chrome/browser/resources/OWNERS File chrome/browser/resources/OWNERS (right): http://codereview.chromium.org/8510077/diff/1/chrome/browser/resources/OWNERS#newcode3 chrome/browser/resources/OWNERS:3: estade@chromium.org This one needs a noparent too
9 years, 1 month ago (2011-11-21 23:04:43 UTC) #3
Evan Stade
done
9 years, 1 month ago (2011-11-22 02:12:24 UTC) #4
arv (Not doing code reviews)
9 years, 1 month ago (2011-11-22 17:55:32 UTC) #5
lgtm

Powered by Google App Engine
This is Rietveld 408576698