Add owners for c/b/ui/views/accessibility

Add owners for c/b/ui/views/accessibility

BUG=none

Committed: https://crrev.com/4a414030466b90f7f11795229199c4e975fb4fcd
Cr-Commit-Position: refs/heads/master@{#296843}

[dmazzoni, 2014-09-25 18:48:23]

dmazzoni@chromium.org changed reviewers:
+ sky@chromium.org

[tfarina, 2014-09-25 19:02:18]

tfarina@chromium.org changed reviewers:
+ tfarina@chromium.org

[tfarina, 2014-09-25 19:02:19]

I looked at ui/accessibility/OWNERS
(https://chromium.googlesource.com/chromium/src/+/master/ui/accessibility/OWNERS).
There we have aboxhall instead of plundblad.

Do you want to replace one by another? Either way, this lgtm (but I'm not an
OWNER of c/b/ui/ though).

https://codereview.chromium.org/608573003/diff/1/chrome/browser/ui/views/acce...
File chrome/browser/ui/views/accessibility/OWNERS (left):

https://codereview.chromium.org/608573003/diff/1/chrome/browser/ui/views/acce...
chrome/browser/ui/views/accessibility/OWNERS:1: dmazzoni@chromium.org
could you tweak the similarity passed to git cl upload, maybe something very
low, say 10?

So it doesn't look like a copy of
chrome/browser/extensions/api/braille_display_private/OWNERS?

Not a big deal though.

[dmazzoni, 2014-09-25 19:14:47]

On 2014/09/25 19:02:19, tfarina wrote:
> I looked at ui/accessibility/OWNERS
>
(https://chromium.googlesource.com/chromium/src/+/master/ui/accessibility/OWNERS).
> There we have aboxhall instead of plundblad.

This is on purpose; some people on the team are more appropriate reviewers for
certain parts of accessibility code than others.

[dmazzoni, 2014-09-25 19:15:15]

https://codereview.chromium.org/608573003/diff/1/chrome/browser/ui/views/acce...
File chrome/browser/ui/views/accessibility/OWNERS (left):

https://codereview.chromium.org/608573003/diff/1/chrome/browser/ui/views/acce...
chrome/browser/ui/views/accessibility/OWNERS:1: dmazzoni@chromium.org
On 2014/09/25 19:02:19, tfarina wrote:
> could you tweak the similarity passed to git cl upload, maybe something very
> low, say 10?
> 
> So it doesn't look like a copy of
> chrome/browser/extensions/api/braille_display_private/OWNERS?
> 
> Not a big deal though.

Done.

[sky, 2014-09-25 23:28:31]

LGTM

[dmazzoni, 2014-09-25 23:35:31]

The CQ bit was checked by dmazzoni@chromium.org

[commit-bot: I haz the power, 2014-09-25 23:36:41]

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

[commit-bot: I haz the power, 2014-09-26 00:50:26]

Committed patchset #2 (id:20001) as fe87e38c669431598f513e5d6b6c70ff8d48be37

[commit-bot: I haz the power, 2014-09-26 00:51:03]

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