Issue 128263007: Add myself to chrome/browser/apps owners.

Issue 128263007: Add myself to chrome/browser/apps owners. (Closed)

Created:
6 years, 11 months ago by not at google - send to devlin

Modified:
6 years, 11 months ago

Reviewers:
benwells

CC:
chromium-reviews

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

Add myself to chrome/browser/apps owners. R=benwells@chromium.org NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=244006

Patch Set 2 : more #

Created: 6 years, 11 months ago

Download [raw] [tar.bz2]

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

Messages

Total messages: 6 (0 generated)

Expand Messages | Collapse Messages

benwells

Also: src/apps chrome/browser/ui/apps https://codereview.chromium.org/128263007/diff/1/chrome/browser/apps/OWNERS File chrome/browser/apps/OWNERS (right): https://codereview.chromium.org/128263007/diff/1/chrome/browser/apps/OWNERS#newcode7 chrome/browser/apps/OWNERS:7: kalman@chromium.org Nit: keep alphabeticized

6 years, 11 months ago (2014-01-09 02:36:48 UTC) #2

not at google - send to devlin

added to ui owners, already in apps owners. https://codereview.chromium.org/128263007/diff/1/chrome/browser/apps/OWNERS File chrome/browser/apps/OWNERS (right): https://codereview.chromium.org/128263007/diff/1/chrome/browser/apps/OWNERS#newcode7 chrome/browser/apps/OWNERS:7: kalman@chromium.org ...

6 years, 11 months ago (2014-01-09 16:05:59 UTC) #3

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kalman@chromium.org/128263007/60001

6 years, 11 months ago (2014-01-09 22:41:04 UTC) #5

Message was sent while issue was closed.

Change committed as 244006

Expand Messages | Collapse Messages