[inspector] fixed all shorten-64-to-32 warnings

src/inspector/String16.cpp

Index: src/inspector/String16.cpp
diff --git a/src/inspector/String16.cpp b/src/inspector/String16.cpp
index e14d19eee7440a8e763b212a6c7ea19142ed63db..746c67bb6925f264d692e4f7e9465c815f4a070b 100644
--- a/src/inspector/String16.cpp
+++ b/src/inspector/String16.cpp
@@ -38,9 +38,9 @@ int charactersToInteger(const UChar* characters, size_t length,
   buffer.push_back('\0');
 
   char* endptr;
-  int result = std::strtol(buffer.data(), &endptr, 10);
-  if (ok) *ok = !(*endptr);
-  return result;
+  long result = std::strtol(buffer.data(), &endptr, 10);
+  if (ok) *ok = !(*endptr) && result <= std::numeric_limits<int>::max();

[dgozman, 2016/09/12 22:44:25]

Also check for >= ::min().

[kozy, 2016/09/12 23:02:59]

Done.

+  return static_cast<int>(result);
 }
 
 const UChar replacementCharacter = 0xFFFD;
@@ -249,7 +249,7 @@ static const UChar32 offsetsFromUTF8[6] = {0x00000000UL,
                                            static_cast<UChar32>(0xFA082080UL),
                                            static_cast<UChar32>(0x82082080UL)};
 
-static inline UChar32 readUTF8Sequence(const char*& sequence, unsigned length) {
+static inline UChar32 readUTF8Sequence(const char*& sequence, size_t length) {
   UChar32 character = 0;
 
   // The cases all fall through.
@@ -367,6 +367,14 @@ String16 String16::fromInteger(int number) {
 }
 
 // static
+String16 String16::fromInteger(size_t number) {
+  const size_t kBufferSize = 50;
+  char buffer[kBufferSize];
+  std::snprintf(buffer, kBufferSize, "%zu", number);
+  return String16(buffer);
+}
+
+// static
 String16 String16::fromDouble(double number) {
   const size_t kBufferSize = 100;
   char buffer[kBufferSize];
@@ -397,8 +405,8 @@ int String16::toInteger(bool* ok) const {
 String16 String16::stripWhiteSpace() const {
   if (!length()) return String16();
 
-  unsigned start = 0;
-  unsigned end = length() - 1;
+  size_t start = 0;
+  size_t end = length() - 1;
 
   // skip white space from start
   while (start <= end && isSpaceOrNewLine(characters16()[start])) ++start;
@@ -456,12 +464,12 @@ String16 String16::fromUTF8(const char* stringStart, size_t length) {
                          true) != conversionOK)
     return String16();
 
-  unsigned utf16Length = bufferCurrent - bufferStart;
+  size_t utf16Length = bufferCurrent - bufferStart;
   return String16(bufferStart, utf16Length);
 }
 
 std::string String16::utf8() const {
-  unsigned length = this->length();
+  size_t length = this->length();
 
   if (!length) return std::string("");