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

Unified Diff: remoting/protocol/negotiating_client_authenticator.h

Issue 1768383004: Cleanup AuthenticatorMethod usage. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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/negotiating_client_authenticator.h
diff --git a/remoting/protocol/negotiating_client_authenticator.h b/remoting/protocol/negotiating_client_authenticator.h
index 1c03afd976eb81c93c0b5c2ffc9137e4e93b85ab..c664a6555915ca51a9722d01a6136c3ba089fb80 100644
--- a/remoting/protocol/negotiating_client_authenticator.h
+++ b/remoting/protocol/negotiating_client_authenticator.h
@@ -11,7 +11,6 @@
#include "base/macros.h"
#include "base/memory/scoped_ptr.h"
#include "base/memory/weak_ptr.h"
-#include "remoting/protocol/authentication_method.h"
#include "remoting/protocol/authenticator.h"
#include "remoting/protocol/negotiating_authenticator_base.h"
#include "remoting/protocol/third_party_client_authenticator.h"
@@ -29,8 +28,7 @@ class NegotiatingClientAuthenticator : public NegotiatingAuthenticatorBase {
const std::string& shared_secret,
const std::string& authentication_tag,
const FetchSecretCallback& fetch_secret_callback,
- scoped_ptr<ThirdPartyClientAuthenticator::TokenFetcher> token_fetcher_,
- const std::vector<AuthenticationMethod>& methods);
+ scoped_ptr<ThirdPartyClientAuthenticator::TokenFetcher> token_fetcher_);
~NegotiatingClientAuthenticator() override;
@@ -80,7 +78,7 @@ class NegotiatingClientAuthenticator : public NegotiatingAuthenticatorBase {
scoped_ptr<ThirdPartyClientAuthenticator::TokenFetcher> token_fetcher_;
// Internal NegotiatingClientAuthenticator data.
- bool method_set_by_host_;
+ bool method_set_by_host_ = false;
base::WeakPtrFactory<NegotiatingClientAuthenticator> weak_factory_;
DISALLOW_COPY_AND_ASSIGN(NegotiatingClientAuthenticator);
« no previous file with comments | « remoting/protocol/negotiating_authenticator_unittest.cc ('k') | remoting/protocol/negotiating_client_authenticator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698