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

Unified Diff: mojo/public/cpp/bindings/tests/validation_unittest.cc

Issue 644963004: Some more virtual/override updates. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 6 years, 2 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: mojo/public/cpp/bindings/tests/validation_unittest.cc
diff --git a/mojo/public/cpp/bindings/tests/validation_unittest.cc b/mojo/public/cpp/bindings/tests/validation_unittest.cc
index 8f3adb88fda6d20d7793c7941365faf7b1f5d72a..c7edf19ec47a2e64033b2a3a2d7e9dab2148be14 100644
--- a/mojo/public/cpp/bindings/tests/validation_unittest.cc
+++ b/mojo/public/cpp/bindings/tests/validation_unittest.cc
@@ -187,7 +187,7 @@ class DummyMessageReceiver : public MessageReceiver {
class ValidationTest : public testing::Test {
public:
- virtual ~ValidationTest() {}
+ ~ValidationTest() override {}
private:
Environment env_;
@@ -197,9 +197,9 @@ class ValidationIntegrationTest : public ValidationTest {
public:
ValidationIntegrationTest() : test_message_receiver_(nullptr) {}
- virtual ~ValidationIntegrationTest() {}
+ ~ValidationIntegrationTest() override {}
- virtual void SetUp() override {
+ void SetUp() override {
ScopedMessagePipeHandle tester_endpoint;
ASSERT_EQ(MOJO_RESULT_OK,
CreateMessagePipe(nullptr, &tester_endpoint, &testee_endpoint_));
@@ -207,7 +207,7 @@ class ValidationIntegrationTest : public ValidationTest {
new TestMessageReceiver(this, tester_endpoint.Pass());
}
- virtual void TearDown() override {
+ void TearDown() override {
delete test_message_receiver_;
test_message_receiver_ = nullptr;
« no previous file with comments | « mojo/public/cpp/bindings/tests/struct_unittest.cc ('k') | mojo/public/cpp/environment/tests/async_waiter_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698