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

Unified Diff: remoting/protocol/fake_authenticator.cc

Issue 1545743002: Move ownership of Transport out of Session. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@move_not_pass_client
Patch Set: Created 5 years 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/protocol/fake_authenticator.cc
diff --git a/remoting/protocol/fake_authenticator.cc b/remoting/protocol/fake_authenticator.cc
index 5cb2f21dd59bad821b33da6b80e56f0eb67e0c6d..705bdf19de4e3fbfd374d299bfec772f23cef058 100644
--- a/remoting/protocol/fake_authenticator.cc
+++ b/remoting/protocol/fake_authenticator.cc
@@ -35,9 +35,9 @@ void FakeChannelAuthenticator::SecureAndAuthenticate(
const DoneCallback& done_callback) {
socket_ = std::move(socket);
- if (async_) {
- done_callback_ = done_callback;
+ done_callback_ = done_callback;
+ if (async_) {
if (result_ != net::OK) {
// Don't write anything if we are going to reject auth to make test
// ordering deterministic.

Powered by Google App Engine
This is Rietveld 408576698