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

Issue 7304015: Add ipc/OWNERS (Closed)

Created:
9 years, 5 months ago by Nico
Modified:
9 years, 5 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Add ipc/OWNERS I looked through some of the changes in the directory and who ok'd them (agl, dmac, jeremy), and threw in cpu because I think he wrote this originally and darin because he ok'd creation of this directory. BUG=88315 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=91453

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -0 lines) Patch
A ipc/OWNERS View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Nico
9 years, 5 months ago (2011-07-02 22:25:05 UTC) #1
jeremy
LGTM
9 years, 5 months ago (2011-07-03 09:00:02 UTC) #2
commit-bot: I haz the power
Change committed as 91453
9 years, 5 months ago (2011-07-03 20:20:25 UTC) #3
darin (slow to review)
On 2011/07/03 20:20:25, I haz the power (commit-bot) wrote: > Change committed as 91453 jam@ ...
9 years, 5 months ago (2011-07-06 19:33:43 UTC) #4
Nico
On Wed, Jul 6, 2011 at 12:33 PM, <darin@chromium.org> wrote: > On 2011/07/03 20:20:25, I ...
9 years, 5 months ago (2011-07-06 20:28:24 UTC) #5
jabdelmalek
9 years, 5 months ago (2011-07-06 20:44:48 UTC) #6
Also tsepez please, he recently made major changes to the macros.
On Jul 6, 2011 4:28 PM, "Nico Weber" <thakis@chromium.org> wrote:
> On Wed, Jul 6, 2011 at 12:33 PM, <darin@chromium.org> wrote:
>> On 2011/07/03 20:20:25, I haz the power (commit-bot) wrote:
>>>
>>> Change committed as 91453
>>
>> jam@ should be on the list.  He wrote SyncChannel as well as much of the
>> code in
>> this directory.
>
> http://codereview.chromium.org/7312012

Powered by Google App Engine
This is Rietveld 408576698