Moves methods for finding browsers to browser_finder.

Moves methods for finding browsers to browser_finder.

BUG=none
TEST=none
R=ben@chromium.org
TBR=tim@chromium.org, davemoore@chromium.org, scottbyer@chromium.org, keybuk@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=137511

[sky, 2012-05-16 17:11:40]

The interesting parts are browser/browser_list and browser_finder. The rest are
changes from Browser/BrowserList:: to browser:: and include fixes since
browser_list no longer includes browser.h :

tim: chrome/browser/net/gaia
dave: chrome/browser/chromeos/status
dave: chrome/browser/profiles
scottbyer: chrome/browser/printing
keybuk: chrome/browser/chromeos/dbus
ben: the rest

I'm going to TBR everyone once Ben signs off on it.

  -Scott

[Scott Byer, 2012-05-16 17:24:30]

chrome/browser/printing LGTM

[Ben Goodger (Google), 2012-05-16 17:49:08]

lgtm

[achuithb, 2012-05-16 18:22:15]

http://codereview.chromium.org/10391158/diff/7003/chrome/browser/ui/browser_f...
File chrome/browser/ui/browser_finder.h (right):

http://codereview.chromium.org/10391158/diff/7003/chrome/browser/ui/browser_f...
chrome/browser/ui/browser_finder.h:72: // Identical in behavior to
BrowserList::GetLastActive(), except that the most
nit: update comment

[sky, 2012-05-16 20:54:15]

http://codereview.chromium.org/10391158/diff/7003/chrome/browser/ui/browser_f...
File chrome/browser/ui/browser_finder.h (right):

http://codereview.chromium.org/10391158/diff/7003/chrome/browser/ui/browser_f...
chrome/browser/ui/browser_finder.h:72: // Identical in behavior to
BrowserList::GetLastActive(), except that the most
On 2012/05/16 18:22:16, achuith.bhandarkar wrote:
> nit: update comment

This comment is correct. I didn't move BrowserList::GetLastActive. My rationale
for that is BrowserList has a SetLastActive, so it only seems appropriate to
keep GetLastActive there. OTOH FindLastActiveProfile does more searching, so I
move it.