Fix for https://code.google.com/p/dart/issues/detail?id=5851 Proper line #s and columns for Dart er…

Source/bindings/dart/DartUtilities.cpp

 // Copyright 2011, Google Inc.
 // All rights reserved.
 //
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //    * Redistributions of source code must retain the above copyright
 // notice, this list of conditions and the following disclaimer.
 //    * Redistributions in binary form must reproduce the above
@@ skipping 17 matching lines @@
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 #include "config.h"
 #include "bindings/dart/DartUtilities.h"
 
 #include "DartBlob.h"
 #include "DartHTMLElement.h"
 #include "DartIDBKeyRange.h"
 #include "DartImageData.h"
 #include "DartMessagePort.h"
+#include "DartScriptState.h"

[vsm, 2013/10/22 00:32:02]

Use the full include path for non-generated files - this is in bindings/dart,
right?

 #include "bindings/dart/DartDOMData.h"
 #include "bindings/dart/DartHandleProxy.h"
 #include "bindings/dart/V8Converter.h"
 #include "bindings/v8/ScriptController.h"
 #include "bindings/v8/SerializedScriptValue.h"
 #include "bindings/v8/V8Binding.h"
 #include "bindings/v8/V8PerIsolateData.h"
 #include "core/dom/Document.h"
 #include "core/html/canvas/DataView.h"
 #include "core/inspector/ScriptArguments.h"
@@ skipping 840 matching lines @@
         return DartDOMData::current()->scriptExecutionContext();
     return 0;
 }
 
 bool DartUtilities::processingUserGesture()
 {
     // FIXME: implement this.
     return false;
 }
 
+// FIXME(dartbug.com/14291): this should be implemented directly by the DartVM.
+intptr_t DartUtilities::libraryHandleToLibraryId(Dart_Handle library)
+{
+    Dart_Handle libraries = Dart_GetLibraryIds();
+    ASSERT(Dart_IsList(libraries));
+
+    intptr_t librariesLength = 0;
+    Dart_Handle result = Dart_ListLength(libraries, &librariesLength);
+    ASSERT(!Dart_IsError(result));
+    UNUSED_PARAM(result);
+    for (intptr_t i = 0; i < librariesLength; ++i) {
+        Dart_Handle libraryIdHandle = Dart_ListGetAt(libraries, i);
+        ASSERT(!Dart_IsError(libraryIdHandle));
+        Dart_Handle exception = 0;
+        intptr_t libraryId = DartUtilities::toInteger(libraryIdHandle, exception);
+        ASSERT(!exception);
+        if (Dart_IdentityEquals(Dart_GetLibraryFromId(libraryId), library))
+            return libraryId;
+    }
+    ASSERT_NOT_REACHED();
+    return 0;
+}
+
+DartScriptState* DartUtilities::currentScriptState()
+{
+    DartDOMData* dartDOMData = DartDOMData::current();
+    if (!dartDOMData->rootScriptState()) {
+        DartController* controller = DartController::retrieve(dartDOMData->scriptExecutionContext());
+        intptr_t libraryId = DartUtilities::libraryHandleToLibraryId(Dart_RootLibrary());
+        DartScriptState* scriptState = controller->lookupScriptState(Dart_CurrentIsolate(), currentV8Context(), libraryId);
+        dartDOMData->setRootScriptState(scriptState);
+        return scriptState;
+    }
+    return dartDOMData->rootScriptState();
+}
+
 PassRefPtr<ScriptArguments> DartUtilities::createScriptArguments(Dart_Handle argument, Dart_Handle& exception)
 {
     v8::Handle<v8::Value> v8Argument = DartHandleProxy::create(argument);
     Vector<ScriptValue> arguments;
     arguments.append(v8Argument);
-    ScriptState* scriptState = ScriptState::forContext(currentV8Context());
-    return ScriptArguments::create(scriptState, arguments);
+    return ScriptArguments::create(DartUtilities::currentScriptState(), arguments);
+}
+
+static PassRefPtr<ScriptCallStack> createScriptCallStackFromParsedStackTrace(Dart_Handle parsedStackTrace, Dart_Handle exception)
+{
+    Vector<Dart_Handle> callFrames;
+    DartUtilities::extractListElements(parsedStackTrace, exception, callFrames);
+    if (exception)
+        return 0;
+
+    size_t frameCount = callFrames.size();
+    if (frameCount > ScriptCallStack::maxCallStackSizeToCapture)
+        frameCount = ScriptCallStack::maxCallStackSizeToCapture;
+
+    Vector<ScriptCallFrame> scriptCallStackFrames;
+    for (size_t i = 0; i < frameCount; i++) {
+        Dart_Handle callFrame = callFrames[i];
+        Vector<Dart_Handle> fields;
+        DartUtilities::extractListElements(callFrame, exception, fields);
+        if (exception)
+            return 0;
+        ASSERT(fields.size() == 4);
+        String functionName = DartUtilities::toString(fields[0]);
+        String sourceName = DartUtilities::toString(fields[1]);
+        int lineNumber = DartUtilities::toInteger(fields[2], exception);
+        if (exception)
+            return 0;
+        int columnNumber = DartUtilities::toInteger(fields[3], exception);
+        if (exception)
+            return 0;
+        scriptCallStackFrames.append(ScriptCallFrame(functionName, sourceName, lineNumber, columnNumber));
+    }
+    if (!frameCount)
+        scriptCallStackFrames.append(ScriptCallFrame("undefined", "undefined", 0, 0));
+
+    return ScriptCallStack::create(scriptCallStackFrames);
+}
+
+static PassRefPtr<ScriptCallStack> createScriptCallStackFromStackTrace(Dart_Handle stackTrace, Dart_Handle& exception)
+{
+    Dart_Handle parsedStackTrace = DartUtilities::invokeUtilsMethod("parseStackTrace", 1, &stackTrace);
+    if (!DartUtilities::checkResult(parsedStackTrace, exception))
+        return 0;
+
+    return createScriptCallStackFromParsedStackTrace(parsedStackTrace, exception);
 }
 
 PassRefPtr<ScriptCallStack> DartUtilities::createScriptCallStack()
 {
-    // FIXME: wrap current dart call stack as ScriptCallStack.
-    Vector<ScriptCallFrame> wrappedCallFrames;
-    wrappedCallFrames.append(ScriptCallFrame("undefined", "undefined", 0));
-    return ScriptCallStack::create(wrappedCallFrames);
+    Dart_ExceptionPauseInfo previousPauseInfo = Dart_GetExceptionPauseInfo();
+    if (previousPauseInfo != kNoPauseOnExceptions)
+        Dart_SetExceptionPauseInfo(kNoPauseOnExceptions);
+    Dart_Handle exception = 0;
+    Dart_Handle parsedStackTrace = DartUtilities::invokeUtilsMethod("captureParsedStackTrace", 0, 0);
+
+    if (previousPauseInfo != kNoPauseOnExceptions)
+        Dart_SetExceptionPauseInfo(previousPauseInfo);
+
+    if (!DartUtilities::checkResult(parsedStackTrace, exception))
+        return 0;
+
+    return createScriptCallStackFromParsedStackTrace(parsedStackTrace, exception);
 }
 
 Dart_WeakPersistentHandle DartUtilities::createPrologueWeakPersistentHandle(Dart_Handle object, void* peer, Dart_WeakPersistentHandleFinalizer weakCallback)
 {
     DartWeakCallback* callback = new DartWeakCallback(peer, weakCallback);
     callback->m_object = Dart_NewPrologueWeakPersistentHandle(object, callback, &weakCallbackWrapper);
     DartDOMData::current()->weakCallbacks()->add(callback);
     return callback->m_object;
 }
 
