third_party/zlib/google/OWNERS - Issue 1564773002: Move components/compression to third_party/zlib/google

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: third_party/zlib/google/OWNERS

Issue 1564773002: Move components/compression to third_party/zlib/google (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Created 4 years, 11 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:

View unified diff | Download patch

OLD	NEW
1 hshi@chromium.org	1 hshi@chromium.org

2 satorux@chromium.org	2 satorux@chromium.org

	3

	4 # compression_utils.h, compression_utils.cc
	Nico 2016/01/06 14:36:50 that's a very long list of owners for such a small that's a very long list of owners for such a small file! (not that there's anything wrong with that -- but e.g. jar@ isn't doing much in chromium-land anymore, so maybe take a second look at this list) Alexei Svitkine (slow) 2016/01/06 16:21:23 I think this was lifted from components/metrics ow Show quoted text On 2016/01/06 14:36:50, Nico wrote: > that's a very long list of owners for such a small file! (not that there's > anything wrong with that -- but e.g. jar@ isn't doing much in chromium-land > anymore, so maybe take a second look at this list) I think this was lifted from components/metrics owners, since that's where the code originally came from. Realistically, it's probably fine to trim this list to e.g. only myself and isherman@. hajimehoshi 2016/01/07 07:40:10 Done (leaving asvitkine@ and isherman@). Show quoted text On 2016/01/06 16:21:23, Alexei Svitkine wrote: > On 2016/01/06 14:36:50, Nico wrote: > > that's a very long list of owners for such a small file! (not that there's > > anything wrong with that -- but e.g. jar@ isn't doing much in chromium-land > > anymore, so maybe take a second look at this list) > > I think this was lifted from components/metrics owners, since that's where the > code originally came from. Realistically, it's probably fine to trim this list > to e.g. only myself and isherman@. Done (leaving asvitkine@ and isherman@).
	5 asvitkine@chromium.org

	6 holte@chromium.org

	7 isherman@chromium.org

	8 jar@chromium.org

	9 mpearson@chromium.org

	10 rkaplow@chromium.org

OLD	NEW