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

Unified Diff: chrome/browser/window_sizer_linux.cc

Issue 5544002: Move a bunch more browser stuff around.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years 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
« no previous file with comments | « chrome/browser/window_sizer.cc ('k') | chrome/browser/window_sizer_mac.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/window_sizer_linux.cc
===================================================================
--- chrome/browser/window_sizer_linux.cc (revision 68008)
+++ chrome/browser/window_sizer_linux.cc (working copy)
@@ -1,131 +0,0 @@
-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/browser/window_sizer.h"
-
-#include <gtk/gtk.h>
-
-#include "base/logging.h"
-#include "chrome/browser/browser_list.h"
-#include "chrome/browser/browser_window.h"
-#include "chrome/browser/ui/browser.h"
-
-// Used to pad the default new window size. On Windows, this is also used for
-// positioning new windows (each window is offset from the previous one).
-// Since we don't position windows, it's only used for the default new window
-// size.
-const int WindowSizer::kWindowTilePixels = 10;
-
-// An implementation of WindowSizer::MonitorInfoProvider that gets the actual
-// monitor information from X via GDK.
-class DefaultMonitorInfoProvider : public WindowSizer::MonitorInfoProvider {
- public:
- DefaultMonitorInfoProvider() { }
-
- virtual gfx::Rect GetPrimaryMonitorWorkArea() const {
- gfx::Rect rect;
- if (GetScreenWorkArea(&rect))
- return rect.Intersect(GetPrimaryMonitorBounds());
-
- // Return the best we've got.
- return GetPrimaryMonitorBounds();
- }
-
- virtual gfx::Rect GetPrimaryMonitorBounds() const {
- GdkScreen* screen = gdk_screen_get_default();
- GdkRectangle rect;
- gdk_screen_get_monitor_geometry(screen, 0, &rect);
- return gfx::Rect(rect);
- }
-
- virtual gfx::Rect GetMonitorWorkAreaMatching(
- const gfx::Rect& match_rect) const {
- // TODO(thestig) Implement multi-monitor support.
- return GetPrimaryMonitorWorkArea();
- }
-
- virtual gfx::Point GetBoundsOffsetMatching(
- const gfx::Rect& match_rect) const {
- // TODO(thestig) Implement multi-monitor support.
- return GetPrimaryMonitorWorkArea().origin();
- }
-
- void UpdateWorkAreas() {
- // TODO(thestig) Implement multi-monitor support.
- work_areas_.clear();
- work_areas_.push_back(GetPrimaryMonitorBounds());
- }
-
- private:
- // Get the available screen space as a gfx::Rect, or return false if
- // if it's unavailable (i.e. the window manager doesn't support
- // retrieving this).
- // TODO(thestig) Use _NET_CURRENT_DESKTOP here as well?
- bool GetScreenWorkArea(gfx::Rect* out_rect) const {
- gboolean ok;
- guchar* raw_data = NULL;
- gint data_len = 0;
- ok = gdk_property_get(gdk_get_default_root_window(), // a gdk window
- gdk_atom_intern("_NET_WORKAREA", FALSE), // property
- gdk_atom_intern("CARDINAL", FALSE), // property type
- 0, // byte offset into property
- 0xff, // property length to retrieve
- false, // delete property after retrieval?
- NULL, // returned property type
- NULL, // returned data format
- &data_len, // returned data len
- &raw_data); // returned data
- if (!ok)
- return false;
-
- // We expect to get four longs back: x, y, width, height.
- if (data_len < static_cast<gint>(4 * sizeof(glong))) {
- NOTREACHED();
- g_free(raw_data);
- return false;
- }
-
- glong* data = reinterpret_cast<glong*>(raw_data);
- gint x = data[0];
- gint y = data[1];
- gint width = data[2];
- gint height = data[3];
- g_free(raw_data);
-
- out_rect->SetRect(x, y, width, height);
- return true;
- }
-
- DISALLOW_COPY_AND_ASSIGN(DefaultMonitorInfoProvider);
-};
-
-// static
-WindowSizer::MonitorInfoProvider*
-WindowSizer::CreateDefaultMonitorInfoProvider() {
- return new DefaultMonitorInfoProvider();
-}
-
-// static
-gfx::Point WindowSizer::GetDefaultPopupOrigin(const gfx::Size& size) {
- scoped_ptr<MonitorInfoProvider> provider(CreateDefaultMonitorInfoProvider());
- gfx::Rect monitor_bounds = provider->GetPrimaryMonitorWorkArea();
- gfx::Point corner(monitor_bounds.x(), monitor_bounds.y());
- if (Browser* browser = BrowserList::GetLastActive()) {
- GtkWindow* window =
- reinterpret_cast<GtkWindow*>(browser->window()->GetNativeHandle());
- int x = 0, y = 0;
- gtk_window_get_position(window, &x, &y);
- // Limit to not overflow the work area right and bottom edges.
- gfx::Point limit(
- std::min(x + kWindowTilePixels, monitor_bounds.right() - size.width()),
- std::min(y + kWindowTilePixels,
- monitor_bounds.bottom() - size.height()));
- // Adjust corner to now overflow the work area left and top edges, so
- // that if a popup does not fit the title-bar is remains visible.
- corner = gfx::Point(
- std::max(corner.x(), limit.x()),
- std::max(corner.y(), limit.y()));
- }
- return corner;
-}
« no previous file with comments | « chrome/browser/window_sizer.cc ('k') | chrome/browser/window_sizer_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698