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

src/inspector/V8Debugger.cpp

 // Copyright 2016 the V8 project 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 "src/inspector/V8Debugger.h"
 
 #include "src/inspector/DebuggerScript.h"
 #include "src/inspector/ScriptBreakpoint.h"
 #include "src/inspector/StringUtil.h"
 #include "src/inspector/V8DebuggerAgentImpl.h"
@@ skipping 360 matching lines @@
       }
       return true;
     }
     // Compile error.
     case 1: {
       *exceptionDetails =
           protocol::Runtime::ExceptionDetails::create()
               .setExceptionId(m_inspector->nextExceptionId())
               .setText(toProtocolStringWithTypeCheck(resultTuple->Get(2)))
               .setLineNumber(
-                  static_cast<int>(
-                      resultTuple->Get(3)->ToInteger(m_isolate)->Value()) -
-                  1)
+                  resultTuple->Get(3)->ToInteger(m_isolate)->Value() - 1)
               .setColumnNumber(
-                  static_cast<int>(
-                      resultTuple->Get(4)->ToInteger(m_isolate)->Value()) -
-                  1)
+                  resultTuple->Get(4)->ToInteger(m_isolate)->Value() - 1)
               .build();
       return false;
     }
   }
   *error = "Unknown error.";
   return false;
 }
 
 JavaScriptCallFrames V8Debugger::currentCallFrames(int limit) {
   if (!m_isolate->InContext()) return JavaScriptCallFrames();
@@ skipping 11 matching lines @@
                                    v8::Integer::New(m_isolate, limit)};
     currentCallFramesV8 =
         callDebuggerMethod("currentCallFrames", V8_INSPECTOR_ARRAY_LENGTH(argv),
                            argv)
             .ToLocalChecked();
   }
   DCHECK(!currentCallFramesV8.IsEmpty());
   if (!currentCallFramesV8->IsArray()) return JavaScriptCallFrames();
   v8::Local<v8::Array> callFramesArray = currentCallFramesV8.As<v8::Array>();
   JavaScriptCallFrames callFrames;
-  for (uint32_t i = 0; i < callFramesArray->Length(); ++i) {
+  for (size_t i = 0; i < callFramesArray->Length(); ++i) {
     v8::Local<v8::Value> callFrameValue;
     if (!callFramesArray->Get(debuggerContext(), i).ToLocal(&callFrameValue))
       return JavaScriptCallFrames();
     if (!callFrameValue->IsObject()) return JavaScriptCallFrames();
     v8::Local<v8::Object> callFrameObject = callFrameValue.As<v8::Object>();
     callFrames.push_back(JavaScriptCallFrame::create(
         debuggerContext(), v8::Local<v8::Object>::Cast(callFrameObject)));
   }
   return callFrames;
 }
@@ skipping 25 matching lines @@
   // Don't allow nested breaks.
   if (m_runningNestedMessageLoop) return;
 
   V8DebuggerAgentImpl* agent =
       m_inspector->enabledDebuggerAgentForGroup(getGroupId(pausedContext));
   if (!agent) return;
 
   std::vector<String16> breakpointIds;
   if (!hitBreakpointNumbers.IsEmpty()) {
     breakpointIds.reserve(hitBreakpointNumbers->Length());
-    for (uint32_t i = 0; i < hitBreakpointNumbers->Length(); i++) {
+    for (size_t i = 0; i < hitBreakpointNumbers->Length(); i++) {
       v8::Local<v8::Value> hitBreakpointNumber = hitBreakpointNumbers->Get(i);
       DCHECK(!hitBreakpointNumber.IsEmpty() && hitBreakpointNumber->IsInt32());
       breakpointIds.push_back(
           String16::fromInteger(hitBreakpointNumber->Int32Value()));
     }
   }
 
   m_pausedContext = pausedContext;
   m_executionState = executionState;
   V8DebuggerAgentImpl::SkipPauseRequest result = agent->didPause(
@@ skipping 100 matching lines @@
 
 void V8Debugger::handleV8AsyncTaskEvent(v8::Local<v8::Context> context,
                                         v8::Local<v8::Object> executionState,
                                         v8::Local<v8::Object> eventData) {
   if (!m_maxAsyncCallStackDepth) return;
 
   String16 type = toProtocolStringWithTypeCheck(
       callInternalGetterFunction(eventData, "type"));
   String16 name = toProtocolStringWithTypeCheck(
       callInternalGetterFunction(eventData, "name"));
-  int id = static_cast<int>(callInternalGetterFunction(eventData, "id")
-                                ->ToInteger(m_isolate)
-                                ->Value());
+  int id = callInternalGetterFunction(eventData, "id")
+               ->ToInteger(m_isolate)
+               ->Value();
   // The scopes for the ids are defined by the eventData.name namespaces. There
   // are currently two namespaces: "Object." and "Promise.".
   void* ptr = reinterpret_cast<void*>(id * 4 + (name[0] == 'P' ? 2 : 0) + 1);
   if (type == v8AsyncTaskEventEnqueue)
     asyncTaskScheduled(name, ptr, false);
   else if (type == v8AsyncTaskEventWillHandle)
     asyncTaskStarted(ptr);
   else if (type == v8AsyncTaskEventDidHandle)
     asyncTaskFinished(ptr);
   else
@@ skipping 312 matching lines @@
 
   size_t stackSize =
       fullStack ? V8StackTraceImpl::maxCallStackSizeToCapture : 1;
   if (m_inspector->enabledRuntimeAgentForGroup(contextGroupId))
     stackSize = V8StackTraceImpl::maxCallStackSizeToCapture;
 
   return V8StackTraceImpl::capture(this, contextGroupId, stackSize);
 }
 
 }  // namespace v8_inspector