Chromium Code Reviews
Help | Chromium Project | Sign in
(124)

Issue 11367055: Add an OWNERS file for src/chrome/browser/webdata/ (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 5 months ago by Ilya Sherman
Modified:
1 year, 5 months ago
CC:
chromium-reviews_chromium.org
Visibility:
Public.

Description

Add an OWNERS file for src/chrome/browser/webdata/

BUG=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=166636

Patch Set 1 #

Patch Set 2 : Re-upload #

Patch Set 3 : Per-file owners #

Patch Set 4 : Add shess@ #

Patch Set 5 : shessqlite #

Patch Set 6 : Sans James #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -0 lines) Lint Patch
A chrome/browser/webdata/OWNERS View 1 2 3 4 5 1 chunk +14 lines, -0 lines 0 comments ? errors Download
Commit:

Messages

Total messages: 10
Ilya Sherman
David, James, and Peter: Do you think it's appropriate for you to be included in ...
1 year, 5 months ago #1
Peter Kasting
On 2012/11/02 03:49:00, Ilya Sherman wrote: > David, James, and Peter: Do you think it's ...
1 year, 5 months ago #2
Ilya Sherman
On 2012/11/02 04:09:51, Peter Kasting wrote: > On 2012/11/02 03:49:00, Ilya Sherman wrote: > > ...
1 year, 5 months ago #3
dhollowa
LGTM. Maybe add shess@ too?
1 year, 5 months ago #4
Ilya Sherman
+Scott for review. On 2012/11/02 15:55:01, dhollowa wrote: > LGTM. Maybe add shess@ too? Done.
1 year, 5 months ago #5
shess
On 2012/11/02 21:20:41, Ilya Sherman wrote: > +Scott for review. > > On 2012/11/02 15:55:01, ...
1 year, 5 months ago #6
Ilya Sherman
On 2012/11/02 21:34:55, shess wrote: > On 2012/11/02 21:20:41, Ilya Sherman wrote: > > +Scott ...
1 year, 5 months ago #7
Ilya Sherman
Removed James due to his silence on this CL. Everyone else seems happy with the ...
1 year, 5 months ago #8
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/isherman@chromium.org/11367055/7002
1 year, 5 months ago #9
I haz the power (commit-bot)
1 year, 5 months ago #10
Change committed as 166636
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6