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

Side by Side Diff: remoting/protocol/negotiating_authenticator_base.h

Issue 205583011: [Draft] Fix canceling pin prompt causes host overload (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix Unittests and also reject connections upon authenticating Created 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef REMOTING_PROTOCOL_NEGOTIATING_AUTHENTICATOR_BASE_H_ 5 #ifndef REMOTING_PROTOCOL_NEGOTIATING_AUTHENTICATOR_BASE_H_
6 #define REMOTING_PROTOCOL_NEGOTIATING_AUTHENTICATOR_BASE_H_ 6 #define REMOTING_PROTOCOL_NEGOTIATING_AUTHENTICATOR_BASE_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 // * The host never sends its own supported methods back to the client, so once 57 // * The host never sends its own supported methods back to the client, so once
58 // the host picks a method from the client's list, it's final. 58 // the host picks a method from the client's list, it's final.
59 // * Any change in this class must maintain compatibility between any version 59 // * Any change in this class must maintain compatibility between any version
60 // mix of webapp, client plugin and host, for both Me2Me and IT2Me. 60 // mix of webapp, client plugin and host, for both Me2Me and IT2Me.
61 class NegotiatingAuthenticatorBase : public Authenticator { 61 class NegotiatingAuthenticatorBase : public Authenticator {
62 public: 62 public:
63 virtual ~NegotiatingAuthenticatorBase(); 63 virtual ~NegotiatingAuthenticatorBase();
64 64
65 // Authenticator interface. 65 // Authenticator interface.
66 virtual State state() const OVERRIDE; 66 virtual State state() const OVERRIDE;
67 virtual bool started() const OVERRIDE;
67 virtual RejectionReason rejection_reason() const OVERRIDE; 68 virtual RejectionReason rejection_reason() const OVERRIDE;
68 virtual scoped_ptr<ChannelAuthenticator> 69 virtual scoped_ptr<ChannelAuthenticator>
69 CreateChannelAuthenticator() const OVERRIDE; 70 CreateChannelAuthenticator() const OVERRIDE;
70 71
71 // Calls |current_authenticator_| to process |message|, passing the supplied 72 // Calls |current_authenticator_| to process |message|, passing the supplied
72 // |resume_callback|. 73 // |resume_callback|.
73 void ProcessMessageInternal(const buzz::XmlElement* message, 74 void ProcessMessageInternal(const buzz::XmlElement* message,
74 const base::Closure& resume_callback); 75 const base::Closure& resume_callback);
75 76
76 const AuthenticationMethod& current_method_for_testing() const { 77 const AuthenticationMethod& current_method_for_testing() const {
(...skipping 24 matching lines...) Expand all
101 RejectionReason rejection_reason_; 102 RejectionReason rejection_reason_;
102 103
103 private: 104 private:
104 DISALLOW_COPY_AND_ASSIGN(NegotiatingAuthenticatorBase); 105 DISALLOW_COPY_AND_ASSIGN(NegotiatingAuthenticatorBase);
105 }; 106 };
106 107
107 } // namespace protocol 108 } // namespace protocol
108 } // namespace remoting 109 } // namespace remoting
109 110
110 #endif // REMOTING_PROTOCOL_NEGOTIATING_AUTHENTICATOR_BASE_H_ 111 #endif // REMOTING_PROTOCOL_NEGOTIATING_AUTHENTICATOR_BASE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698