Make hbos@ OWNER of WebRTC related files in platform.
Make hbos@... 1. per-file web_rtc_* OWNER of third_party/blink/renderer/platform/exported hbos@ is already per-file web_rtc_* OWNER of third_party/blink/public/platform. This is where the header files of interfaces are placed, but often times the .cc files which are placed in "exported" need to be modified too for things like out-of-line constructor and destructor. 2. per-file testing_platform_support_with_web_rtc.* OWNER of third_party/blink/renderer/platform/testing/. Any substantial changes in WebRTC requires updating some "platform" interface and then these files have to be updated to update the mocks. We frequently have to bring in non-webrtc reviewers for the sake of stamping some minor webrtc interface being updated. The webrtc reviewers already do a good job of reviewing these things, better to make a webrtc owner OWN these few files too. NOTRY=True Bug: 777617 Change-Id: Ie0e5dbd43d93df89524c50bed3683f681d47778d Reviewed-on: https://chromium-review.googlesource.com/1140063Reviewed-by:Philip Jägenstedt <foolip@chromium.org> Commit-Queue: Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#575615}
Showing
Please register or sign in to comment