[DevTools] Move getCollectionEntries to native

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

 // Copyright 2015 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/V8DebuggerAgentImpl.h"
 
 #include "platform/inspector_protocol/String16.h"
 #include "platform/inspector_protocol/Values.h"
 #include "platform/v8_inspector/IgnoreExceptionsScope.h"
 #include "platform/v8_inspector/InjectedScript.h"
@@ skipping 684 matching lines @@
     if (!injectedScript->findObject(errorString, *remoteId, &value))
         return;
     if (!value->IsFunction()) {
         *errorString = "Value with given id is not a function";
         return;
     }
     v8::Local<v8::Function> function = value.As<v8::Function>();
 
     v8::Local<v8::Value> scopesValue;
     v8::Local<v8::Array> scopes;
-    String16 objectGroupName = injectedScript->objectGroupName(*remoteId);
+    String16 groupName = injectedScript->objectGroupName(*remoteId);
     if (m_debugger->functionScopes(function).ToLocal(&scopesValue) && scopesValue->IsArray()) {
         scopes = scopesValue.As<v8::Array>();
-        for (size_t i = 0; i < scopes->Length(); ++i) {
-            v8::Local<v8::Value> scope;
-            if (!scopes->Get(injectedScript->context(), i).ToLocal(&scope) || !scope->IsObject()) {
-                *errorString = "Internal error";
-                return;
-            }
-            if (!injectedScript->wrapObjectProperty(errorString, scope.As<v8::Object>(), toV8String(injectedScript->isolate(), "object"), objectGroupName))
-                return;
-        }
+        if (!injectedScript->wrapObjectsInArray(errorString, scopes, toV8StringInternalized(injectedScript->isolate(), "object"), groupName))
+            return;
     }
 
     OwnPtr<protocol::Debugger::Location> location = protocol::Debugger::Location::create()
         .setScriptId(String16::number(function->ScriptId()))
         .setLineNumber(function->GetScriptLineNumber())
         .setColumnNumber(function->GetScriptColumnNumber()).build();
 
     OwnPtr<FunctionDetails> functionDetails = FunctionDetails::create()
         .setLocation(location.release())
         .setFunctionName(toProtocolStringWithTypeCheck(function->GetDebugName()))
         .setIsGenerator(function->IsGeneratorFunction()).build();
 
     if (!scopes.IsEmpty()) {
         protocol::ErrorSupport errorSupport;
         OwnPtr<protocol::Array<protocol::Debugger::Scope>> scopeChain = protocol::Array<protocol::Debugger::Scope>::parse(toProtocolValue(injectedScript->context(), scopes).get(), &errorSupport);
-        if (errorSupport.hasErrors()) {
-            *errorString = "Internal error";
+        if (V8RuntimeAgentImpl::hasInternalError(errorString, errorSupport.hasErrors()))

[dgozman, 2016/03/21 17:21:02]

Let's have a copy per file.

[kozy, 2016/03/21 19:45:01]

Done.

             return;
-        }
         functionDetails->setScopeChain(scopeChain.release());
     }
 
     *details = functionDetails.release();
 }
 
 void V8DebuggerAgentImpl::getGeneratorObjectDetails(ErrorString* errorString, const String16& objectId, OwnPtr<GeneratorObjectDetails>* outDetails)
 {
     if (!checkEnabled(errorString))
         return;
@@ skipping 12 matching lines @@
     v8::Local<v8::Value> value;
     if (!injectedScript->findObject(errorString, *remoteId, &value))
         return;
     if (!value->IsObject() || !value->ToObject(context).ToLocal(&object)) {
         *errorString = "Value with given id is not an Object";
         return;
     }
 
     v8::Local<v8::Object> detailsObject;
     v8::Local<v8::Value> detailsValue = debugger().generatorObjectDetails(object);
-    if (!detailsValue->IsObject() || !detailsValue->ToObject(context).ToLocal(&detailsObject)) {
-        *errorString = "Internal error";
+    if (V8RuntimeAgentImpl::hasInternalError(errorString, !detailsValue->IsObject() || !detailsValue->ToObject(context).ToLocal(&detailsObject)))
         return;
-    }
 
     String16 groupName = injectedScript->objectGroupName(*remoteId);
-    if (!injectedScript->wrapObjectProperty(errorString, detailsObject, toV8String(m_isolate, "function"), groupName))
+    if (!injectedScript->wrapObjectProperty(errorString, detailsObject, toV8StringInternalized(m_isolate, "function"), groupName))
         return;
 
     protocol::ErrorSupport errors;
     OwnPtr<GeneratorObjectDetails> protocolDetails = GeneratorObjectDetails::parse(toProtocolValue(context, detailsObject).get(), &errors);
-    if (!protocolDetails) {
-        *errorString = "Internal error";
+    if (V8RuntimeAgentImpl::hasInternalError(errorString, !protocolDetails))
         return;
-    }
     *outDetails = protocolDetails.release();
 }
 
