Make WidgetGtk::SetBounds use a single call if it can.

views/widget/widget_gtk.cc

 // 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 "views/widget/widget_gtk.h"
 
 #include "app/gfx/path.h"
 #include "base/compiler_specific.h"
 #include "views/fill_layout.h"
 #include "views/widget/default_theme_provider.h"
@@ skipping 244 matching lines @@
   }
 
   return out->SetRect(x, y, w, h);
 }
 
 void WidgetGtk::SetBounds(const gfx::Rect& bounds) {
   if (type_ == TYPE_CHILD) {
     WidgetGtk* parent_widget = GetViewForNative(gtk_widget_get_parent(widget_));
     parent_widget->PositionChild(widget_, bounds.x(), bounds.y(),
                                  bounds.width(), bounds.height());
+  } else if (GTK_WIDGET_MAPPED(widget_)) {
+    // If the widget is mapped (on screen), we can move and resize with one
+    // call, which avoids two separate window manager steps.
+    gdk_window_move_resize(widget_->window, bounds.x(), bounds.y(),
+                           bounds.width(), bounds.height());
   } else {
     GtkWindow* gtk_window = GTK_WINDOW(widget_);
     // TODO: this may need to set an initial size if not showing.
     // TODO: need to constrain based on screen size.
     gtk_window_resize(gtk_window, bounds.width(), bounds.height());
     gtk_window_move(gtk_window, bounds.x(), bounds.y());
   }
 }
 
 void WidgetGtk::SetShape(const gfx::Path& shape) {
@@ skipping 516 matching lines @@
 void WidgetGtk::HandleGrabBroke() {
   if (has_capture_) {
     if (is_mouse_down_)
       root_view_->ProcessMouseDragCanceled();
     is_mouse_down_ = false;
     has_capture_ = false;
   }
 }
 
 }  // namespace views