Disable agents when client disconnects

components/ui_devtools/devtools_client.h

 // Copyright 2016 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 COMPONENTS_UI_DEVTOOLS_DEVTOOLS_CLIENT_H_
 #define COMPONENTS_UI_DEVTOOLS_DEVTOOLS_CLIENT_H_
 
 #include <string>
 
 #include "components/ui_devtools/DOM.h"
@@ skipping 11 matching lines @@
 // etc). This client is then attached to the UiDevToolsServer and all messages
 // from this client are sent over the web socket owned by the server.
 class UiDevToolsClient : public protocol::FrontendChannel {
  public:
   static const int kNotConnected = -1;
 
   UiDevToolsClient(const std::string& name, UiDevToolsServer* server);
   ~UiDevToolsClient() override;
 
   void AddAgent(std::unique_ptr<UiDevToolsAgent> agent);
+  void DisableAllAgents();

[sadrul, 2016/11/02 16:05:57]

This should go into private section? Or do you expect a use-case for
DisableAllAgents() without also disconnecting at the same time? If not, maybe
just remove, and move the code into Disconnect().

[Sarmad Hashmi, 2016/11/02 16:38:57]

Done.

+  void Disconnect();
   void Dispatch(const std::string& data);
 
   bool connected() const;
   void set_connection_id(int connection_id);
   const std::string& name() const;
 
  private:
   // protocol::FrontendChannel
   void sendProtocolResponse(int callId, const String& message) override;
   void sendProtocolNotification(const String& message) override;
   void flushProtocolNotifications() override;
 
   std::string name_;
   int connection_id_;
 
   std::vector<std::unique_ptr<UiDevToolsAgent>> agents_;
   protocol::UberDispatcher dispatcher_;
   UiDevToolsServer* server_;
 
   DISALLOW_COPY_AND_ASSIGN(UiDevToolsClient);
 };
 
 }  // namespace devtools
 }  // namespace ui
 
 #endif  // COMPONENTS_UI_DEVTOOLS_DEVTOOLS_CLIENT_H_