-void V8DebuggerAgentImpl::getCollectionEntries(ErrorString* errorString, const String16& objectId, OwnPtr<protocol::Array<CollectionEntry>>* entries)
+void V8DebuggerAgentImpl::getCollectionEntries(ErrorString* errorString, const String16& objectId, OwnPtr<protocol::Array<CollectionEntry>>* outEntries)
 {
     if (!checkEnabled(errorString))
         return;
     OwnPtr<RemoteObjectId> remoteId = RemoteObjectId::parse(errorString, objectId);
     if (!remoteId)
         return;
     InjectedScript* injectedScript = m_injectedScriptManager->findInjectedScript(errorString, remoteId.get());
     if (!injectedScript)
         return;
-    injectedScript->getCollectionEntries(errorString, objectId, entries);
+
+    v8::HandleScope scope(m_isolate);
+    v8::Local<v8::Context> context = injectedScript->context();
+    v8::Context::Scope contextScope(context);
+
+    v8::Local<v8::Value> value;
+    if (!injectedScript->findObject(errorString, *remoteId, &value))
+        return;
+    if (!value->IsObject()) {
+        *errorString = "Object with given id is not a collection";
+        return;
+    }
+    v8::Local<v8::Object> object = value.As<v8::Object>();
+    v8::Local<v8::Value> entriesValue = m_debugger->collectionEntries(object);
+    if (V8RuntimeAgentImpl::hasInternalError(errorString, entriesValue.IsEmpty()))
+        return;
+    if (entriesValue->IsUndefined()) {
+        *errorString = "Object with given id is not a collection";
+        return;
+    }
+    if (V8RuntimeAgentImpl::hasInternalError(errorString, !entriesValue->IsArray()))
+        return;
+    String16 groupName = injectedScript->objectGroupName(*remoteId);
+    v8::Local<v8::Array> entriesArray = entriesValue.As<v8::Array>();
+    if (!injectedScript->wrapObjectsInArray(errorString, entriesArray, toV8StringInternalized(injectedScript->isolate(), "key"), groupName))
+        return;
+    if (!injectedScript->wrapObjectsInArray(errorString, entriesArray, toV8StringInternalized(injectedScript->isolate(), "value"), groupName))
+        return;
+    protocol::ErrorSupport errors;
+    OwnPtr<protocol::Array<CollectionEntry>> entries = protocol::Array<CollectionEntry>::parse(toProtocolValue(injectedScript->context(), entriesArray).get(), &errors);
+    if (V8RuntimeAgentImpl::hasInternalError(errorString, !entries))
+        return;
+    *outEntries = entries.release();
 }
 
 void V8DebuggerAgentImpl::schedulePauseOnNextStatement(const String16& breakReason, PassOwnPtr<protocol::DictionaryValue> data)
 {
     ASSERT(enabled());
     if (m_scheduledDebuggerStep == StepInto || m_javaScriptPauseScheduled || isPaused())
         return;
     m_breakReason = breakReason;
     m_breakAuxData = data;
     m_pausingOnNativeEvent = true;
@@ skipping 816 matching lines @@
         return;
     m_scheduledDebuggerStep = NoStep;
     m_scripts.clear();
     m_blackboxedPositions.clear();
     m_breakpointIdToDebuggerBreakpointIds.clear();
     resetAsyncCallTracker();
     m_promiseTracker->clear();
 }
 
 } // namespace blink