some fixes and warning disables in ui/ for building on win x64

ui/base/range/range_win.cc

 // Copyright (c) 2011 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 "ui/base/range/range.h"
 
 #include <limits>
 
 #include "base/logging.h"
 
@@ skipping 16 matching lines @@
       set_end(range.cpMax);
     }
   }
 }
 
 CHARRANGE Range::ToCHARRANGE() const {
   CHARRANGE r = { -1, -1 };
   if (!IsValid())
     return r;
 
   const LONG kLONGMax = std::numeric_limits<LONG>::max();

[jschuh, 2013/01/14 21:31:39]

Any reason why we use a LONG here?

[scottmg, 2013/01/14 21:45:00]

cpMin/cpMax are LONG in windows headers.

[jschuh, 2013/01/14 22:14:35]

Sorry I asked.

-  DCHECK_LE(static_cast<LONG>(start()), kLONGMax);
-  DCHECK_LE(static_cast<LONG>(end()), kLONGMax);
-  r.cpMin = start();
-  r.cpMax = end();
+  CHECK_LE(static_cast<LONG>(start()), kLONGMax);
+  CHECK_LE(static_cast<LONG>(end()), kLONGMax);
+  r.cpMin = static_cast<LONG>(start());
+  r.cpMax = static_cast<LONG>(end());
   return r;
 }
 
 }  // namespace gfx