Issue 12217145: Add an OWNERS file to top-level extensions.

Issue 12217145: Add an OWNERS file to top-level extensions. (Closed)

Created:
7 years, 10 months ago by Yoyo Zhou

Modified:
7 years, 10 months ago

Reviewers:
Aaron Boodman, darin (slow to review), Ben Goodger (Google)

CC:
chromium-reviews, Aaron Boodman, chromium-apps-reviews_chromium.org, not at google - send to devlin

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

Visibility:
Public.

More Reviews

Description

Add an OWNERS file to top-level extensions. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182472

Patch Set 1 #

Total comments: 2

Created: 7 years, 10 months ago

Download [raw] [tar.bz2]

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

Messages

Total messages: 11 (0 generated)

Expand Messages | Collapse Messages

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yoz@chromium.org/12217145/1

7 years, 10 months ago (2013-02-13 23:43:33 UTC) #5

commit-bot: I haz the power

Retried try job too often on linux_aura for step(s) aura_unittests, browser_tests, compositor_unittests, content_browsertests, content_unittests, interactive_ui_tests, ...

7 years, 10 months ago (2013-02-14 00:10:53 UTC) #6

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yoz@chromium.org/12217145/1

7 years, 10 months ago (2013-02-14 02:39:40 UTC) #7

Aaron Boodman

lgtm https://codereview.chromium.org/12217145/diff/1/extensions/OWNERS File extensions/OWNERS (right): https://codereview.chromium.org/12217145/diff/1/extensions/OWNERS#newcode2 extensions/OWNERS:2: aa@chromium.org :)

7 years, 10 months ago (2013-02-14 02:43:04 UTC) #8

Yoyo Zhou

https://codereview.chromium.org/12217145/diff/1/extensions/OWNERS File extensions/OWNERS (right): https://codereview.chromium.org/12217145/diff/1/extensions/OWNERS#newcode2 extensions/OWNERS:2: aa@chromium.org On 2013/02/14 02:43:04, Aaron Boodman wrote: > :) ...

7 years, 10 months ago (2013-02-14 02:46:24 UTC) #9

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yoz@chromium.org/12217145/1

7 years, 10 months ago (2013-02-14 16:17:51 UTC) #10

Message was sent while issue was closed.

Change committed as 182472

Expand Messages | Collapse Messages