Remove call to IsTryingToQuit from BrowserListImpl::SetLastActive

Remove call to IsTryingToQuit from BrowserListImpl::SetLastActive

Instead the profile manager watches notifications to determine this.

BUG=130456
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150743

[sail, 2012-07-27 16:58:40]

LGTM
CL description doesn't match code

http://codereview.chromium.org/10826044/diff/1/chrome/browser/profiles/profil...
File chrome/browser/profiles/profile_manager.cc (right):

http://codereview.chromium.org/10826044/diff/1/chrome/browser/profiles/profil...
chrome/browser/profiles/profile_manager.cc:642: if
(profile_manager_->closing_all_browsers_)
this needs a comment

[benwells, 2012-07-30 03:56:41]

Description updated / comment added.

sky: care to take a look?

[sky, 2012-08-02 20:47:06]

LGTM, as long as you inspected all overrides of OnBrowserSetLastActive to make
sure there aren't any problems with getting the event at this point.

[benwells, 2012-08-09 03:59:10]

On 2012/08/02 20:47:06, sky wrote:
> LGTM, as long as you inspected all overrides of OnBrowserSetLastActive to make
> sure there aren't any problems with getting the event at this point.

Cool, I have checked the sites and nothing looks worrying. I've been waiting for
the branch point to pass as there is no hurry for this.

[commit-bot: I haz the power, 2012-08-09 03:59:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benwells@chromium.org/10826044/7001

[commit-bot: I haz the power, 2012-08-09 05:22:40]

Change committed as 150743