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

Unified Diff: chrome/browser/ui/window_sizer/window_sizer_mac.mm

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_mac.mm
diff --git a/chrome/browser/ui/window_sizer/window_sizer_mac.mm b/chrome/browser/ui/window_sizer/window_sizer_mac.mm
index 2587bdc3bf3a01328a0859ebc38479c5f4b78c93..68e3377e806595f40ea80ca482f37cf15bb11353 100644
--- a/chrome/browser/ui/window_sizer/window_sizer_mac.mm
+++ b/chrome/browser/ui/window_sizer/window_sizer_mac.mm
@@ -7,20 +7,22 @@
#import <Cocoa/Cocoa.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 = 22;
// static
-gfx::Point WindowSizer::GetDefaultPopupOrigin(const gfx::Size& size) {
+gfx::Point WindowSizer::GetDefaultPopupOrigin(const gfx::Size& size,
+ chrome::HostDesktopType type) {
NSRect work_area = [[NSScreen mainScreen] visibleFrame];
NSRect main_area = [[[NSScreen screens] objectAtIndex:0] frame];
NSPoint corner = NSMakePoint(NSMinX(work_area), NSMaxY(work_area));
- if (Browser* b = BrowserList::GetLastActive()) {
+ if (Browser* b = browser::FindLastActiveWithHostDesktopType(type)) {
MAD 2012/08/13 21:25:10 I would change b for browser.
robertshield 2012/08/13 21:37:41 Done.
NSWindow* window = b->window()->GetNativeWindow();
NSRect window_frame = [window frame];

Powered by Google App Engine
This is Rietveld 408576698