Fixes naming issues in Browser protocol handler + adds tests.

components/devtools_discovery/devtools_discovery_manager.cc

 // Copyright 2015 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/devtools_discovery/devtools_discovery_manager.h"
 
 #include "base/stl_util.h"
 #include "components/devtools_discovery/basic_target_descriptor.h"
 #include "content/public/browser/devtools_agent_host.h"
 
@@ skipping 45 matching lines @@
 DevToolsDiscoveryManager::GetDescriptorsFromProviders() {
   DevToolsTargetDescriptor::List result;
   for (auto* provider : providers_) {
     DevToolsTargetDescriptor::List partial = provider->GetDescriptors();
     result.insert(result.begin(), partial.begin(), partial.end());
   }
   return result;
 }
 
 std::unique_ptr<base::DictionaryValue>
-DevToolsDiscoveryManager::HandleNewTargetCommand(
+DevToolsDiscoveryManager::HandleCreateTargetCommand(
     base::DictionaryValue* command_dict) {
   int id;
   std::string method;
   std::string url;
   const base::DictionaryValue* params_dict = nullptr;
   if (command_dict->GetInteger("id", &id) &&
       command_dict->GetString("method", &method) &&
       method == "Browser.createTarget" &&
       command_dict->GetDictionary("params", &params_dict) &&
       params_dict->GetString("url", &url)) {
     std::unique_ptr<devtools_discovery::DevToolsTargetDescriptor> descriptor =
         CreateNew(GURL(url));
     if (!descriptor)
       return nullptr;
     std::unique_ptr<base::DictionaryValue> result(new base::DictionaryValue());
     result->SetInteger("id", id);
     std::unique_ptr<base::DictionaryValue> cmd_result(
         new base::DictionaryValue());
-    cmd_result->SetString("pageId", descriptor->GetId());
+    cmd_result->SetString("targetId", descriptor->GetId());
     result->Set("result", std::move(cmd_result));
     return result;
   }
   return nullptr;
 }
 
 }  // namespace devtools_discovery