Add tkchin to video_coding OWNERS.

Add tkchin to video_coding OWNERS.

BUG=
R=stefan@webrtc.org

Committed: https://chromium.googlesource.com/external/webrtc/+/d830aeafe943b498fd0db51f949dc183be3b4e4f

[tkchin_webrtc, 2015-06-29 22:48:03]

tkchin@webrtc.org changed reviewers:
+ stefan@webrtc.org

[tkchin_webrtc, 2015-06-29 22:48:04]

I should probably be OWNER for at least the iOS h264 things.

[stefan-webrtc, 2015-06-30 08:44:34]

On 2015/06/29 22:48:04, tkchin_webrtc wrote:
> I should probably be OWNER for at least the iOS h264 things.

Do you feel you have good knowledge of how the rest of the video_coding module
works (e.g., jitter buffer etc?). Maybe it would make sense to limit the
ownership to video_coding/codecs for now?

[tkchin_webrtc, 2015-06-30 18:21:19]

I want OWNERS for the gyp file, which lives in video_coding/. But no I don't
want owners for the rest of the video_coding module.
I'll remove it, but you are signing up for every review that touches the build
config now :)

[stefan-webrtc, 2015-06-30 18:24:22]

On 2015/06/30 18:21:19, tkchin_webrtc wrote:
> I want OWNERS for the gyp file, which lives in video_coding/. But no I don't
> want owners for the rest of the video_coding module.
> I'll remove it, but you are signing up for every review that touches the build
> config now :)

Hehe... Feel free to add yourself to the video_coding/ as well, and add a
comment about the gyp file maybe?

Otherwise I'm ok with reviewing that piece as well, but it might be good to
reduce the RTTs. :)

[stefan-webrtc, 2015-06-30 18:26:30]

On 2015/06/30 18:24:22, stefan-webrtc (holmer) wrote:
> On 2015/06/30 18:21:19, tkchin_webrtc wrote:
> > I want OWNERS for the gyp file, which lives in video_coding/. But no I don't
> > want owners for the rest of the video_coding module.
> > I'll remove it, but you are signing up for every review that touches the
build
> > config now :)
> 
> Hehe... Feel free to add yourself to the video_coding/ as well, and add a
> comment about the gyp file maybe?
> 
> Otherwise I'm ok with reviewing that piece as well, but it might be good to
> reduce the RTTs. :)

lgtm

[tkchin_webrtc, 2015-06-30 18:33:27]

On 2015/06/30 18:26:30, stefan-webrtc (holmer) wrote:
> On 2015/06/30 18:24:22, stefan-webrtc (holmer) wrote:
> > On 2015/06/30 18:21:19, tkchin_webrtc wrote:
> > > I want OWNERS for the gyp file, which lives in video_coding/. But no I
don't
> > > want owners for the rest of the video_coding module.
> > > I'll remove it, but you are signing up for every review that touches the
> build
> > > config now :)
> > 
> > Hehe... Feel free to add yourself to the video_coding/ as well, and add a
> > comment about the gyp file maybe?
> > 
> > Otherwise I'm ok with reviewing that piece as well, but it might be good to
> > reduce the RTTs. :)
> 
> lgtm

Actually modules/OWNERS has these so I think we're good:
per-file *.gyp=*
per-file *.gypi=*

[tkchin_webrtc, 2015-06-30 18:33:56]

The CQ bit was checked by tkchin@webrtc.org

[commit-bot: I haz the power, 2015-06-30 18:34:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1217853006/20001

[commit-bot: I haz the power, 2015-06-30 19:30:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-30 19:30:34]

Exceeded global retry quota

[tkchin_webrtc, 2015-06-30 20:10:10]

The CQ bit was checked by tkchin@webrtc.org

[commit-bot: I haz the power, 2015-06-30 20:10:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1217853006/20001

[commit-bot: I haz the power, 2015-06-30 20:17:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-30 20:17:05]

Exceeded global retry quota

[tkchin_webrtc, 2015-07-01 20:46:31]

The CQ bit was checked by tkchin@webrtc.org

[commit-bot: I haz the power, 2015-07-01 20:46:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1217853006/20001

[commit-bot: I haz the power, 2015-07-01 22:47:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-07-01 22:47:05]

Try jobs failed on following builders:
  android on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[tkchin_webrtc, 2015-07-01 23:02:03]

Committed patchset #2 (id:20001) manually as
d830aeafe943b498fd0db51f949dc183be3b4e4f (presubmit successful).