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

Issue 1042803003: Add OWNERS file to Java org.chromium.chrome.browers.media.remote (Closed)

Created:
5 years, 8 months ago by aberent
Modified:
5 years, 8 months ago
CC:
chromium-reviews, posciak+watch_chromium.org, feature-media-reviews_chromium.org, wjia+watch_chromium.org, mcasas+watch_chromium.org, aruslan, David Trainor- moved to gerrit, Maria, Miguel Garcia, nyquist, Ted C, Yaron, newt (away), Bernhard Bauer, whywhat
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add OWNERS file to Java org.chromium.chrome.browers.media.remote This package deals specifically with playing video remotely, and is reasonably selfcontained. It should be owned by the people who know about it. BUG=none Committed: https://crrev.com/bcd6dbb3279e50391c5994bb282eb198133332c4 Cr-Commit-Position: refs/heads/master@{#323002}

Patch Set 1 #

Patch Set 2 : Remove dgn@ since he isn't yet a committer. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
A chrome/android/java/src/org/chromium/chrome/browser/media/remote/OWNERS View 1 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 11 (4 generated)
aberent
miguelg@chromium.org: Please approve as owner of /chrome/android/. I have cc'ed the other owners.
5 years, 8 months ago (2015-03-30 11:55:07 UTC) #2
whywhat
lgtm
5 years, 8 months ago (2015-03-30 11:55:59 UTC) #4
Ted C
lgtm
5 years, 8 months ago (2015-03-30 15:23:13 UTC) #6
Miguel Garcia
lgtm
5 years, 8 months ago (2015-03-31 08:37:16 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1042803003/20001
5 years, 8 months ago (2015-03-31 09:07:15 UTC) #9
commit-bot: I haz the power
Committed patchset #2 (id:20001)
5 years, 8 months ago (2015-03-31 09:29:06 UTC) #10
commit-bot: I haz the power
5 years, 8 months ago (2015-03-31 09:30:03 UTC) #11
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/bcd6dbb3279e50391c5994bb282eb198133332c4
Cr-Commit-Position: refs/heads/master@{#323002}

Powered by Google App Engine
This is Rietveld 408576698