DevTools: [remote debugging] introduce json/new and json/close for creating and closing the tabs.

chrome/browser/debugger/browser_list_tabcontents_provider.cc

 // 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.
 
 #include "chrome/browser/debugger/browser_list_tabcontents_provider.h"
 
 #include "base/path_service.h"
 #include "chrome/browser/history/top_sites.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/browser_tabstrip.h"
+#include "chrome/browser/ui/tab_contents/tab_contents.h"
 #include "chrome/common/chrome_paths.h"
 #include "content/public/browser/browser_thread.h"
+#include "content/public/browser/web_contents.h"
 #include "grit/devtools_discovery_page_resources.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "ui/base/layout.h"
 #include "ui/base/resource/resource_bundle.h"
 
 using content::DevToolsHttpHandlerDelegate;
+using content::RenderViewHost;
 
 BrowserListTabContentsProvider::BrowserListTabContentsProvider() {
 }
 
 BrowserListTabContentsProvider::~BrowserListTabContentsProvider() {
 }
 
 std::string BrowserListTabContentsProvider::GetDiscoveryPageHTML() {
   std::set<Profile*> profiles;
   for (BrowserList::const_iterator it = BrowserList::begin(),
@@ skipping 41 matching lines @@
     if (!top_sites)
       continue;
     scoped_refptr<base::RefCountedMemory> data;
     if (top_sites->GetPageThumbnail(url, &data))
       return std::string(
           reinterpret_cast<const char*>(data->front()), data->size());
   }
 
   return std::string();
 }
+
+RenderViewHost* BrowserListTabContentsProvider::CreateNewTarget() {
+  for (BrowserList::const_iterator it = BrowserList::begin(),
+       end = BrowserList::end(); it != end; ++it) {
+    TabContents* tab_contents = chrome::AddSelectedTabWithURL(

[yurys, 2012/10/05 14:29:49]

Should we specify browser instance where to create the target?

[pfeldman, 2012/10/05 14:36:47]

There is not such term on the content level where the handler is defined.

+        *it,
+        GURL("about:blank"),
+        content::PAGE_TRANSITION_LINK);
+    return tab_contents->web_contents()->GetRenderViewHost();
+  }
+  return NULL;
+}