Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5762)

Unified Diff: chrome/browser/ui/window_sizer/window_sizer_win.cc

Issue 10836218: Remove BrowserList::GetLastActive calls from window_sizer.cc et al. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/window_sizer/window_sizer_win.cc
diff --git a/chrome/browser/ui/window_sizer/window_sizer_win.cc b/chrome/browser/ui/window_sizer/window_sizer_win.cc
index 2a9b0b2c227ae14fb68e73aa197e6e33638d7e4c..066fcb5dc84313df6d73132c6a9b9f58644315e3 100644
--- a/chrome/browser/ui/window_sizer/window_sizer_win.cc
+++ b/chrome/browser/ui/window_sizer/window_sizer_win.cc
@@ -5,20 +5,22 @@
#include "chrome/browser/ui/window_sizer/window_sizer.h"
#include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/browser_finder.h"
#include "chrome/browser/ui/browser_window.h"
+#include "chrome/browser/ui/host_desktop.h"
// How much horizontal and vertical offset there is between newly
// opened windows.
const int WindowSizer::kWindowTilePixels = 10;
// static
-gfx::Point WindowSizer::GetDefaultPopupOrigin(const gfx::Size& size) {
+gfx::Point WindowSizer::GetDefaultPopupOrigin(const gfx::Size& size,
+ chrome::HostDesktopType type) {
RECT area;
SystemParametersInfo(SPI_GETWORKAREA, 0, &area, 0);
gfx::Point corner(area.left, area.top);
- if (Browser* b = BrowserList::GetLastActive()) {
+ if (Browser* b = browser::FindLastActiveWithHostDesktopType(type)) {
MAD 2012/08/13 21:25:10 b -> browser
robertshield 2012/08/13 21:37:41 Done.
RECT browser;
HWND window = reinterpret_cast<HWND>(b->window()->GetNativeWindow());
if (GetWindowRect(window, &browser)) {

Powered by Google App Engine
This is Rietveld 408576698