Add myself (vabr) to autofill/OWNERS for passwords

Add myself (vabr) to autofill/OWNERS for passwords

This should match vabr being already in components/password_manager/OWNERS.

BUG=None
NOTRY=True

Committed: https://crrev.com/26c1c52ff4376cb78f9ffb6098f237d80952aa59
Cr-Commit-Position: refs/heads/master@{#299868}

[vabr (Chromium), 2014-10-14 08:36:43]

vabr@chromium.org changed reviewers:
+ gcasto@chromium.org, isherman@chromium.org

[vabr (Chromium), 2014-10-14 08:36:43]

Garrett,
Thanks for your suggestion in https://codereview.chromium.org/649583003/#msg6
(but let me know if I misunderstood "this directory").

Ilya,
Please let me know if you are fine with this. I won't add myself if you are not.

Cheers,
Vaclav

[vabr (Chromium), 2014-10-14 08:37:02]

vabr@chromium.org changed required reviewers:
+ gcasto@chromium.org, isherman@chromium.org

[Garrett Casto, 2014-10-14 18:07:38]

lgtm

[Ilya Sherman, 2014-10-14 20:37:17]

FWIW, [ http://crbug.com/423445 ] suggests that this might not currently work
(unless Garrett really isn't a committer -- if so, we should remedy that for
sure!).

https://codereview.chromium.org/659463002/diff/1/components/autofill/OWNERS
File components/autofill/OWNERS (right):

https://codereview.chromium.org/659463002/diff/1/components/autofill/OWNERS#n...
components/autofill/OWNERS:6: vabr@chromium.org
Is it viable to add you both as per-file OWNERS instead, now that the
component's structure is fairly stable?

[Garrett Casto, 2014-10-14 20:45:53]

https://codereview.chromium.org/659463002/diff/1/components/autofill/OWNERS
File components/autofill/OWNERS (right):

https://codereview.chromium.org/659463002/diff/1/components/autofill/OWNERS#n...
components/autofill/OWNERS:6: vabr@chromium.org
On 2014/10/14 20:37:17, Ilya Sherman wrote:
> Is it viable to add you both as per-file OWNERS instead, now that the
> component's structure is fairly stable

I guess all the password related files have password in their name, so maybe 

per-file *password*=...

Might be confusing though. Long term we should finish untangling these two
components, but it's not likely to happen this quarter.

[vabr (Chromium), 2014-10-15 10:34:01]

Ilya, Garrett,

PTAL at the per-file version.

Cheers,
Vaclav

https://codereview.chromium.org/659463002/diff/1/components/autofill/OWNERS
File components/autofill/OWNERS (right):

https://codereview.chromium.org/659463002/diff/1/components/autofill/OWNERS#n...
components/autofill/OWNERS:6: vabr@chromium.org
On 2014/10/14 20:45:53, Garrett Casto wrote:
> On 2014/10/14 20:37:17, Ilya Sherman wrote:
> > Is it viable to add you both as per-file OWNERS instead, now that the
> > component's structure is fairly stable
> 
> I guess all the password related files have password in their name, so maybe 
> 
> per-file *password*=...
> 
> Might be confusing though. Long term we should finish untangling these two
> components, but it's not likely to happen this quarter.

I added per-file for *password* and also
(content/common/)autofill_param_traits_macros.h, which is relevant for
PasswordForm (and is not covered by the security team set of OWNERS).

[Ilya Sherman, 2014-10-15 21:40:44]

LGTM.  Thanks!

[vabr (Chromium), 2014-10-16 08:22:05]

The CQ bit was checked by vabr@chromium.org

[commit-bot: I haz the power, 2014-10-16 08:22:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/659463002/20001

[commit-bot: I haz the power, 2014-10-16 08:22:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-16 08:23:32]

Patchset 2 (id:??) landed as
https://crrev.com/26c1c52ff4376cb78f9ffb6098f237d80952aa59
Cr-Commit-Position: refs/heads/master@{#299868}