Rename ClampToMin and ClampToMax

ui/gfx/point_base.h

 // Copyright (c) 2012 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.
 
 #ifndef UI_GFX_POINT_BASE_H_
 #define UI_GFX_POINT_BASE_H_
 
 #include <string>
 
 #include "base/compiler_specific.h"
@@ skipping 25 matching lines @@
   void operator+=(const VectorClass& vector) {
     x_ += vector.x();
     y_ += vector.y();
   }
 
   void operator-=(const VectorClass& vector) {
     x_ -= vector.x();
     y_ -= vector.y();
   }
 
-  void ClampToMax(const Class& max) {
+  void ClampToUpperBound(const Class& max) {

[sky, 2013/04/22 16:44:18]

I agree the names are confusing, but I'm not sure I like this one better. It's
still confusing. Seems like this should be named SetToMin and the other
SetToMax. The argument should be renamed too.

     x_ = x_ <= max.x_ ? x_ : max.x_;
     y_ = y_ <= max.y_ ? y_ : max.y_;
   }
 
-  void ClampToMin(const Class& min) {
+  void ClampToLowerBound(const Class& min) {
     x_ = x_ >= min.x_ ? x_ : min.x_;
     y_ = y_ >= min.y_ ? y_ : min.y_;
   }
 
   bool IsOrigin() const {
     return x_ == 0 && y_ == 0;
   }
 
   VectorClass OffsetFromOrigin() const {
     return VectorClass(x_, y_);
@@ skipping 16 matching lines @@
   ~PointBase() {}
 
  private:
   Type x_;
   Type y_;
 };
 
 }  // namespace gfx
 
 #endif  // UI_GFX_POINT_BASE_H_