media/gpu/ipc/common/OWNERS - Issue 2345123002: Attach color space information to hardware decoded NV12 video frames.

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: media/gpu/ipc/common/OWNERS

Issue 2345123002: Attach color space information to hardware decoded NV12 video frames. (Closed)

Patch Set: comments addressed + compile fixes Created 4 years, 3 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 # Changes to IPCs require a security review to avoid introducing	1 # Changes to IPCs require a security review to avoid introducing

2 # new sandbox escapes.	2 # new sandbox escapes.

3 set noparent	3 set noparent

4 dcheng@chromium.org	4 file://ipc/SECURITY_OWNERS
	sandersd (OOO until July 31) 2016/09/16 21:54:13 I don't see any mention of these changes in the CL I don't see any mention of these changes in the CL description, and they seem unrelated? hubbe 2016/09/21 22:04:26 presubmit complained about this, so I fixed it the Show quoted text On 2016/09/16 21:54:13, sandersd wrote: > I don't see any mention of these changes in the CL description, and they seem > unrelated? presubmit complained about this, so I fixed it the right way. sandersd (OOO until July 31) 2016/09/23 00:33:50 Is your version more correct than the current cont Show quoted text On 2016/09/21 22:04:26, hubbe wrote: > On 2016/09/16 21:54:13, sandersd wrote: > > I don't see any mention of these changes in the CL description, and they seem > > unrelated? > > presubmit complained about this, so I fixed it the right way. Is your version more correct than the current content? dcheng@ suggested the current content, and I don't think there should be any presubmit errors.
5 inferno@chromium.org	5

6 jln@chromium.org	6 # The following lines are redundant, they're just to silence the presubmit

7 jschuh@chromium.org	7 per-file _messages.h=set noparent

8 kenrb@chromium.org	8 per-file _messages.h=file://ipc/SECURITY_OWNERS

9 mkwst@chromium.org	9 per-file *.mojom=set noparent

10 nasko@chromium.org	10 per-file *.mojom=file://ipc/SECURITY_OWNERS

11 palmer@chromium.org	11 per-file _param_traits.*=set noparent

12 tsepez@chromium.org	12 per-file _param_traits.*=file://ipc/SECURITY_OWNERS

13 wfh@chromium.org	13 per-file _struct_traits.*=set noparent

	14 per-file _struct_traits.*=file://ipc/SECURITY_OWNERS

	15

OLD	NEW

« media/gpu/dxva_video_decode_accelerator_win.cc ('K') | « media/gpu/ipc/common/BUILD.gn ('k') | media/gpu/ipc/common/media_messages.h » ('j') | media/gpu/ipc/common/media_messages.h » ('J')