Adding deadbeef@ as owner of Objc-C PeerConnection-related headers.

Adding deadbeef@ as owner of Objc-C PeerConnection-related headers.

deadbeef@'s team in Kirkland works on the C++ PeerConnection
implementation, and often makes changes to the Java/Obj-C binding code
as new features are added. But the general Obj-C/Android owners are in
Stockholm. So adding deadbeef@ as an owner of this code should help
expedite code reviews for simple API changes.

BUG=None
NOTRY=True

Review-Url: https://codereview.webrtc.org/2998443002
Cr-Commit-Position: refs/heads/master@{#19259}
Committed: https://chromium.googlesource.com/external/webrtc/+/6327b6491e6c84fc451de445c17facbb50c58d9c

[Taylor Brandstetter, 2017-08-04 17:40:34]

deadbeef@webrtc.org changed reviewers:
+ magjed@webrtc.org, tkchin@webrtc.org

[Taylor Brandstetter, 2017-08-04 17:40:35]

PTAL. Is it ok to stick OWNERs files under "Framework", or do I need to use
objc/OWNERS?

[tkchin_webrtc, 2017-08-04 23:00:14]

On 2017/08/04 17:40:35, Taylor Brandstetter wrote:
> PTAL. Is it ok to stick OWNERs files under "Framework", or do I need to use
> objc/OWNERS?

Should be fine to put stuff under Framework? I'll defer to magjed@ though.

lgtm

[magjed_webrtc, 2017-08-05 14:38:36]

On 2017/08/04 23:00:14, tkchin_webrtc wrote:
> On 2017/08/04 17:40:35, Taylor Brandstetter wrote:
> > PTAL. Is it ok to stick OWNERs files under "Framework", or do I need to use
> > objc/OWNERS?
> 
> Should be fine to put stuff under Framework? I'll defer to magjed@ though.
> 
> lgtm

Yes, it's fine :)

lgtm

[Taylor Brandstetter, 2017-08-07 16:57:57]

Description was changed from

==========
Adding deadbeef@ as owner of Objc-C PeerConnection-related headers.

deadbeef@'s team in Kirkland works on the C++ PeerConnection
implementation, and often makes changes to the Java/Obj-C binding code
as new features are added. But the general Obj-C/Android owners are in
Stockholm. So adding deadbeef@ as an owner of this code should help
expedite code reviews for simple API changes.

BUG=None
==========

to

==========
Adding deadbeef@ as owner of Objc-C PeerConnection-related headers.

deadbeef@'s team in Kirkland works on the C++ PeerConnection
implementation, and often makes changes to the Java/Obj-C binding code
as new features are added. But the general Obj-C/Android owners are in
Stockholm. So adding deadbeef@ as an owner of this code should help
expedite code reviews for simple API changes.

BUG=None
NOTRY=True
==========

[Taylor Brandstetter, 2017-08-07 16:58:03]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2017-08-07 16:58:11]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-08-07 17:00:43]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1502125083045680, "parent_rev":
"62ad60561ba9eabc11e557725b105c8a22d01d66", "commit_rev":
"6327b6491e6c84fc451de445c17facbb50c58d9c"}

[commit-bot: I haz the power, 2017-08-07 17:00:48]

Description was changed from

==========
Adding deadbeef@ as owner of Objc-C PeerConnection-related headers.

deadbeef@'s team in Kirkland works on the C++ PeerConnection
implementation, and often makes changes to the Java/Obj-C binding code
as new features are added. But the general Obj-C/Android owners are in
Stockholm. So adding deadbeef@ as an owner of this code should help
expedite code reviews for simple API changes.

BUG=None
NOTRY=True
==========

to

==========
Adding deadbeef@ as owner of Objc-C PeerConnection-related headers.

deadbeef@'s team in Kirkland works on the C++ PeerConnection
implementation, and often makes changes to the Java/Obj-C binding code
as new features are added. But the general Obj-C/Android owners are in
Stockholm. So adding deadbeef@ as an owner of this code should help
expedite code reviews for simple API changes.

BUG=None
NOTRY=True

Review-Url: https://codereview.webrtc.org/2998443002
Cr-Commit-Position: refs/heads/master@{#19259}
Committed:
https://chromium.googlesource.com/external/webrtc/+/6327b6491e6c84fc451de445c...
==========

[commit-bot: I haz the power, 2017-08-07 17:00:49]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/6327b6491e6c84fc451de445c...