wm: Update a lot of code to use structs from geometry.h.

transient_window_collection.cc

 // Copyright (c) 2010 The Chromium OS 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 "window_manager/transient_window_collection.h"
 
 #include <list>
 
 #include "window_manager/stacking_manager.h"
 #include "window_manager/window_manager.h"
@@ skipping 87 matching lines @@
   transients_[transient_win->xid()] = transient;
 
   // Info bubbles always keep their initial positions.
   if (transient_win->type() == chromeos::WM_IPC_WINDOW_CHROME_INFO_BUBBLE) {
     transient->SaveOffsetsRelativeToWindow(
         owner_win_,
         Point(transient_win->composited_x(), transient_win->composited_y()));
     transient->centered = false;
   } else {
     transient->UpdateOffsetsToCenterOverWindow(
-        owner_win_,
-        Rect(0, 0, wm()->width(), wm()->height()),
-        constrain_onscreen_);
+        owner_win_, wm()->root_bounds(), constrain_onscreen_);
     transient->centered = true;
   }
 
   // If the new transient is non-modal, stack it above the top non-modal
   // transient that we have.  If it's modal, just put it on top of all
   // other transients.
   TransientWindow* transient_to_stack_above = NULL;
   for (list<TransientWindow*>::const_iterator it =
            stacked_transients_->items().begin();
        it != stacked_transients_->items().end(); ++it) {
@@ skipping 59 matching lines @@
            stacked_transients_->items().rbegin();
        it != stacked_transients_->items().rend();
        ++it) {
     TransientWindow* transient = *it;
     ApplyStackingForTransientWindow(transient, prev_win);
     prev_win = transient->win;
   }
 }
 
 void TransientWindowCollection::HandleConfigureRequest(
-    Window* transient_win,
-    int req_x, int req_y,
-    int req_width, int req_height) {
+    Window* transient_win, const Rect& requested_bounds) {
   CHECK(transient_win);
   TransientWindow* transient = GetTransientWindow(*transient_win);
   CHECK(transient);
 
-  Rect orig_client_bounds = transient_win->client_bounds();
+  const Rect orig_client_bounds = transient_win->client_bounds();
 
   // Move and resize the transient window as requested (only let info bubbles
   // move themselves).
   if (transient_win->type() == chromeos::WM_IPC_WINDOW_CHROME_INFO_BUBBLE) {
-    transient->SaveOffsetsRelativeToWindow(owner_win_, Point(req_x, req_y));
+    transient->SaveOffsetsRelativeToWindow(
+        owner_win_, requested_bounds.position());
     transient->centered = false;
   }
 
-  if (req_width != transient_win->client_width() ||
-      req_height != transient_win->client_height()) {
-    transient_win->ResizeClient(req_width, req_height, GRAVITY_NORTHWEST);
+  if (requested_bounds.size() != transient_win->client_size()) {
+    transient_win->Resize(requested_bounds.size(), GRAVITY_NORTHWEST);
     if (transient->centered) {
       transient->UpdateOffsetsToCenterOverWindow(
-          owner_win_,
-          Rect(0, 0, wm()->width(), wm()->height()),
-          constrain_onscreen_);
+          owner_win_, wm()->root_bounds(), constrain_onscreen_);
     }
   }
 
   ConfigureTransientWindow(transient, 0);
 
   // If the window didn't change, send a fake ConfigureNotify to the
   // client to let it know that we at least considered its request.
   if (transient_win->client_bounds() == orig_client_bounds)
     transient_win->SendSyntheticConfigureNotify();
 }
@@ skipping 132 matching lines @@
   DCHECK(stacked_transients_->Contains(transient));
   DCHECK_GT(stacked_transients_->items().size(), 1U);
   TransientWindow* transient_to_stack_above =
       stacked_transients_->items().front();
   stacked_transients_->Remove(transient);
   stacked_transients_->AddOnTop(transient);
   ApplyStackingForTransientWindow(transient, transient_to_stack_above->win);
 }
 
 }  // namespace window_manager