content/browser/compositor/OWNERS - Issue 635573003: Add more owners to content.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: content/browser/compositor/OWNERS

Issue 635573003: Add more owners to content. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Created 6 years, 2 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

OLD	NEW
1 piman@chromium.org

2 danakj@chromium.org	1 danakj@chromium.org

3 sievers@chromium.org
danakj 2014/10/06 19:10:30 It's nice to have owners like this in subdirectori It's nice to have owners like this in subdirectories to help people find who should review things here even if they are redundant. Prefer to keep these. jam 2014/10/07 00:26:02 Yeah, there are pros and cons to this. The con is Show quoted text On 2014/10/06 19:10:30, danakj wrote: > It's nice to have owners like this in subdirectories to help people find who > should review things here even if they are redundant. Prefer to keep these. Yeah, there are pros and cons to this. The con is that top level owners are familiar with many different subdirs, and we would have to keep putting their names in all of them and also new ones as they get created or as new owners files are added. so that's why we've stuck by top-level owners aren't listed in other owners files inside content
4 jbauman@chromium.org	2 jbauman@chromium.org

5 ccameron@chromium.org	3 ccameron@chromium.org

OLD	NEW

« no previous file with comments | « content/browser/android/OWNERS ('k') | content/browser/frame_host/OWNERS » ('j') | no next file with comments »