Revert of Don't use barcodes in ProtocolPerfTests

remoting/host/chromoting_host.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef REMOTING_HOST_CHROMOTING_HOST_H_
 #define REMOTING_HOST_CHROMOTING_HOST_H_
 
+#include <list>
 #include <memory>
 #include <string>
-#include <vector>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_vector.h"
 #include "base/memory/weak_ptr.h"
 #include "base/observer_list.h"
 #include "base/threading/non_thread_safe.h"
 #include "base/threading/thread.h"
 #include "net/base/backoff_entry.h"
 #include "remoting/host/client_session.h"
@@ skipping 38 matching lines @@
 //
 // 3. When the user is disconnected, we will pause the ScreenRecorder
 //    and try to terminate the threads we have created. This will allow
 //    all pending tasks to complete. After all of that completed we
 //    return to the idle state. We then go to step (2) if there a new
 //    incoming connection.
 class ChromotingHost : public base::NonThreadSafe,
                        public ClientSession::EventHandler,
                        public HostStatusMonitor {
  public:
-  typedef std::vector<std::unique_ptr<ClientSession>> ClientSessions;
-
   // |desktop_environment_factory| must outlive this object.
   ChromotingHost(
       DesktopEnvironmentFactory* desktop_environment_factory,
       std::unique_ptr<protocol::SessionManager> session_manager,
       scoped_refptr<protocol::TransportContext> transport_context,
       scoped_refptr<base::SingleThreadTaskRunner> audio_task_runner,
       scoped_refptr<base::SingleThreadTaskRunner> video_encode_task_runner);
   ~ChromotingHost() override;
 
   // Asynchronously starts the host.
@@ skipping 47 matching lines @@
   // The host uses a pairing registry to generate and store pairing information
   // for clients for PIN-less authentication.
   scoped_refptr<protocol::PairingRegistry> pairing_registry() const {
     return pairing_registry_;
   }
   void set_pairing_registry(
       scoped_refptr<protocol::PairingRegistry> pairing_registry) {
     pairing_registry_ = pairing_registry;
   }
 
-  const ClientSessions& client_sessions_for_tests() { return clients_; }
-
   base::WeakPtr<ChromotingHost> AsWeakPtr() {
     return weak_factory_.GetWeakPtr();
   }
 
  private:
   friend class ChromotingHostTest;
 
+  typedef std::list<ClientSession*> ClientList;
   typedef ScopedVector<HostExtension> HostExtensionList;
 
   // Immediately disconnects all active clients. Host-internal components may
   // shutdown asynchronously, but the caller is guaranteed not to receive
   // callbacks for disconnected clients after this call returns.
   void DisconnectAllClients();
 
   // Unless specified otherwise all members of this class must be
   // used on the network thread only.
 
   // Parameters specified when the host was created.
   DesktopEnvironmentFactory* desktop_environment_factory_;
   std::unique_ptr<protocol::SessionManager> session_manager_;
   scoped_refptr<protocol::TransportContext> transport_context_;
   scoped_refptr<base::SingleThreadTaskRunner> audio_task_runner_;
   scoped_refptr<base::SingleThreadTaskRunner> video_encode_task_runner_;
 
   // Must be used on the network thread only.
   base::ObserverList<HostStatusObserver> status_observers_;
 
   // The connections to remote clients.
-  ClientSessions clients_;
+  ClientList clients_;
 
   // True if the host has been started.
   bool started_;
 
   // Login backoff state.
   net::BackoffEntry login_backoff_;
 
   // True if the curtain mode is enabled.
   bool enable_curtaining_;
 
   // The maximum duration of any session.
   base::TimeDelta max_session_duration_;
 
   // The pairing registry for PIN-less authentication.
   scoped_refptr<protocol::PairingRegistry> pairing_registry_;
 
   // List of host extensions.
   HostExtensionList extensions_;
 
   base::WeakPtrFactory<ChromotingHost> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromotingHost);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_CHROMOTING_HOST_H_