[DevTools] don't truncate [[Entries]] from Runtime.getProperties

third_party/WebKit/Source/platform/v8_inspector/V8Debugger.cpp

 // Copyright 2016 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 "platform/v8_inspector/V8Debugger.h"
 
 #include "platform/v8_inspector/DebuggerScript.h"
 #include "platform/v8_inspector/ScriptBreakpoint.h"
 #include "platform/v8_inspector/V8Compat.h"
 #include "platform/v8_inspector/V8DebuggerAgentImpl.h"
@@ skipping 637 matching lines @@
 }
 
 v8::Local<v8::Value> V8Debugger::collectionEntries(v8::Local<v8::Context> context, v8::Local<v8::Object> object)
 {
     if (!enabled()) {
         NOTREACHED();
         return v8::Undefined(m_isolate);
     }
     v8::Local<v8::Value> argv[] = { object };
     v8::Local<v8::Value> entriesValue = callDebuggerMethod("getCollectionEntries", 1, argv).ToLocalChecked();
-    v8::Local<v8::Value> copied;
-    if (!copyValueFromDebuggerContext(m_isolate, debuggerContext(), context, entriesValue).ToLocal(&copied) || !copied->IsArray())
+    if (!entriesValue->IsArray())
         return v8::Undefined(m_isolate);
-    if (!markArrayEntriesAsInternal(context, v8::Local<v8::Array>::Cast(copied), V8InternalValueType::kEntry))
+    v8::Local<v8::Array> entries = entriesValue.As<v8::Array>();
+    v8::Local<v8::Array> copiedArray = v8::Array::New(m_isolate, entries->Length());
+    if (!copiedArray->SetPrototype(context, v8::Null(m_isolate)).FromMaybe(false))
         return v8::Undefined(m_isolate);
-    return copied;
+    for (uint32_t i = 0; i < entries->Length(); ++i) {
+        v8::Local<v8::Value> item;
+        if (!entries->Get(debuggerContext(), i).ToLocal(&item))
+            return v8::Undefined(m_isolate);
+        v8::Local<v8::Value> copied;
+        if (!copyValueFromDebuggerContext(m_isolate, debuggerContext(), context, item).ToLocal(&copied))
+            return v8::Undefined(m_isolate);
+        if (!createDataProperty(context, copiedArray, i, copied).FromMaybe(false))
+            return v8::Undefined(m_isolate);
+    }
+    if (!markArrayEntriesAsInternal(context, v8::Local<v8::Array>::Cast(copiedArray), V8InternalValueType::kEntry))
+        return v8::Undefined(m_isolate);
+    return copiedArray;
 }
 
 v8::Local<v8::Value> V8Debugger::generatorObjectLocation(v8::Local<v8::Context> context, v8::Local<v8::Object> object)
 {
     if (!enabled()) {
         NOTREACHED();
         return v8::Null(m_isolate);
     }
     v8::Local<v8::Value> argv[] = { object };
     v8::Local<v8::Value> location = callDebuggerMethod("getGeneratorObjectLocation", 1, argv).ToLocalChecked();
@@ skipping 156 matching lines @@
         return nullptr;
 
     size_t stackSize = fullStack ? V8StackTraceImpl::maxCallStackSizeToCapture : 1;
     if (m_inspector->enabledRuntimeAgentForGroup(contextGroupId))
         stackSize = V8StackTraceImpl::maxCallStackSizeToCapture;
 
     return V8StackTraceImpl::capture(this, contextGroupId, stackSize);
 }
 
 } // namespace v8_inspector