remove unused headers from sandbox_seccomp_bpf_linux.cc

remove unused headers from sandbox_seccomp_bpf_linux.cc

sandbox_seccomp_bpf_linux.cc includes linux/audit.h and linux/filter.h but does not seem to use any of the macros or types defined within.

BUG=279413
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=219630

[Mostyn Bramley-Moore, 2013-08-26 14:51:49]

@jln, piman: PTAL.

[jln (very slow on Chromium), 2013-08-26 19:02:17]

On 2013/08/26 14:51:49, Mostyn Bramley-Moore wrote:
> @jln, piman: PTAL.

lgtm

(I removed piman, who is already overwhelmed and doesn't need to look at this).

[jln (very slow on Chromium), 2013-08-26 19:02:39]

https://codereview.chromium.org/23226004/diff/1/content/common/sandbox_seccom...
File content/common/sandbox_seccomp_bpf_linux.cc (right):

https://codereview.chromium.org/23226004/diff/1/content/common/sandbox_seccom...
content/common/sandbox_seccomp_bpf_linux.cc:9: #include <linux/filter.h>
I don't think we'd need this one either?

[jln (very slow on Chromium), 2013-08-26 19:04:38]

Optional: open a bug and add a BUG= line. Removing headers often results in
reverts, BUG= is a good way to be notified of them.

[Mostyn Bramley-Moore, 2013-08-26 19:13:58]

On 2013/08/26 19:04:38, Julien Tinnes wrote:
> Optional: open a bug and add a BUG= line. Removing headers often results in
> reverts, BUG= is a good way to be notified of them.

Done: https://code.google.com/p/chromium/issues/detail?id=279413

I'm not sure what type to file it as- I'm not sure this is a "bug" per se, but
none of the other options looked more suitable.

[Mostyn Bramley-Moore, 2013-08-26 19:14:04]

https://codereview.chromium.org/23226004/diff/1/content/common/sandbox_seccom...
File content/common/sandbox_seccomp_bpf_linux.cc (right):

https://codereview.chromium.org/23226004/diff/1/content/common/sandbox_seccom...
content/common/sandbox_seccomp_bpf_linux.cc:9: #include <linux/filter.h>
On 2013/08/26 19:02:39, Julien Tinnes wrote:
> I don't think we'd need this one either?

Done.

[commit-bot: I haz the power, 2013-08-26 19:32:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/23226004/8001

[piman, 2013-08-26 20:12:18]

lgtm

[commit-bot: I haz the power, 2013-08-26 22:55:04]

Change committed as 219630