Adding security owners for content/shell/common/*_messages*.h

Adding security owners for content/shell/common/*_messages*.h

This CL avoids future presubmit violations like the
one quoted below:

    ** Presubmit Warnings **
    Found changes to IPC files without a security OWNER!

    ***************
    content/shell/common/OWNERS is missing the following lines:

    per-file *_messages*.h=set noparent
    per-file *_messages*.h=file://ipc/SECURITY_OWNERS

    for changed files:
      content/shell/common/shell_messages.h
    ***************

The presubmit checks performed above were added in r400748.

BUG=611905
Committed: https://crrev.com/cca690a0484a12b084313905841dc17e71ca11bc
Cr-Commit-Position: refs/heads/master@{#419285}

[Łukasz Anforowicz, 2016-09-16 19:11:19]

lukasza@chromium.org changed reviewers:
+ nasko@chromium.org

[Łukasz Anforowicz, 2016-09-16 19:11:20]

nasko@, can you take a look please? (as both a //content OWNER and a
security/IPC reviewer)

In theory content shell messages and layout test messages don't need a security
review (they shouldn't be included in  the product code we actually ship to end
users), but having security reviewers 1) doesn't hurt and 2) keeps presubmit
checks simple.

WDYT?

[dcheng, 2016-09-16 19:16:12]

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

[dcheng, 2016-09-16 19:16:12]

rs lgtm

[nasko, 2016-09-16 20:44:10]

LGTM

[Łukasz Anforowicz, 2016-09-16 20:51:52]

The CQ bit was checked by lukasza@chromium.org

[commit-bot: I haz the power, 2016-09-16 20:52:21]

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-09-16 21:44:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-16 21:46:59]

Description was changed from

==========
Adding security owners for content/shell/common/*_messages*.h

This CL avoids future presubmit violations like the
one quoted below:

    ** Presubmit Warnings **
    Found changes to IPC files without a security OWNER!

    ***************
    content/shell/common/OWNERS is missing the following lines:

    per-file *_messages*.h=set noparent
    per-file *_messages*.h=file://ipc/SECURITY_OWNERS

    for changed files:
      content/shell/common/shell_messages.h
    ***************

The presubmit checks performed above were added in r400748.

BUG=611905
==========

to

==========
Adding security owners for content/shell/common/*_messages*.h

This CL avoids future presubmit violations like the
one quoted below:

    ** Presubmit Warnings **
    Found changes to IPC files without a security OWNER!

    ***************
    content/shell/common/OWNERS is missing the following lines:

    per-file *_messages*.h=set noparent
    per-file *_messages*.h=file://ipc/SECURITY_OWNERS

    for changed files:
      content/shell/common/shell_messages.h
    ***************

The presubmit checks performed above were added in r400748.

BUG=611905

Committed: https://crrev.com/cca690a0484a12b084313905841dc17e71ca11bc
Cr-Commit-Position: refs/heads/master@{#419285}
==========

[commit-bot: I haz the power, 2016-09-16 21:47:00]

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