Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(212)

Unified Diff: sandbox/linux/seccomp-bpf/bpf_tester_compatibility_delegate.h

Issue 1124763003: Update from https://crrev.com/327068 (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: update nacl, buildtools, fix display_change_notifier_unittest Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sandbox/linux/seccomp-bpf/bpf_tester_compatibility_delegate.h
diff --git a/sandbox/linux/seccomp-bpf/bpf_tester_compatibility_delegate.h b/sandbox/linux/seccomp-bpf/bpf_tester_compatibility_delegate.h
index 18699138d551258601ad4f571f246e108b4c8803..7736c1506fbf475762b6aa69983c21e833266d3c 100644
--- a/sandbox/linux/seccomp-bpf/bpf_tester_compatibility_delegate.h
+++ b/sandbox/linux/seccomp-bpf/bpf_tester_compatibility_delegate.h
@@ -25,9 +25,9 @@ class BPFTesterCompatibilityDelegate : public BPFTesterDelegate {
explicit BPFTesterCompatibilityDelegate(TestFunction test_function)
: aux_(), test_function_(test_function) {}
- virtual ~BPFTesterCompatibilityDelegate() {}
+ ~BPFTesterCompatibilityDelegate() override {}
- virtual scoped_ptr<bpf_dsl::Policy> GetSandboxBPFPolicy() override {
+ scoped_ptr<bpf_dsl::Policy> GetSandboxBPFPolicy() override {
// The current method is guaranteed to only run in the child process
// running the test. In this process, the current object is guaranteed
// to live forever. So it's ok to pass aux_pointer_for_policy_ to
@@ -35,7 +35,7 @@ class BPFTesterCompatibilityDelegate : public BPFTesterDelegate {
return scoped_ptr<bpf_dsl::Policy>(new Policy(&aux_));
}
- virtual void RunTestFunction() override {
+ void RunTestFunction() override {
// Run the actual test.
// The current object is guaranteed to live forever in the child process
// where this will run.

Powered by Google App Engine
This is Rietveld 408576698