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

Side by Side Diff: chrome/browser/ui/window_sizer_gtk.cc

Issue 9960042: Refactor screen/monitor so that gfx::Screen returns monitor object. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix command line Created 8 years, 8 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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.h" 5 #include "chrome/browser/ui/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_list.h"
12 #include "chrome/browser/ui/browser_window.h" 12 #include "chrome/browser/ui/browser_window.h"
13 #include "ui/gfx/monitor.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) {
23 gfx::Rect monitor_bounds = gfx::Screen::GetPrimaryMonitorWorkArea(); 24 gfx::Rect monitor_bounds = gfx::Screen::GetPrimaryMonitor()->GetWorkArea();
24 gfx::Point corner(monitor_bounds.x(), monitor_bounds.y()); 25 gfx::Point corner(monitor_bounds.x(), monitor_bounds.y());
25 if (Browser* browser = BrowserList::GetLastActive()) { 26 if (Browser* browser = BrowserList::GetLastActive()) {
26 GtkWindow* window = 27 GtkWindow* window =
27 reinterpret_cast<GtkWindow*>(browser->window()->GetNativeHandle()); 28 reinterpret_cast<GtkWindow*>(browser->window()->GetNativeHandle());
28 int x = 0, y = 0; 29 int x = 0, y = 0;
29 gtk_window_get_position(window, &x, &y); 30 gtk_window_get_position(window, &x, &y);
30 // Limit to not overflow the work area right and bottom edges. 31 // Limit to not overflow the work area right and bottom edges.
31 gfx::Point limit( 32 gfx::Point limit(
32 std::min(x + kWindowTilePixels, monitor_bounds.right() - size.width()), 33 std::min(x + kWindowTilePixels, monitor_bounds.right() - size.width()),
33 std::min(y + kWindowTilePixels, 34 std::min(y + kWindowTilePixels,
34 monitor_bounds.bottom() - size.height())); 35 monitor_bounds.bottom() - size.height()));
35 // Adjust corner to now overflow the work area left and top edges, so 36 // 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. 37 // that if a popup does not fit the title-bar is remains visible.
37 corner = gfx::Point( 38 corner = gfx::Point(
38 std::max(corner.x(), limit.x()), 39 std::max(corner.x(), limit.x()),
39 std::max(corner.y(), limit.y())); 40 std::max(corner.y(), limit.y()));
40 } 41 }
41 return corner; 42 return corner;
42 } 43 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698