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

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

BUG=82696
TEST=see bug report.

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

[Miranda Callahan, 2011-06-12 16:29:37]

jianli, I see you've done work in this area, and Peter for OWNERship.

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

http://codereview.chromium.org/7134085/diff/4003/chrome/browser/tabs/tab_find...
File chrome/browser/tabs/tab_finder.cc (right):

http://codereview.chromium.org/7134085/diff/4003/chrome/browser/tabs/tab_find...
chrome/browser/tabs/tab_finder.cc:101: if (!(*i)->profile()->IsOffTheRecord()) {
I think it will be simpler to check for the matching profile in this loop.

[Miranda Callahan, 2011-06-13 19:25:21]

On 2011/06/13 18:55:39, jianli wrote:
>
http://codereview.chromium.org/7134085/diff/4003/chrome/browser/tabs/tab_find...
> File chrome/browser/tabs/tab_finder.cc (right):
> 
>
http://codereview.chromium.org/7134085/diff/4003/chrome/browser/tabs/tab_find...
> chrome/browser/tabs/tab_finder.cc:101: if (!(*i)->profile()->IsOffTheRecord())
{
> I think it will be simpler to check for the matching profile in this loop.

Ha, of course -- changed, please take another look. :-)

[jianli, 2011-06-13 19:42:36]

LGTM.

You still need owner's approval on this patch.

[Miranda Callahan, 2011-06-14 12:27:54]

On 2011/06/13 19:42:36, jianli wrote:
> LGTM.
> 
> You still need owner's approval on this patch.

Substituting sky for pkasting as OWNER is now only in chrome/browser/tabs.

[sky, 2011-06-14 15:30:16]

LGTM

[commit-bot: I haz the power, 2011-06-14 16:45:38]

Change committed as 89013