HexStringToUInt64 should fail for negative input

base/strings/string_number_conversions_unittest.cc

 // 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.
 
 #include <math.h>
 
 #include <limits>
 
 #include "base/strings/string_number_conversions.h"
 #include "base/utf_string_conversions.h"
@@ skipping 299 matching lines @@
 }
 
 TEST(StringNumberConversionsTest, HexStringToUInt64) {
   static const struct {
     std::string input;
     uint64 output;
     bool success;
   } cases[] = {
     {"0", 0, true},
     {"42", 66, true},
-    {"-42", static_cast<uint64>(-66), true},
+    {"-42", 0, false},
     {"+42", 66, true},
     {"40acd88557b", GG_INT64_C(4444444448123), true},
     {"7fffffff", INT_MAX, true},
-    {"-80000000", static_cast<uint64>(INT_MIN), true},
+    {"-80000000", 0, false},
     {"ffffffff", 0xffffffff, true},
     {"DeadBeef", 0xdeadbeef, true},
     {"0x42", 66, true},
-    {"-0x42", static_cast<uint64>(-66), true},
+    {"-0x42", 0, false},
     {"+0x42", 66, true},
     {"0x40acd88557b", GG_INT64_C(4444444448123), true},
     {"0x7fffffff", INT_MAX, true},
-    {"-0x80000000", static_cast<uint64>(INT_MIN), true},
+    {"-0x80000000", 0, false},
     {"0xffffffff", 0xffffffff, true},
     {"0XDeadBeef", 0xdeadbeef, true},
     {"0x7fffffffffffffff", kint64max, true},
-    {"-0x8000000000000000", GG_UINT64_C(0x8000000000000000), true},
+    {"-0x8000000000000000", 0, false},
     {"0x8000000000000000", GG_UINT64_C(0x8000000000000000), true},
-    {"-0x8000000000000001", GG_UINT64_C(0x7fffffffffffffff), true},
+    {"-0x8000000000000001", 0, false},
     {"0xFFFFFFFFFFFFFFFF", kuint64max, true},
     {"FFFFFFFFFFFFFFFF", kuint64max, true},
     {"0x0000000000000000", 0, true},
     {"0000000000000000", 0, true},
     {"1FFFFFFFFFFFFFFFF", kuint64max, false}, // Overflow test.
     {"0x0f", 15, true},
     {"0f", 15, true},
     {" 45", 0x45, false},
     {"\t\n\v\f\r 0x45", 0x45, false},
     {" 45", 0x45, false},
@@ skipping 146 matching lines @@
 
 TEST(StringNumberConversionsTest, HexEncode) {
   std::string hex(HexEncode(NULL, 0));
   EXPECT_EQ(hex.length(), 0U);
   unsigned char bytes[] = {0x01, 0xff, 0x02, 0xfe, 0x03, 0x80, 0x81};
   hex = HexEncode(bytes, sizeof(bytes));
   EXPECT_EQ(hex.compare("01FF02FE038081"), 0);
 }
 
 }  // namespace base