From d3ff52b38ae7096cf95b9cce4308f4081a38a662 Mon Sep 17 00:00:00 2001 From: "sky@chromium.org" Date: Tue, 11 Aug 2009 19:14:11 +0000 Subject: Removes bogus comments in WidgetGtk::GetBounds. BUG=none TEST=none Review URL: http://codereview.chromium.org/165306 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23062 0039d316-1c4b-4281-b951-d872f2087c98 --- views/widget/widget_gtk.cc | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) (limited to 'views/widget') diff --git a/views/widget/widget_gtk.cc b/views/widget/widget_gtk.cc index cb6cbfe..8a65022 100644 --- a/views/widget/widget_gtk.cc +++ b/views/widget/widget_gtk.cc @@ -237,22 +237,15 @@ void WidgetGtk::GetBounds(gfx::Rect* out, bool including_frame) const { int x = 0, y = 0, w, h; if (GTK_IS_WINDOW(widget_)) { gtk_window_get_position(GTK_WINDOW(widget_), &x, &y); + // NOTE: this doesn't include frame decorations, but it should be good + // enough for our uses. gtk_window_get_size(GTK_WINDOW(widget_), &w, &h); } else { - // TODO: make sure this is right. Docs indicate gtk_window_get_position - // returns a value useful to the window manager, which may not be the same - // as the actual location on the screen. GetWidgetPositionOnScreen(widget_, &x, &y); w = widget_->allocation.width; h = widget_->allocation.height; } - if (including_frame) { - // TODO: Docs indicate it isn't possible to get at this value. We may need - // to turn off all decorations so that the frame is always of a 0x0 size. - NOTIMPLEMENTED(); - } - return out->SetRect(x, y, w, h); } -- cgit v1.1