Drop unused NaCl Debugger whitelisting.

Drop unused NaCl Debugger whitelisting.

BUG=374955
R=michaelpg@chromium.org,mkwst@chromium.org

Committed: https://crrev.com/8a872353e0eea9fe9e7af2877f218a14a4bbfebe
Cr-Commit-Position: refs/heads/master@{#422817}

[michaelpg, 2016-10-03 23:15:43]

lgtm (but not an owner), thanks!

[bradnelson, 2016-10-04 05:36:04]

Description was changed from

==========
Drop unused NaCl Debugger whitelisting.

BUG=374955
R=michaelpg@chromium.org
==========

to

==========
Drop unused NaCl Debugger whitelisting.

BUG=374955
R=michaelpg@chromium.org,mkwst@chromium.org
==========

[bradnelson, 2016-10-04 05:36:04]

bradnelson@chromium.org changed reviewers:
+ mkwst@chromium.org

[bradnelson, 2016-10-04 05:37:08]

+mkwst@chromium.org

Mike, OWNERS says you're only for docs, as the extensions owners is now empty,
any recommendation on who to bubble up to ?

Thanks

[michaelpg, 2016-10-04 05:45:00]

michaelpg@chromium.org changed reviewers:
+ rdevlin.cronin@chromium.org

[michaelpg, 2016-10-04 05:45:00]

On 2016/10/04 05:37:08, bradnelson wrote:
> mailto:+mkwst@chromium.org
> 
> Mike, OWNERS says you're only for docs, as the extensions owners is now empty,
> any recommendation on who to bubble up to ?
> 
> Thanks

OWNERS files work recursively, so you should be able to use
src/chrome/common/extensions/OWNERS, which references src/extensions/OWNERS.
Does `git cl owners` suggest someone from there?

(also, adding devlin both as an owner, and having context for this CL)

[bradnelson, 2016-10-04 06:07:46]

mkwst was what git cl owners suggests, but probably just because it's not able
to read the "docs only" comment.
The extensions/OWNERS is empty, so it bubbles up to chrome/OWNERS.

[bradnelson, 2016-10-04 06:08:44]

Oh, misread, I see, there is the link to src/extensions/OWNERS.

Devlin, can you please review? Thanks!

[Devlin, 2016-10-04 15:04:39]

lgtm

[bradn, 2016-10-04 15:18:09]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2016-10-04 15:18:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-04 16:03:37]

Description was changed from

==========
Drop unused NaCl Debugger whitelisting.

BUG=374955
R=michaelpg@chromium.org,mkwst@chromium.org
==========

to

==========
Drop unused NaCl Debugger whitelisting.

BUG=374955
R=michaelpg@chromium.org,mkwst@chromium.org
==========

[commit-bot: I haz the power, 2016-10-04 16:03:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-04 16:06:46]

Description was changed from

==========
Drop unused NaCl Debugger whitelisting.

BUG=374955
R=michaelpg@chromium.org,mkwst@chromium.org
==========

to

==========
Drop unused NaCl Debugger whitelisting.

BUG=374955
R=michaelpg@chromium.org,mkwst@chromium.org

Committed: https://crrev.com/8a872353e0eea9fe9e7af2877f218a14a4bbfebe
Cr-Commit-Position: refs/heads/master@{#422817}
==========

[commit-bot: I haz the power, 2016-10-04 16:06:47]

Patchset 1 (id:??) landed as
https://crrev.com/8a872353e0eea9fe9e7af2877f218a14a4bbfebe
Cr-Commit-Position: refs/heads/master@{#422817}