Make focus-existing-tab-on-open multi-profile-aware.

chrome/browser/tabs/tab_finder.cc

 // Copyright (c) 2011 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/tabs/tab_finder.h"
 
 #include "base/command_line.h"
 #include "base/stl_util-inl.h"
 #include "chrome/browser/history/history.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 74 matching lines @@
   if (TabMatchesURL(selected_tab, url))
     return NULL;
 
   // See if the current browser has a tab matching the specified url.
   TabContents* tab_in_browser = FindTabInBrowser(browser, url);
   if (tab_in_browser) {
     *existing_browser = browser;
     return tab_in_browser;
   }
 
+  std::vector<Browser*> browsers = BrowserList::GetBrowsersWithProfile(
+      browser->profile());
+
   // Then check other browsers.
-  for (BrowserList::const_iterator i = BrowserList::begin();
-       i != BrowserList::end(); ++i) {
+  for (BrowserList::const_iterator i = browsers.begin();
+       i != browsers.end(); ++i) {
     if (!(*i)->profile()->IsOffTheRecord()) {

[jianli, 2011/06/13 18:55:39]

I think it will be simpler to check for the matching profile in this loop.

       tab_in_browser = FindTabInBrowser(*i, url);
       if (tab_in_browser) {
         *existing_browser = *i;
         return tab_in_browser;
       }
     }
   }
 
   return NULL;
 }
@@ skipping 125 matching lines @@
                                          GURL url,
                                          bool success,
                                          history::RedirectList* redirects) {
   if (success && !redirects->empty()) {
     TabContents* tab_contents =
         callback_consumer_.GetClientDataForCurrentRequest();
     DCHECK(tab_contents);
     tab_contents_to_url_[tab_contents] = redirects->back();
   }
 }