Fixing Intermittent SecurityKey Unittest Failures

remoting/host/security_key/fake_security_key_ipc_client.cc

Index: remoting/host/security_key/fake_security_key_ipc_client.cc
diff --git a/remoting/host/security_key/fake_security_key_ipc_client.cc b/remoting/host/security_key/fake_security_key_ipc_client.cc
index 70eb8bc053a0c9797730a00333fed5ce0572f8c3..47f0c53148365be665d2f49ded6b2e09cdc937b2 100644
--- a/remoting/host/security_key/fake_security_key_ipc_client.cc
+++ b/remoting/host/security_key/fake_security_key_ipc_client.cc
@@ -109,7 +109,12 @@ void FakeSecurityKeyIpcClient::OnInvalidSession() {
 
 void FakeSecurityKeyIpcClient::OnChannelConnected(int32_t peer_pid) {
   ipc_channel_connected_ = true;
-  channel_event_callback_.Run();
+
+  // We don't always want to fire this event as only a subset of tests care
+  // about the channel being connected.  Tests that do care can register for it.
+  if (on_channel_connected_callback_) {
+    on_channel_connected_callback_.Run();
+  }
 }
 
 void FakeSecurityKeyIpcClient::OnChannelError() {