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

Issue 15809005: Add chrome/renderer/net/OWNERS (Closed)

Created:
7 years, 6 months ago by Nico
Modified:
7 years, 3 months ago
CC:
chromium-reviews, cbentzel+watch_chromium.org, Deprecated (see juliatuttle)
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : mmenke #

Patch Set 3 : ttuttle #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
A chrome/renderer/net/OWNERS View 1 2 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 14 (0 generated)
Nico
Hi jar, do you think you're a good owner for this directory? You've written and ...
7 years, 6 months ago (2013-05-30 00:41:23 UTC) #1
Nico
jar: ping
7 years, 5 months ago (2013-06-28 20:06:14 UTC) #2
cbentzel
May want mmenke in there as well due to network error work.
7 years, 5 months ago (2013-06-28 20:56:45 UTC) #3
jar (doing other things)
Sorry I missed this. I don't think having a solo owner is good. I think ...
7 years, 5 months ago (2013-06-28 22:40:55 UTC) #4
Nico
Thanks for the comments! Added mmenke too, looks like he worked on prerendering a while ...
7 years, 5 months ago (2013-06-28 23:45:30 UTC) #5
mmenke
On 2013/06/28 23:45:30, Nico wrote: > Thanks for the comments! > > Added mmenke too, ...
7 years, 5 months ago (2013-06-29 00:50:08 UTC) #6
Nico
jar: looks good?
7 years, 4 months ago (2013-07-29 20:19:22 UTC) #7
jar (doing other things)
Nico: Can you indicate why you want owners for this submodule? If Nico has a ...
7 years, 4 months ago (2013-07-30 20:53:58 UTC) #8
Nico
On Tue, Jul 30, 2013 at 1:53 PM, <jar@chromium.org> wrote: > Nico: Can you indicate ...
7 years, 4 months ago (2013-07-30 21:00:33 UTC) #9
cbentzel
On 2013/07/30 21:00:33, Nico wrote: > On Tue, Jul 30, 2013 at 1:53 PM, <mailto:jar@chromium.org> ...
7 years, 4 months ago (2013-07-31 19:37:38 UTC) #10
jar (doing other things)
Per Chris's comment... please add ttuttle, and then LGTM.
7 years, 4 months ago (2013-08-01 01:57:29 UTC) #11
mmenke
On 2013/08/01 01:57:29, jar wrote: > Per Chris's comment... please add ttuttle, and then LGTM. ...
7 years, 3 months ago (2013-09-16 18:56:22 UTC) #12
Nico
Sorry, forgot about this CL. Landing now. ttuttle: FYI :-)
7 years, 3 months ago (2013-09-16 19:00:01 UTC) #13
Nico
7 years, 3 months ago (2013-09-16 19:01:14 UTC) #14
Message was sent while issue was closed.
Committed patchset #3 manually as r223375 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698