Comment #8 on issue 17498 by [email protected]: new tab in popup
window opens in first chrome window rather than parent
http://code.google.com/p/chromium/issues/detail?id=17498
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=30430
------------------------------------------------------------------------
r30430 | [email protected] | 2009-10-28 20:24:00 -0700 (Wed, 28 Oct 2009) |
10 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser.cc?r1=30430&r2=30429
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser.h?r1=30430&r2=30429
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_list.cc?r1=30430&r2=30429
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_list.h?r1=30430&r2=30429
Rework the way the FindBrowserWithProfile/Type methods work.
We now always walk the last active list backwards rather than consulting
the last active then walking the registered browser list forwards.
This ensures that when the last active browser is a popup or app frame the
last active TYPE_NORMAL browser is located when opening a new tab.
http://crbug.com/17498
TEST=Open an app frame. Open a browser window (Ctrl+N) and load a page.
Minimize it. Open another browser window and minimize it. Activate the app
frame. Press Ctrl+T. The second browser window should be restored and have
a new tab added to it rather than the first.
Review URL: http://codereview.chromium.org/330013
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
