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

Unified Diff: remoting/protocol/third_party_authenticator_unittest.cc

Issue 667123002: Standardize usage of virtual/override/final in remoting/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.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: remoting/protocol/third_party_authenticator_unittest.cc
diff --git a/remoting/protocol/third_party_authenticator_unittest.cc b/remoting/protocol/third_party_authenticator_unittest.cc
index 9da75d3762077e0ce22f7f58e52e0956a61ed358..f14484f7a42cf92de4223fb361eb02fa2c28fbd3 100644
--- a/remoting/protocol/third_party_authenticator_unittest.cc
+++ b/remoting/protocol/third_party_authenticator_unittest.cc
@@ -40,7 +40,7 @@ namespace protocol {
class ThirdPartyAuthenticatorTest : public AuthenticatorTestBase {
class FakeTokenFetcher : public ThirdPartyClientAuthenticator::TokenFetcher {
public:
- virtual void FetchThirdPartyToken(
+ void FetchThirdPartyToken(
const GURL& token_url,
const std::string& scope,
const TokenFetchedCallback& token_fetched_callback) override {
@@ -68,9 +68,9 @@ class ThirdPartyAuthenticatorTest : public AuthenticatorTestBase {
: token_url_(kTokenUrl),
token_scope_(kTokenScope) {}
- virtual ~FakeTokenValidator() {}
+ ~FakeTokenValidator() override {}
- virtual void ValidateThirdPartyToken(
+ void ValidateThirdPartyToken(
const std::string& token,
const TokenValidatedCallback& token_validated_callback) override {
ASSERT_FALSE(token_validated_callback.is_null());
@@ -84,13 +84,9 @@ class ThirdPartyAuthenticatorTest : public AuthenticatorTestBase {
on_token_validated.Run(shared_secret);
}
- virtual const GURL& token_url() const override {
- return token_url_;
- }
+ const GURL& token_url() const override { return token_url_; }
- virtual const std::string& token_scope() const override {
- return token_scope_;
- }
+ const std::string& token_scope() const override { return token_scope_; }
private:
GURL token_url_;
« no previous file with comments | « remoting/protocol/third_party_authenticator_base.h ('k') | remoting/protocol/third_party_client_authenticator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698