Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(38)

Issue 1604233002: Move talk/media/webrtc/OWNERS to talk/media. (Closed)

Created:
4 years, 11 months ago by pbos-webrtc
Modified:
4 years, 11 months ago
Reviewers:
pthatcher1
CC:
webrtc-reviews_webrtc.org
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Move talk/media/webrtc/OWNERS to talk/media. Permits changing talk/media/base without root ownership approval. BUG= R=pthatcher@webrtc.org Committed: https://crrev.com/b11e97a5527b2b1e15cd599d81580d5df39dbfe5 Cr-Commit-Position: refs/heads/master@{#11392}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+-1 lines, -3 lines) Patch
A + talk/media/OWNERS View 0 chunks +-1 lines, --1 lines 0 comments Download
D talk/media/webrtc/OWNERS View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
pbos-webrtc
PTAL, I don't think the split inside talk/media makes sense. This is just a git ...
4 years, 11 months ago (2016-01-20 11:46:16 UTC) #1
pthatcher1
lgtm On 2016/01/20 11:46:16, pbos-webrtc wrote: > PTAL, I don't think the split inside talk/media ...
4 years, 11 months ago (2016-01-27 01:48:39 UTC) #2
pbos-webrtc
Committed patchset #1 (id:1) manually as b11e97a5527b2b1e15cd599d81580d5df39dbfe5 (presubmit successful).
4 years, 11 months ago (2016-01-27 11:39:40 UTC) #4
commit-bot: I haz the power
4 years, 11 months ago (2016-01-27 11:39:41 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/b11e97a5527b2b1e15cd599d81580d5df39dbfe5
Cr-Commit-Position: refs/heads/master@{#11392}

Powered by Google App Engine
This is Rietveld 408576698