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

Issue 197853006: Remove akalin and szym from OWNERS files. (Closed)

Created:
6 years, 9 months ago by mmenke
Modified:
6 years, 9 months ago
CC:
chromium-reviews, tim+watch_chromium.org, cbentzel+watch_chromium.org, haitaol+watch_chromium.org, arv+watch_chromium.org, maniscalco+watch_chromium.org, akalin, szym
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Add back szym to src/net/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -15 lines) Patch
M chrome/browser/invalidation/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/network_time/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/resources/sync_internals/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/sync/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/ui/sync/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M jingle/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M net/OWNERS View 1 2 chunks +5 lines, -2 lines 0 comments Download
M net/test/OWNERS View 1 chunk +0 lines, -3 lines 0 comments Download
M sync/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M sync/tools/testserver/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M third_party/cacheinvalidation/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M third_party/v4l2capture/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 10 (0 generated)
mmenke
Paul: net/ and v4l2capture/ tim/zea: Everything else (Each of you owns one or two of ...
6 years, 9 months ago (2014-03-14 17:59:02 UTC) #1
Nicolas Zea
LGTM
6 years, 9 months ago (2014-03-14 18:02:05 UTC) #2
pauljensen
The net/ and v4l2capture/ lgtm. Have you run this by cbentzel@? I'm wondering if either ...
6 years, 9 months ago (2014-03-14 18:03:51 UTC) #3
mmenke
On 2014/03/14 18:03:51, pauljensen wrote: > The net/ and v4l2capture/ lgtm. Have you run this ...
6 years, 9 months ago (2014-03-14 18:08:50 UTC) #4
cbentzel
On 2014/03/14 18:08:50, mmenke wrote: > On 2014/03/14 18:03:51, pauljensen wrote: > > The net/ ...
6 years, 9 months ago (2014-03-14 18:59:24 UTC) #5
cbentzel
On 2014/03/14 18:59:24, cbentzel wrote: > On 2014/03/14 18:08:50, mmenke wrote: > > On 2014/03/14 ...
6 years, 9 months ago (2014-03-15 18:51:48 UTC) #6
tim (not reviewing)
On 2014/03/15 18:51:48, cbentzel wrote: > On 2014/03/14 18:59:24, cbentzel wrote: > > On 2014/03/14 ...
6 years, 9 months ago (2014-03-17 18:58:38 UTC) #7
szym
I'd be available to review future CL of the work already under way: websockets, cronet, ...
6 years, 9 months ago (2014-03-17 19:08:01 UTC) #8
mmenke
szym: I'll leave you in the net/ OWNERS file for now, then (At the bottom, ...
6 years, 9 months ago (2014-03-17 19:16:27 UTC) #9
mmenke
6 years, 9 months ago (2014-03-18 15:30:08 UTC) #10
Message was sent while issue was closed.
Committed patchset #2 manually as r257639 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698