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

Unified Diff: remoting/host/client_session.h

Issue 2967793002: [Chromoting] Remove DataChannelManagerCallbacks constructor parameter of ClientSession (Closed)
Patch Set: Resolve review comments Created 3 years, 5 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
« no previous file with comments | « remoting/host/chromoting_host_unittest.cc ('k') | remoting/host/client_session.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/client_session.h
diff --git a/remoting/host/client_session.h b/remoting/host/client_session.h
index d0c830bc1c50308b1af2b51e8fa154a747bb55cc..c439355b61f7c51c33d7a45a014c379099bfe876 100644
--- a/remoting/host/client_session.h
+++ b/remoting/host/client_session.h
@@ -97,9 +97,7 @@ class ClientSession : public protocol::HostStub,
const DesktopEnvironmentOptions& desktop_environment_options,
const base::TimeDelta& max_duration,
scoped_refptr<protocol::PairingRegistry> pairing_registry,
- const std::vector<HostExtension*>& extensions,
- const std::vector<protocol::DataChannelManager::NameCallbackPair>&
- data_channel_callbacks);
+ const std::vector<HostExtension*>& extensions);
~ClientSession() override;
// Returns the set of capabilities negotiated between client and host.
@@ -145,6 +143,11 @@ class ClientSession : public protocol::HostStub,
return client_capabilities_.get();
}
+ // Registers a DataChannelManager callback for testing.
+ void RegisterCreateHandlerCallbackForTesting(
+ const std::string& prefix,
+ protocol::DataChannelManager::CreateHandlerCallback constructor);
+
void SetEventTimestampsSourceForTests(
scoped_refptr<protocol::InputEventTimestampsSource>
event_timestamp_source);
« no previous file with comments | « remoting/host/chromoting_host_unittest.cc ('k') | remoting/host/client_session.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698