Fixes naming issues in Browser protocol handler + adds tests.

content/browser/devtools/protocol/browser_handler.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 "content/browser/devtools/protocol/browser_handler.h"
 
 namespace content {
 namespace devtools {
 namespace browser {
 
@@ skipping 69 matching lines @@
   DevToolsAgentHost::List agents = DevToolsAgentHost::GetOrCreateAll();
   for (DevToolsAgentHost::List::iterator it = agents.begin();
        it != agents.end(); ++it) {
     DevToolsAgentHost* agent_host = (*it).get();
     scoped_refptr<devtools::browser::TargetInfo> info =
         devtools::browser::TargetInfo::Create()->
             set_target_id(agent_host->GetId())->
             set_type(GetTypeString(agent_host))->
             set_title(agent_host->GetTitle())->
             set_url(agent_host->GetURL().spec());
+    infos->push_back(info);
   }
   return Response::OK();
 }
 
 Response BrowserHandler::Attach(const std::string& targetId) {
   scoped_refptr<DevToolsAgentHost> agent_host =
       DevToolsAgentHost::GetForId(targetId);
   if (!agent_host)
     return Response::ServerError("No target with given id found");
   bool success = agent_host->AttachClient(this);
@@ skipping 28 matching lines @@
       set_message(message));
 }
 
 void BrowserHandler::AgentHostClosed(DevToolsAgentHost* agent_host,
                                      bool replaced_with_another_client) {
 }
 
 }  // namespace browser
 }  // namespace devtools
 }  // namespace content