Rework the way the FindBrowserWithProfile/Type methods work....

chrome/browser/browser_list.h

 // Copyright (c) 2009 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.
 
 #ifndef CHROME_BROWSER_BROWSER_LIST_H__
 #define CHROME_BROWSER_BROWSER_LIST_H__
 
 #include <vector>
 
 #include "chrome/browser/browser.h"
@@ skipping 40 matching lines @@
   // WARNING: this is NULL until a browser becomes active. If during startup
   // a browser does not become active (perhaps the user launches Chrome, then
   // clicks on another app before the first browser window appears) then this
   // returns NULL.
   static Browser* GetLastActive();
 
   // Identical in behavior to GetLastActive(), except that the most recently
   // open browser owned by |profile| is returned. If none exist, returns NULL.
   static Browser* GetLastActiveWithProfile(Profile *profile);
 
-  // Find an existing browser window with the provided type. If the last active
-  // has the right type, it is returned. Otherwise, the next available browser
-  // is returned. Returns NULL if no such browser currently exists.
+  // Find an existing browser window with the provided type. Searches in the
+  // order of last activation. Only browsers that have been active can be
+  // returned. Returns NULL if no such browser currently exists.
   static Browser* FindBrowserWithType(Profile* p, Browser::Type t);
 
-  // Find an existing browser window with the provided profile. If the last
-  // active has the right profile, it is returned.  Returns NULL if no such
-  // browser currently exists.
+  // Find an existing browser window with the provided profile. Searches in the
+  // order of last activation. Only browsers that have been active can be
+  // returned. Returns NULL if no such browser currently exists.
   static Browser* FindBrowserWithProfile(Profile* p);
 
   // Find an existing browser with the provided ID. Returns NULL if no such
   // browser currently exists.
   static Browser* FindBrowserWithID(SessionID::id_type desired_id);
 
   // Closes all browsers. If use_post is true the windows are closed by way of
   // posting a WM_CLOSE message, otherwise the windows are closed directly. In
   // almost all cases you'll want to use true, the one exception is ending
   // the session. use_post should only be false when invoked from end session.
@@ skipping 112 matching lines @@
   // tab index into the current Browser of the current web view
   int web_view_index_;
 
   // Current TabContents, or NULL if we're at the end of the list. This can
   // be extracted given the browser iterator and index, but it's nice to cache
   // this since the caller may access the current host many times.
   TabContents* cur_;
 };
 
 #endif  // CHROME_BROWSER_BROWSER_LIST_H__