Disable agents when client disconnects

components/ui_devtools/devtools_server.cc

 // 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.
 
 #include "components/ui_devtools/devtools_server.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/format_macros.h"
@@ skipping 143 matching lines @@
   task_runner_->PostTask(FROM_HERE, base::Bind(&UiDevToolsClient::Dispatch,
                                                base::Unretained(client), data));
 }
 
 void UiDevToolsServer::OnClose(int connection_id) {
   ConnectionsMap::iterator it = connections_.find(connection_id);
   if (it == connections_.end())
     return;
   UiDevToolsClient* client = it->second;
   DCHECK(client);
+  client->DisableAllAgents();
   client->set_connection_id(UiDevToolsClient::kNotConnected);

[sadrul, 2016/11/02 01:45:39]

Would it make sense to combine these two into a single Disconnect() method?

[Sarmad Hashmi, 2016/11/02 02:43:53]

Great idea! Done.

   connections_.erase(it);
 }
 
 }  // namespace devtools
 }  // namespace ui