@@ skipping 78 matching lines @@
     }
 }
 
 void DartUtilities::reportProblem(ScriptExecutionContext* context, Dart_Handle result)
 {
     // FIXME: provide sourceURL.
     String sourceURL = "FIXME";
     reportProblem(context, result, sourceURL);
 }
 
-static PassRefPtr<ScriptCallStack> createScriptCallStackFromStackTrace(Dart_Handle stackTrace, Dart_Handle& exception) {
-    Dart_Handle parsedStackTrace = DartUtilities::invokeUtilsMethod("parseStackTrace", 1, &stackTrace);
-    if (!DartUtilities::checkResult(parsedStackTrace, exception))
-        return 0;
-
-    Vector<Dart_Handle> callFrames;
-    DartUtilities::extractListElements(parsedStackTrace, exception, callFrames);
-    if (exception)
-        return 0;
-
-    size_t frameCount = callFrames.size();
-    if (frameCount > ScriptCallStack::maxCallStackSizeToCapture)
-        frameCount = ScriptCallStack::maxCallStackSizeToCapture;
-
-    Vector<ScriptCallFrame> scriptCallStackFrames;
-    for (size_t i = 0; i < frameCount; i++) {
-        Dart_Handle callFrame = callFrames[i];
-        Vector<Dart_Handle> fields;
-        DartUtilities::extractListElements(callFrame, exception, fields);
-        if (exception)
-            return 0;
-        ASSERT(fields.size() == 4);
-        String functionName = DartUtilities::toString(fields[0]);
-        String sourceName = DartUtilities::toString(fields[1]);
-        int lineNumber = DartUtilities::toInteger(fields[2], exception);
-        if (exception)
-            return 0;
-        int columnNumber = DartUtilities::toInteger(fields[3], exception);
-        if (exception)
-            return 0;
-        scriptCallStackFrames.append(ScriptCallFrame(functionName, sourceName, lineNumber, columnNumber));
-    }
-    if (!frameCount)
-        scriptCallStackFrames.append(ScriptCallFrame("undefined", "undefined", 0, 0));
-
-    return ScriptCallStack::create(scriptCallStackFrames);
-}
-
 void DartUtilities::reportProblem(ScriptExecutionContext* context, Dart_Handle result, const String& sourceURL)
 {
     ASSERT(Dart_IsError(result));
     ASSERT(!sourceURL.isEmpty());
 
     const String internalErrorPrefix("Internal error: ");
 
     String errorMessage;
     // FIXME: line number info.
     int lineNumber = 0;
@@ skipping 42 matching lines @@
     Dart_PersistentHandle library = domData->htmlLibrary();
     ASSERT(!Dart_IsError(library));
 
     Dart_Handle utilsClass = Dart_GetType(library, Dart_NewStringFromCString("_Utils"), 0, 0);
     ASSERT(!Dart_IsError(utilsClass));
 
     return Dart_Invoke(utilsClass, Dart_NewStringFromCString(methodName), argCount, args);
 }
 
 }