Support custom border widths. Allow user to resize the window (instead of do...

chrome/views/non_client_view.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/views/non_client_view.h"
 
 namespace views {
 
 const int NonClientView::kClientEdgeThickness = 1;
 
 int NonClientView::GetHTComponentForFrame(const gfx::Point& point,
                                           int top_resize_border_height,
                                           int resize_border_thickness,
                                           int resize_corner_size,
                                           bool can_resize) {
-  int component = HTNOWHERE;
+  // Tricky: In XP, native behavior is to return HTTOPLEFT and HTTOPRIGHT for
+  // a |resize_corner_size|-length strip of both the side and top borders, but
+  // only to return HTBOTTOMLEFT/HTBOTTOMRIGHT along the bottom border + corner
+  // (not the side border).  Vista goes further and doesn't return these on any
+  // of the side borders.  Here we copy the XP behavior.
+  int component;
   if (point.x() < resize_border_thickness) {
-    if (point.y() < resize_corner_size) {
+    if (point.y() < resize_corner_size)
       component = HTTOPLEFT;
-    } else if (point.y() >= (height() - resize_corner_size)) {
+    else if (point.y() >= (height() - resize_border_thickness))
       component = HTBOTTOMLEFT;
-    } else {
+    else
       component = HTLEFT;
-    }
-  } else if (point.x() < resize_corner_size) {
-    if (point.y() < top_resize_border_height) {
+  } else if (point.x() >= (width() - resize_border_thickness)) {
+    if (point.y() < resize_corner_size)
+      component = HTTOPRIGHT;
+    else if (point.y() >= (height() - resize_border_thickness))
+      component = HTBOTTOMRIGHT;
+    else
+      component = HTRIGHT;
+  } else if (point.y() < top_resize_border_height) {
+    if (point.x() < resize_corner_size)
       component = HTTOPLEFT;
-    } else if (point.y() >= (height() - resize_border_thickness)) {
+    else if (point.x() >= (width() - resize_corner_size))
+      component = HTTOPRIGHT;
+    else
+      component = HTTOP;
+  } else if (point.y() >= (height() - resize_border_thickness)) {
+    if (point.x() < resize_corner_size)
       component = HTBOTTOMLEFT;
-    }
-  } else if (point.x() >= (width() - resize_border_thickness)) {
-    if (point.y() < resize_corner_size) {
-      component = HTTOPRIGHT;
-    } else if (point.y() >= (height() - resize_corner_size)) {
+    else if (point.x() >= (width() - resize_corner_size))
       component = HTBOTTOMRIGHT;
-    } else {
-      component = HTRIGHT;
-    }
-  } else if (point.x() >= (width() - resize_corner_size)) {
-    if (point.y() < top_resize_border_height) {
-      component = HTTOPRIGHT;
-    } else if (point.y() >= (height() - resize_border_thickness)) {
-      component = HTBOTTOMRIGHT;
-    }
-  } else if (point.y() < top_resize_border_height) {
-    component = HTTOP;
-  } else if (point.y() >= (height() - resize_border_thickness)) {
-    component = HTBOTTOM;
+    else
+      component = HTBOTTOM;
+  } else {
+    return HTNOWHERE;
   }
 
   // If the window can't be resized, there are no resize boundaries, just
   // window borders.
-  if (component != HTNOWHERE)
-    return can_resize ? component : HTBORDER;
-  return HTNOWHERE;
+  return can_resize ? component : HTBORDER;
 }
 
 }  // namespace views