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

Issue 1314123004: Add magjed@ as owner for talk/app/webrtc/androidtests/ and talk/app/webrtc/java/jni/ (Closed)

Created:
5 years, 3 months ago by magjed_webrtc
Modified:
5 years, 3 months ago
Reviewers:
tommi
CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com, qiang.lu, niklas.enbom, yujie_mao (webrtc), Andrew MacDonald
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Add magjed@ as owner for talk/app/webrtc/androidtests/ and talk/app/webrtc/java/jni/ magjed@ has done a lot of work in these folders. R=tommi@webrtc.org Committed: https://chromium.googlesource.com/external/webrtc/+/7764973e1d5f8afaddab981cefb76b25477d8d94

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+-2 lines, --2 lines) Patch
A + talk/app/webrtc/androidtests/OWNERS View 0 chunks +-1 lines, --1 lines 0 comments Download
A + talk/app/webrtc/java/jni/OWNERS View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 9 (4 generated)
magjed_webrtc
tommi - Please take a look. I would like to be an owner for Android ...
5 years, 3 months ago (2015-09-08 12:09:24 UTC) #3
tommi
lgtm
5 years, 3 months ago (2015-09-08 12:35:10 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1314123004/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1314123004/20001
5 years, 3 months ago (2015-09-08 12:35:19 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: android on tryserver.webrtc (JOB_TIMED_OUT, no build URL) android_arm64_rel on ...
5 years, 3 months ago (2015-09-08 14:35:37 UTC) #8
magjed_webrtc
5 years, 3 months ago (2015-09-08 15:13:49 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:20001) manually as
7764973e1d5f8afaddab981cefb76b25477d8d94 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698