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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 18178014: Remove the OWNERS that no longer work on the project. (Closed)

Created:
7 years, 5 months ago by aurimas (slooooooooow)
Modified:
7 years, 5 months ago
CC:
chromium-reviews, dbeam+watch-ntp_chromium.org, tim+watch_chromium.org, craigdh+watch_chromium.org, rsimha+watch_chromium.org, bulach+watch_chromium.org, haitaol+watch_chromium.org, frankf+watch_chromium.org, jam, joi+watch-content_chromium.org, feature-media-reviews_chromium.org, darin-cc_chromium.org, arv+watch_chromium.org, ilevy-cc_chromium.org, yfriedman+watch_chromium.org, estade+watch_chromium.org, klundberg+watch_chromium.org, erikwright+watch_chromium.org, pedrosimonetti+watch_chromium.org, Jay Civelli, Steve Block
Visibility:
Public.

Description

Remove the OWNERS that no longer work on the project. Remove jcivelli@, and nileshagrawal@ who no longer work on Chrome for Android. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=209531

Patch Set 1 #

Patch Set 2 : Added a few missing files #

Total comments: 6

Patch Set 3 : yfriedman's comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -30 lines) Patch
M base/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M base/test/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M build/android/buildbot/OWNERS View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/app/android/OWNERS View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/android/OWNERS View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/browser/resources/ntp_android/OWNERS View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/browser/ui/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/ui/webui/ntp/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M content/browser/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M content/common/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M content/public/android/OWNERS View 1 chunk +0 lines, -2 lines 0 comments Download
M content/public/browser/android/OWNERS View 1 chunk +0 lines, -2 lines 0 comments Download
M content/public/test/android/OWNERS View 1 2 1 chunk +0 lines, -2 lines 0 comments Download
M content/renderer/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M content/renderer/media/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M content/shell/android/OWNERS View 1 1 chunk +0 lines, -1 line 0 comments Download
M content/test/OWNERS View 1 1 chunk +0 lines, -1 line 0 comments Download
M media/base/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M sync/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M sync/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M sync/test/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M testing/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M ui/android/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 14 (0 generated)
aurimas (slooooooooow)
PTAL
7 years, 5 months ago (2013-06-28 23:40:56 UTC) #1
nilesh
LGTM
7 years, 5 months ago (2013-06-28 23:51:12 UTC) #2
Ted C
lgtm
7 years, 5 months ago (2013-06-28 23:52:31 UTC) #3
Yaron
https://codereview.chromium.org/18178014/diff/3001/content/browser/renderer_host/java/OWNERS File content/browser/renderer_host/java/OWNERS (left): https://codereview.chromium.org/18178014/diff/3001/content/browser/renderer_host/java/OWNERS#oldcode2 content/browser/renderer_host/java/OWNERS:2: steveblock@chromium.org Steve still works on chromium and probably knows ...
7 years, 5 months ago (2013-06-28 23:58:25 UTC) #4
bulach
lgtm with yaron's comments. sad, sad day. :(
7 years, 5 months ago (2013-06-29 00:01:44 UTC) #5
aurimas (slooooooooow)
https://codereview.chromium.org/18178014/diff/3001/content/browser/renderer_host/java/OWNERS File content/browser/renderer_host/java/OWNERS (left): https://codereview.chromium.org/18178014/diff/3001/content/browser/renderer_host/java/OWNERS#oldcode2 content/browser/renderer_host/java/OWNERS:2: steveblock@chromium.org On 2013/06/28 23:58:25, Yaron wrote: > Steve still ...
7 years, 5 months ago (2013-06-29 00:09:16 UTC) #6
Yaron
lgtm
7 years, 5 months ago (2013-06-29 00:11:24 UTC) #7
aurimas (slooooooooow)
+thakis for chrome/OWNERS +sky for content/test/OWNERS +akalin for sync/OWNERS
7 years, 5 months ago (2013-06-29 00:13:56 UTC) #8
akalin
On 2013/06/29 00:13:56, aurimas wrote: > +thakis for chrome/OWNERS > +sky for content/test/OWNERS > +akalin ...
7 years, 5 months ago (2013-06-29 00:18:05 UTC) #9
Nico
s/the OWNERS/some OWNERS/ in the CL description. Did Jay and Nilesh ask for this? Several ...
7 years, 5 months ago (2013-06-29 00:21:03 UTC) #10
Jay Civelli
lgtm
7 years, 5 months ago (2013-06-29 00:21:30 UTC) #11
Steve Block
lgtm Thanks for leaving me in for the Java Bridge!
7 years, 5 months ago (2013-06-30 06:59:53 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/aurimas@chromium.org/18178014/15001
7 years, 5 months ago (2013-07-01 20:08:37 UTC) #13
commit-bot: I haz the power
7 years, 5 months ago (2013-07-01 23:02:22 UTC) #14
Message was sent while issue was closed. 
Change committed as 209531

Powered by Google App Engine
This is Rietveld 408576698