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

Unified Diff: remoting/host/chromoting_host.cc

Issue 2967793002: [Chromoting] Remove DataChannelManagerCallbacks constructor parameter of ClientSession (Closed)
Patch Set: Created 3 years, 6 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 | « no previous file | remoting/host/chromoting_host_unittest.cc » ('j') | remoting/protocol/data_channel_manager.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/chromoting_host.cc
diff --git a/remoting/host/chromoting_host.cc b/remoting/host/chromoting_host.cc
index 3e2ce47a3855cecfa1ab73a8f3228ae54fc71160..4bfd9e2b994a85513d6bcd3afb1adcc14451b06f 100644
--- a/remoting/host/chromoting_host.cc
+++ b/remoting/host/chromoting_host.cc
@@ -263,8 +263,7 @@ void ChromotingHost::OnIncomingSession(
clients_.push_back(base::MakeUnique<ClientSession>(
this, std::move(connection), desktop_environment_factory_,
desktop_environment_options_, max_session_duration_, pairing_registry_,
- extension_ptrs,
- std::vector<protocol::DataChannelManager::NameCallbackPair>()));
jarhar 2017/07/05 17:51:42 In order to register callbacks from this file, wou
Hzj_jie 2017/07/05 18:34:31 That's usually not necessary: Optional data channe
+ extension_ptrs));
}
} // namespace remoting
« no previous file with comments | « no previous file | remoting/host/chromoting_host_unittest.cc » ('j') | remoting/protocol/data_channel_manager.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698