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

Issue 6271011: Add OWNERS for tabbed options and DOMUI in general. (Closed)

Created:
9 years, 11 months ago by Evan Stade
Modified:
9 years, 6 months ago
CC:
chromium-reviews, arv (Not doing code reviews)
Visibility:
Public.

Description

Add OWNERS for tabbed options and DOMUI in general. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=71898

Patch Set 1 #

Total comments: 2

Patch Set 2 : no commoas #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -0 lines) Patch
A chrome/browser/dom_ui/OWNERS View 1 chunk +1 line, -0 lines 0 comments Download
A chrome/browser/dom_ui/options/OWNERS View 1 1 chunk +4 lines, -0 lines 0 comments Download
A chrome/browser/resources/OWNERS View 1 chunk +1 line, -0 lines 0 comments Download
A chrome/browser/resources/options/OWNERS View 1 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Evan Stade
9 years, 11 months ago (2011-01-19 23:06:11 UTC) #1
James Hawkins
LGTM
9 years, 11 months ago (2011-01-19 23:07:02 UTC) #2
stuartmorgan
LGTM http://codereview.chromium.org/6271011/diff/1/chrome/browser/dom_ui/options/OWNERS File chrome/browser/dom_ui/options/OWNERS (right): http://codereview.chromium.org/6271011/diff/1/chrome/browser/dom_ui/options/OWNERS#newcode1 chrome/browser/dom_ui/options/OWNERS:1: csilv@chromium.org, Same. http://codereview.chromium.org/6271011/diff/1/chrome/browser/resources/options/OWNERS File chrome/browser/resources/options/OWNERS (right): http://codereview.chromium.org/6271011/diff/1/chrome/browser/resources/options/OWNERS#newcode1 chrome/browser/resources/options/OWNERS:1: ...
9 years, 11 months ago (2011-01-19 23:18:09 UTC) #3
arv (Not doing code reviews)
LGTM erik On Wed, Jan 19, 2011 at 15:18, <stuartmorgan@chromium.org> wrote: > LGTM > > ...
9 years, 11 months ago (2011-01-19 23:37:22 UTC) #4
csilv
9 years, 11 months ago (2011-01-20 01:49:48 UTC) #5
LGTM

Powered by Google App Engine
This is Rietveld 408576698