tkchin and deadbeef are not working on webrtc on a daily basis at the
moment, so non-urgent approvals should not go to them.
Not mentioning this has led to misunderstandings.
Bug: chromium:1371843
Change-Id: I91e99249d32e52d6083de9c2b1bfebfc4693acac
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/278201
Reviewed-by: Taylor Brandstetter <deadbeef@google.com>
Reviewed-by: Tomas Gunnarsson <tommi@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#38314}
kron is an owner of UMA metrics for WebRTC in chromium,
see tools/metrics/histograms/histograms_xml/web_rtc/OWNERS
Bug: webrtc:12096
Change-Id: I9804d747fc4e52d2ed2a9d96cc4ed315639210da
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/221961
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Johannes Kron <kron@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#34268}
As requested on bugs.webrtc.org/12096#c2, this CL adds a Chromium
metric OWNERS in order to always have their review when WebRTC's UMA
metrics are updated.
Bug: webrtc:12096
Change-Id: Icd9ab7dda5f7a4ba6ac078f667c1fd39f3314123
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/191702
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#32728}
I'll hold on to the root OWNER for a bit longer for convenience.
Bug: None
Change-Id: I13303ba726fed612adc74008eeaaeadf9595e084
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/170047
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#30727}
Specifically api, pc and p2p.
Bug: None
Change-Id: I2ba19aaac5ca11a5282593f0db06bba326fe6891
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/169041
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Reviewed-by: Tommi <tommi@webrtc.org>
Commit-Queue: Tommi <tommi@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#30609}
No-Try: True
Bug: None
Change-Id: I632d5384321c88202a23cc3fa6938afac0f796ba
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/168460
Commit-Queue: Steve Anton <steveanton@webrtc.org>
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#30494}
No-Try: True
Bug: None
Change-Id: I411cdb48c9da23e72c575c59ede8f5d140a437e5
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/128223
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Commit-Queue: Steve Anton <steveanton@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27165}
This CL updates the checkdeps configuration for the api/ folder in
order to explicitly avoid to #include non API headers from API headers.
In order to force a careful review of potential exceptions to this
rule, the CL also adds mbonadei@ and kwiberg@ as OWNERS of api/DEPS.
Bug: webrtc:9887
Change-Id: I0ada6f1020186b2782c7d060af36079c452ba1aa
Reviewed-on: https://webrtc-review.googlesource.com/c/106800
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25338}
With deadbeef removed from these OWNERS files, Steve is the only OWNER
on our team. I'm adding myself, because I have worked in these
directories and it makes sense to be able to distribute the code
reviews.
NOTRY=True
Bug: None
Change-Id: I48e88a07ee42254d937391f500f273656853d98b
Reviewed-on: https://webrtc-review.googlesource.com/86980
Commit-Queue: Steve Anton <steveanton@webrtc.org>
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23826}
hbos knows and makes changes to the webrtc-pc spec[1] and works on
making Chromium's RTCPeerConnection spec-compliant. This includes
knowing and interacting with WebRTC-layer PeerConnection/Interface and
sometimes making changes to it.
hbos would like to share the peerconnection* ownership responsibilty as
it is relevant and owning it will speed up some of the process.
[1] https://w3c.github.io/webrtc-pc/
Bug: None
NOTRY: True
Change-Id: I8f419b7fc6c7fcf19951aa3f304769c915300d1b
Reviewed-on: https://webrtc-review.googlesource.com/21327
Reviewed-by: Tommi <tommi@webrtc.org>
Reviewed-by: Peter Thatcher <pthatcher@webrtc.org>
Commit-Queue: Henrik Boström <hbos@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20649}
In order to eliminate the WebRTC Subtree mirror in Chromium,
WebRTC is moving the content of the src/webrtc directory up
to the src/ directory.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
TBR=tommi@webrtc.org
Bug: chromium:611808
Change-Id: Iac59c5b51b950f174119565bac87955a7994bc38
Reviewed-on: https://webrtc-review.googlesource.com/1560
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Henrik Kjellander <kjellander@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#19845}
