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

Side by Side Diff: remoting/host/it2me/it2me_host.h

Issue 2310303002: Moving It2Me confirmation prompt into the Validation callback flow (Closed)
Patch Set: Addressing CR Feedback Created 4 years, 3 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_HOST_IT2ME_IT2ME_HOST_H_ 5 #ifndef REMOTING_HOST_IT2ME_IT2ME_HOST_H_
6 #define REMOTING_HOST_IT2ME_IT2ME_HOST_H_ 6 #define REMOTING_HOST_IT2ME_IT2ME_HOST_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <string> 9 #include <string>
10 10
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 } 115 }
116 base::WeakPtr<It2MeHost::Observer> observer() { return observer_; } 116 base::WeakPtr<It2MeHost::Observer> observer() { return observer_; }
117 117
118 private: 118 private:
119 // Updates state of the host. Can be called only on the network thread. 119 // Updates state of the host. Can be called only on the network thread.
120 void SetState(It2MeHostState state, const std::string& error_message); 120 void SetState(It2MeHostState state, const std::string& error_message);
121 121
122 // Returns true if the host is connected. 122 // Returns true if the host is connected.
123 bool IsConnected() const; 123 bool IsConnected() const;
124 124
125 // Presents a confirmation dialog to the user before starting the connection
126 // process.
127 void ShowConfirmationPrompt();
128
129 // Processes the result of the confirmation dialog. 125 // Processes the result of the confirmation dialog.
130 void OnConfirmationResult(It2MeConfirmationDialog::Result result); 126 void OnConfirmationResult(
127 const protocol::ValidatingAuthenticator::ResultCallback& result_callback,
128 It2MeConfirmationDialog::Result result);
131 129
132 // Called by Connect() to check for policies and start connection process. 130 // Called by Connect() to check for policies and start connection process.
133 void ReadPolicyAndConnect(); 131 void ReadPolicyAndConnect();
134 132
135 // Called by ReadPolicyAndConnect once policies have been read. 133 // Called by ReadPolicyAndConnect once policies have been read.
136 void FinishConnect(); 134 void FinishConnect();
137 135
138 // Called when the support host registration completes. 136 // Called when the support host registration completes.
139 void OnReceivedSupportID(const std::string& support_id, 137 void OnReceivedSupportID(const std::string& support_id,
140 const base::TimeDelta& lifetime, 138 const base::TimeDelta& lifetime,
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
226 const XmppSignalStrategy::XmppServerConfig& xmpp_server_config, 224 const XmppSignalStrategy::XmppServerConfig& xmpp_server_config,
227 const std::string& directory_bot_jid); 225 const std::string& directory_bot_jid);
228 226
229 private: 227 private:
230 DISALLOW_COPY_AND_ASSIGN(It2MeHostFactory); 228 DISALLOW_COPY_AND_ASSIGN(It2MeHostFactory);
231 }; 229 };
232 230
233 } // namespace remoting 231 } // namespace remoting
234 232
235 #endif // REMOTING_HOST_IT2ME_IT2ME_HOST_H_ 233 #endif // REMOTING_HOST_IT2ME_IT2ME_HOST_H_
OLDNEW
« no previous file with comments | « remoting/host/it2me/it2me_confirmation_dialog_proxy_unittest.cc ('k') | remoting/host/it2me/it2me_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698