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

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

BUG=none


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

[Ilya Sherman, 2012-11-02 03:49:00]

David, James, and Peter: Do you think it's appropriate for you to be included in
the OWNERS file for this directory?  Are there any others who should be
included?  Will wait until I hear from each of you prior to committing.

[Peter Kasting, 2012-11-02 04:09:51]

On 2012/11/02 03:49:00, Ilya Sherman wrote:
> David, James, and Peter: Do you think it's appropriate for you to be included
in
> the OWNERS file for this directory?  Are there any others who should be
> included?  Will wait until I hear from each of you prior to committing.

It might be nice to make use of dpranke's new per-file OWNERS syntax, since I
really only know the keyword table and some of the web database/web data service
wrapping code, but none of the rest.

[Ilya Sherman, 2012-11-02 07:00:35]

On 2012/11/02 04:09:51, Peter Kasting wrote:
> On 2012/11/02 03:49:00, Ilya Sherman wrote:
> > David, James, and Peter: Do you think it's appropriate for you to be
included
> in
> > the OWNERS file for this directory?  Are there any others who should be
> > included?  Will wait until I hear from each of you prior to committing.
> 
> It might be nice to make use of dpranke's new per-file OWNERS syntax, since I
> really only know the keyword table and some of the web database/web data
service
> wrapping code, but none of the rest.

Done.  If others would also like to only own some of the files, let me know
which ones.

[dhollowa, 2012-11-02 15:55:01]

LGTM.  Maybe add shess@ too?

[Ilya Sherman, 2012-11-02 21:20:41]

+Scott for review.

On 2012/11/02 15:55:01, dhollowa wrote:
> LGTM.  Maybe add shess@ too?

Done.

[Scott Hess - ex-Googler, 2012-11-02 21:34:55]

On 2012/11/02 21:20:41, Ilya Sherman wrote:
> +Scott for review.
> 
> On 2012/11/02 15:55:01, dhollowa wrote:
> > LGTM.  Maybe add shess@ too?
> 
> Done.

LGTM, but it's probably appropriate to include some "for sqlite stuff"
annotation with me, because I would be lost for reviews of actual functionality.

[Ilya Sherman, 2012-11-02 21:48:00]

On 2012/11/02 21:34:55, shess wrote:
> On 2012/11/02 21:20:41, Ilya Sherman wrote:
> > +Scott for review.
> > 
> > On 2012/11/02 15:55:01, dhollowa wrote:
> > > LGTM.  Maybe add shess@ too?
> > 
> > Done.
> 
> LGTM, but it's probably appropriate to include some "for sqlite stuff"
> annotation with me, because I would be lost for reviews of actual
functionality.

Done.

[Ilya Sherman, 2012-11-08 05:39:42]

Removed James due to his silence on this CL.  Everyone else seems happy with the
file as it is now, so submitting.

[commit-bot: I haz the power, 2012-11-08 05:39:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/isherman@chromium.org/11367055/7002

[commit-bot: I haz the power, 2012-11-08 08:13:07]

Change committed as 166636