Fixes bug in window positioning that resulted in positioning...

chrome/common/win_util.cc

 // Copyright (c) 2006-2008 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/common/win_util.h"
 
 #include <atlbase.h>
 #include <atlapp.h>
 #include <commdlg.h>
 #include <dwmapi.h>
@@ skipping 566 matching lines @@
   // the containing parent.
   if (child_rect->width() > (parent_rect.width() + twice_padding))
     child_rect->set_width(std::max(0, parent_rect.width() - twice_padding));
   if (child_rect->height() > parent_rect.height() + twice_padding)
     child_rect->set_height(std::max(0, parent_rect.height() - twice_padding));
 
   // SECOND, clamp x,y position to padding,padding so we don't position child
   // windows in hyperspace.
   // TODO(mpcomplete): I don't see what the second check in each 'if' does that
   // isn't handled by the LAST set of 'ifs'.  Maybe we can remove it.
-  if (child_rect->x() < 0 || child_rect->x() > parent_rect.right())
-    child_rect->set_x(padding);
-  if (child_rect->y() < 0 || child_rect->y() > parent_rect.bottom())
-    child_rect->set_y(padding);
+  if (child_rect->x() < parent_rect.x() ||
+      child_rect->x() > parent_rect.right()) {
+    child_rect->set_x(parent_rect.x() + padding);
+  }
+  if (child_rect->y() < parent_rect.y() ||
+      child_rect->y() > parent_rect.bottom()) {
+    child_rect->set_y(parent_rect.y() + padding);
+  }
 
   // LAST, nudge the window back up into the client area if its x,y position is
   // within the parent bounds but its width/height place it off-screen.
   if (child_rect->bottom() > parent_rect.bottom())
     child_rect->set_y(parent_rect.bottom() - child_rect->height() - padding);
   if (child_rect->right() > parent_rect.right())
     child_rect->set_x(parent_rect.right() - child_rect->width() - padding);
 }
 
 void SetChildBounds(HWND child_window, HWND parent_window,
@@ skipping 213 matching lines @@
 
 ChromeFont GetWindowTitleFont() {
   NONCLIENTMETRICS ncm;
   win_util::GetNonClientMetrics(&ncm);
   ScopedHFONT caption_font(CreateFontIndirect(&(ncm.lfCaptionFont)));
   return ChromeFont::CreateFont(caption_font);
 }
 
 }  // namespace win_util