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

Side by Side Diff: chrome/browser/ui/window_sizer/window_sizer_gtk.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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/window_sizer/window_sizer.h" 5 #include "chrome/browser/ui/window_sizer/window_sizer.h"
6 6
7 #include <gtk/gtk.h> 7 #include <gtk/gtk.h>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "chrome/browser/ui/browser.h" 10 #include "chrome/browser/ui/browser.h"
11 #include "chrome/browser/ui/browser_list.h" 11 #include "chrome/browser/ui/browser_finder.h"
12 #include "chrome/browser/ui/browser_window.h" 12 #include "chrome/browser/ui/browser_window.h"
13 #include "chrome/browser/ui/host_desktop.h"
13 #include "ui/gfx/screen.h" 14 #include "ui/gfx/screen.h"
14 15
15 // Used to pad the default new window size. On Windows, this is also used for 16 // Used to pad the default new window size. On Windows, this is also used for
16 // positioning new windows (each window is offset from the previous one). 17 // positioning new windows (each window is offset from the previous one).
17 // Since we don't position windows, it's only used for the default new window 18 // Since we don't position windows, it's only used for the default new window
18 // size. 19 // size.
19 const int WindowSizer::kWindowTilePixels = 10; 20 const int WindowSizer::kWindowTilePixels = 10;
20 21
21 // static 22 // static
22 gfx::Point WindowSizer::GetDefaultPopupOrigin(const gfx::Size& size) { 23 gfx::Point WindowSizer::GetDefaultPopupOrigin(const gfx::Size& size,
24 chrome::HostDesktopType type) {
23 gfx::Rect monitor_bounds = gfx::Screen::GetPrimaryDisplay().work_area(); 25 gfx::Rect monitor_bounds = gfx::Screen::GetPrimaryDisplay().work_area();
24 gfx::Point corner(monitor_bounds.x(), monitor_bounds.y()); 26 gfx::Point corner(monitor_bounds.x(), monitor_bounds.y());
25 if (Browser* browser = BrowserList::GetLastActive()) { 27
28 if (Browser* browser = browser::FindLastActiveWithHostDesktopType(type)) {
26 GtkWindow* window = 29 GtkWindow* window =
27 reinterpret_cast<GtkWindow*>(browser->window()->GetNativeWindow()); 30 reinterpret_cast<GtkWindow*>(browser->window()->GetNativeWindow());
28 int x = 0, y = 0; 31 int x = 0, y = 0;
29 gtk_window_get_position(window, &x, &y); 32 gtk_window_get_position(window, &x, &y);
30 // Limit to not overflow the work area right and bottom edges. 33 // Limit to not overflow the work area right and bottom edges.
31 gfx::Point limit( 34 gfx::Point limit(
32 std::min(x + kWindowTilePixels, monitor_bounds.right() - size.width()), 35 std::min(x + kWindowTilePixels, monitor_bounds.right() - size.width()),
33 std::min(y + kWindowTilePixels, 36 std::min(y + kWindowTilePixels,
34 monitor_bounds.bottom() - size.height())); 37 monitor_bounds.bottom() - size.height()));
35 // Adjust corner to now overflow the work area left and top edges, so 38 // Adjust corner to now overflow the work area left and top edges, so
36 // that if a popup does not fit the title-bar is remains visible. 39 // that if a popup does not fit the title-bar is remains visible.
37 corner = gfx::Point( 40 corner = gfx::Point(
38 std::max(corner.x(), limit.x()), 41 std::max(corner.x(), limit.x()),
39 std::max(corner.y(), limit.y())); 42 std::max(corner.y(), limit.y()));
40 } 43 }
41 return corner; 44 return corner;
42 } 45 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/window_sizer/window_sizer_aura.cc ('k') | chrome/browser/ui/window_sizer/window_sizer_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698