Add per-file OWNERS for content/browser/encrypted_media_browsertest.cc

Add per-file OWNERS for content/browser/encrypted_media_browsertest.cc

BUG=none
TEST=none


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

[xhwang, 2012-11-08 21:12:57]

As commented in issue 11383004. PTAL!

[jam, 2012-11-08 21:20:17]

On 2012/11/08 21:12:57, xhwang wrote:
> As commented in issue 11383004. PTAL!

also content/browser/media_browsertest.cc, and with that lgtm, thanks

[commit-bot: I haz the power, 2012-11-08 21:40:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/11358168/3001

[jam, 2012-11-08 21:49:21]

https://chromiumcodereview.appspot.com/11358168/diff/3001/content/browser/OWNERS
File content/browser/OWNERS (right):

https://chromiumcodereview.appspot.com/11358168/diff/3001/content/browser/OWN...
content/browser/OWNERS:4: per-file *media_browsertest.cc=acolwell@chromium.org
are you sure *media_browsertest works?

[xhwang, 2012-11-09 01:36:45]

On 2012/11/08 21:49:21, John Abd-El-Malek wrote:
>
https://chromiumcodereview.appspot.com/11358168/diff/3001/content/browser/OWNERS
> File content/browser/OWNERS (right):
> 
>
https://chromiumcodereview.appspot.com/11358168/diff/3001/content/browser/OWN...
> content/browser/OWNERS:4: per-file
mailto:*media_browsertest.cc=acolwell@chromium.org
> are you sure *media_browsertest works?

This is the first time I add per-file OWNERS, so I read the guide and "*" should
be supported:
http://www.chromium.org/developers/owners-files

Filename globs follow the simple UNIX shell conventions ("*" and "?" are
supported), and relative and absolute paths are not allowed (globs may only
refer to the files in the current directory).

There's also an example:
per-file *.gypi=baz@chromium.org

If you feel this might not work I can specify the file names explicitly.

[jam, 2012-11-09 01:39:18]

On 2012/11/09 01:36:45, xhwang wrote:
> On 2012/11/08 21:49:21, John Abd-El-Malek wrote:
> >
>
https://chromiumcodereview.appspot.com/11358168/diff/3001/content/browser/OWNERS
> > File content/browser/OWNERS (right):
> > 
> >
>
https://chromiumcodereview.appspot.com/11358168/diff/3001/content/browser/OWN...
> > content/browser/OWNERS:4: per-file
> mailto:*media_browsertest.cc=acolwell@chromium.org
> > are you sure *media_browsertest works?
> 
> This is the first time I add per-file OWNERS, so I read the guide and "*"
should
> be supported:
> http://www.chromium.org/developers/owners-files
> 
> Filename globs follow the simple UNIX shell conventions ("*" and "?" are
> supported), and relative and absolute paths are not allowed (globs may only
> refer to the files in the current directory).
> 
> There's also an example:
> per-file mailto:*.gypi=baz@chromium.org
> 
> If you feel this might not work I can specify the file names explicitly.

ah ok, thanks for checking (I didn't know this)

[commit-bot: I haz the power, 2012-11-09 01:39:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/11358168/3001

[commit-bot: I haz the power, 2012-11-09 01:47:08]

Change committed as 166839