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

Unified Diff: remoting/host/it2me/it2me_host.h

Issue 2650443002: Webapp share dialog is not closed when client end of the connection is closed (Closed)
Patch Set: Comment tweaks Created 3 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/host/it2me/it2me_host.h
diff --git a/remoting/host/it2me/it2me_host.h b/remoting/host/it2me/it2me_host.h
index efeb6428f0e64eee0a95115fdd8f04442c89ef60..e8e0c4d2807cff461ab00eefab10fd7a52976d0f 100644
--- a/remoting/host/it2me/it2me_host.h
+++ b/remoting/host/it2me/it2me_host.h
@@ -65,7 +65,7 @@ class It2MeHost : public base::RefCountedThreadSafe<It2MeHost>,
It2MeHost(std::unique_ptr<ChromotingHostContext> context,
std::unique_ptr<PolicyWatcher> policy_watcher,
- std::unique_ptr<It2MeConfirmationDialog> confirmation_dialog,
+ std::unique_ptr<It2MeConfirmationDialogFactory> dialog_factory_,
base::WeakPtr<It2MeHost::Observer> observer,
std::unique_ptr<SignalStrategy> signal_strategy,
const std::string& username,
@@ -182,7 +182,7 @@ class It2MeHost : public base::RefCountedThreadSafe<It2MeHost>,
int failed_login_attempts_ = 0;
std::unique_ptr<PolicyWatcher> policy_watcher_;
- std::unique_ptr<It2MeConfirmationDialog> confirmation_dialog_;
+ std::unique_ptr<It2MeConfirmationDialogFactory> confirmation_dialog_factory_;
std::unique_ptr<It2MeConfirmationDialogProxy> confirmation_dialog_proxy_;
// Host the current nat traversal policy setting.
@@ -192,6 +192,9 @@ class It2MeHost : public base::RefCountedThreadSafe<It2MeHost>,
std::string required_client_domain_;
std::string required_host_domain_;
+ // Tracks the JID of the remote user when in a connecting state.
+ std::string connecting_jid_;
+
// Indicates whether or not a policy has ever been read. This is to ensure
// that on startup, we do not accidentally start a connection before we have
// queried our policy restrictions.

Powered by Google App Engine
This is Rietveld 408576698