Make devtools_window.cc multi-desktop aware.

chrome/browser/devtools/browser_list_tabcontents_provider.cc

Index: chrome/browser/devtools/browser_list_tabcontents_provider.cc
diff --git a/chrome/browser/devtools/browser_list_tabcontents_provider.cc b/chrome/browser/devtools/browser_list_tabcontents_provider.cc
index 4e461ce624ea03f37bd3fa32e0d4f9d3a470e998..ced0e5c62f2d0832fc1c897166abbe8a18aaf039 100644
--- a/chrome/browser/devtools/browser_list_tabcontents_provider.cc
+++ b/chrome/browser/devtools/browser_list_tabcontents_provider.cc
@@ -12,7 +12,9 @@
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/browser_iterator.h"
 #include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/browser_list_impl.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
+#include "chrome/browser/ui/host_desktop.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_iterator.h"
 #include "chrome/common/chrome_paths.h"
 #include "content/public/browser/web_contents.h"
@@ -87,8 +89,12 @@ RenderViewHost* BrowserListTabContentsProvider::CreateNewTarget() {
   if (BrowserList::empty())
     return NULL;
 
+  // TODO(gab): Do not hardcode HOST_DESKTOP_TYPE_NATIVE below once
+  // chrome::NewEmptyWindow() above has been made multi-desktop friendly.
+  const chrome::BrowserListImpl* browser_list =
+      chrome::BrowserListImpl::GetInstance(chrome::HOST_DESKTOP_TYPE_NATIVE);
   content::WebContents* web_contents = chrome::AddSelectedTabWithURL(
-      *BrowserList::begin(),
+      browser_list->get(0),

[pfeldman, 2013/02/07 20:12:49]

Will it tolerate 0 in case of empty list?

[gab, 2013/02/07 20:31:45]

This is no different than the previous code which was dereferencing
BrowserList::begin().

       GURL(chrome::kAboutBlankURL),
       content::PAGE_TRANSITION_LINK);
   return web_contents->GetRenderViewHost();