Update Autofill component ownership to avoid bugs in OWNERS file parsing

components/OWNERS

Index: components/OWNERS
diff --git a/components/OWNERS b/components/OWNERS
index 220ef1011aafb61f99fe70b94704cea3c2827f55..ed713276d9d95e7810c2af26ee9ece64b17a620a 100644
--- a/components/OWNERS
+++ b/components/OWNERS
@@ -3,8 +3,11 @@ caitkp@chromium.org
 erikwright@chromium.org
 jochen@chromium.org
 
-per-file autofill*=estade@chromium.org
-per-file autofill*=isherman@chromium.org
+# Can not match autofill* due to crbug.com/397984
+per-file autofill.gypi=estade@chromium.org
+per-file autofill.gypi=isherman@chromium.org
+per-file autofill_strings.grdp=estade@chromium.org
+per-file autofill_strings.grpd=isherman@chromium.org

[Ilya Sherman, 2014/10/17 22:02:57]

nit: "grpd" -> "grdp"

[Ilya Sherman, 2014/10/17 22:02:57]

Also, I wonder if we can move this file to live within the autofill directory,
rather than at the top-level //components directory.  (Food for thought, not
anything you need to address as part of this CL.)

[Garrett Casto, 2014/10/17 23:17:00]

Done.

[Garrett Casto, 2014/10/17 23:17:00]

Acknowledged.

[blundell, 2014/10/20 11:31:07]

+1, I think it's reasonable to have all the foo-related files live under
//components/foo, as I mentioned in the chromium-dev@ thread. We should be
consistent though, so if someone makes this change, it would be great if they
did it for the handful of other grdp files in //components too. Also, this would
affect the translation process, so you would have to coordinate with someone who
knows about how to make sure that gets adjusted accordingly (e.g., one of the
TPMs).

[tfarina, 2014/10/21 15:17:03]

There are some complications in doing that. But I don't recall what they are.
Tony or Joi might remember. I will look into my mailbox to see if I found the
thread about it. I think it is something related to relative paths and how grit
deals with that. 

 
 per-file bookmarks.gypi=sky@chromium.org