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

Unified Diff: remoting/host/security_key/gnubby_extension.cc

Issue 2085353004: Update GnubbyAuthHandler to use the current session ID (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@host_extension
Patch Set: Fixing a build break Created 4 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
Index: remoting/host/security_key/gnubby_extension.cc
diff --git a/remoting/host/security_key/gnubby_extension.cc b/remoting/host/security_key/gnubby_extension.cc
index 469baa1c60239e83334fa6195a209c40f8dddd9f..885ed3045d4f24b52c67547dbde152b305d0627e 100644
--- a/remoting/host/security_key/gnubby_extension.cc
+++ b/remoting/host/security_key/gnubby_extension.cc
@@ -24,13 +24,12 @@ std::string GnubbyExtension::capability() const {
}
std::unique_ptr<HostExtensionSession> GnubbyExtension::CreateExtensionSession(
- ClientSessionDetails* client_session_details,
+ ClientSessionDetails* details,
protocol::ClientStub* client_stub) {
// TODO(joedow): Update this mechanism to allow for multiple sessions. The
// extension will only send messages through the initial
- // |client_stub| with the current design.
- // TODO(joedow): Update GnubbyExtensionSession for |client_session_details|.
- return base::WrapUnique(new GnubbyExtensionSession(client_stub));
+ // |client_stub| and |details| with the current design.
+ return base::WrapUnique(new GnubbyExtensionSession(details, client_stub));
}
} // namespace remoting

Powered by Google App Engine
This is Rietveld 408576698