From 67da469cda07377f38d161b4226beda37cc77bee Mon Sep 17 00:00:00 2001
From: Vladimir Krivosheev <vladimir.krivosheev@jetbrains.com>
Date: Wed, 9 Apr 2014 13:13:08 +0200
Subject: [PATCH] jump to source: ability to jump to native V8 script (actual
 only for V8 debugger protocol)

---
 .../intellij/openapi/util/AsyncResult.java    |  2 +-
 .../org/jetbrains/debugger/ScriptManager.java |  5 ++-
 .../jetbrains/debugger/ScriptManagerBase.java |  7 ++++
 .../org/jetbrains/debugger/VariableView.java  | 11 ++++---
 .../jetbrains/jsonProtocol/OutMessage.java    | 32 +++++++++++++++++--
 5 files changed, 49 insertions(+), 8 deletions(-)

diff --git a/platform/core-api/src/com/intellij/openapi/util/AsyncResult.java b/platform/core-api/src/com/intellij/openapi/util/AsyncResult.java
index 097c4e71a535..6bc7d756dc43 100644
--- a/platform/core-api/src/com/intellij/openapi/util/AsyncResult.java
+++ b/platform/core-api/src/com/intellij/openapi/util/AsyncResult.java
@@ -182,7 +182,7 @@ public class AsyncResult<T> extends ActionCallback {
   }
 
   @NotNull
-  public static <R> AsyncResult<R> done(@NotNull R result) {
+  public static <R> AsyncResult<R> done(@Nullable R result) {
     return new AsyncResult<R>().setDone(result);
   }
 
diff --git a/platform/script-debugger/backend/src/org/jetbrains/debugger/ScriptManager.java b/platform/script-debugger/backend/src/org/jetbrains/debugger/ScriptManager.java
index ba2e320cbe17..bc0210bdab58 100644
--- a/platform/script-debugger/backend/src/org/jetbrains/debugger/ScriptManager.java
+++ b/platform/script-debugger/backend/src/org/jetbrains/debugger/ScriptManager.java
@@ -33,7 +33,10 @@ public interface ScriptManager {
   Script forEachScript(@NotNull CommonProcessors.FindProcessor<Script> scriptProcessor);
 
   @Nullable
-  Script getScript(FunctionValue function);
+  Script getScript(@NotNull FunctionValue function);
+
+  @NotNull
+  AsyncResult<Script> getOrLoadScript(@NotNull FunctionValue function);
 
   @Nullable("if call frame script is native (at least in Google Chrome)")
   Script getScript(CallFrame frame);
diff --git a/platform/script-debugger/backend/src/org/jetbrains/debugger/ScriptManagerBase.java b/platform/script-debugger/backend/src/org/jetbrains/debugger/ScriptManagerBase.java
index f5b622babfdc..3b70cd4443b4 100644
--- a/platform/script-debugger/backend/src/org/jetbrains/debugger/ScriptManagerBase.java
+++ b/platform/script-debugger/backend/src/org/jetbrains/debugger/ScriptManagerBase.java
@@ -8,6 +8,7 @@ import com.intellij.util.Processor;
 import com.intellij.util.Url;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
+import org.jetbrains.debugger.values.FunctionValue;
 
 public abstract class ScriptManagerBase<SCRIPT extends ScriptBase> implements ScriptManager {
   @SuppressWarnings("unchecked")
@@ -56,4 +57,10 @@ public abstract class ScriptManagerBase<SCRIPT extends ScriptBase> implements Sc
   public static boolean isSpecial(@NotNull Url url) {
     return !url.isInLocalFileSystem() && (url.getScheme() == null || url.getScheme().equals(ScriptManager.VM_SCHEME) || url.getAuthority() == null);
   }
+
+  @NotNull
+  @Override
+  public AsyncResult<Script> getOrLoadScript(@NotNull FunctionValue function) {
+    return AsyncResult.done(getScript(function));
+  }
 }
\ No newline at end of file
diff --git a/platform/script-debugger/debugger-ui/src/org/jetbrains/debugger/VariableView.java b/platform/script-debugger/debugger-ui/src/org/jetbrains/debugger/VariableView.java
index 95cff932ad8c..07d38eba8d7a 100644
--- a/platform/script-debugger/debugger-ui/src/org/jetbrains/debugger/VariableView.java
+++ b/platform/script-debugger/debugger-ui/src/org/jetbrains/debugger/VariableView.java
@@ -418,10 +418,13 @@ public final class VariableView extends XNamedValue implements VariableContext {
     if (fun != null) {
       fun.doWhenDone(new Consumer<FunctionValue>() {
         @Override
-        public void consume(FunctionValue function) {
-          DebuggerViewSupport debugProcess = getDebugProcess();
-          Script script = debugProcess.getVm().getScriptManager().getScript(function);
-          navigatable.setSourcePosition(script == null ? null : debugProcess.getSourceInfo(null, script, function.getOpenParenLine(), function.getOpenParenColumn()));
+        public void consume(final FunctionValue function) {
+          getDebugProcess().getVm().getScriptManager().getOrLoadScript(function).doWhenDone(new Consumer<Script>() {
+            @Override
+            public void consume(Script script) {
+              navigatable.setSourcePosition(script == null ? null : getDebugProcess().getSourceInfo(null, script, function.getOpenParenLine(), function.getOpenParenColumn()));
+            }
+          });
         }
       });
     }
diff --git a/platform/script-debugger/protocol/protocol-reader-runtime/src/org/jetbrains/jsonProtocol/OutMessage.java b/platform/script-debugger/protocol/protocol-reader-runtime/src/org/jetbrains/jsonProtocol/OutMessage.java
index c0eb315129c1..0e0bd1002eef 100644
--- a/platform/script-debugger/protocol/protocol-reader-runtime/src/org/jetbrains/jsonProtocol/OutMessage.java
+++ b/platform/script-debugger/protocol/protocol-reader-runtime/src/org/jetbrains/jsonProtocol/OutMessage.java
@@ -2,6 +2,9 @@ package org.jetbrains.jsonProtocol;
 
 import com.google.gson.stream.JsonWriter;
 import gnu.trove.TIntArrayList;
+import gnu.trove.TIntHashSet;
+import gnu.trove.TIntProcedure;
+import org.jetbrains.annotations.NotNull;
 
 import java.io.IOException;
 import java.io.StringWriter;
@@ -91,7 +94,7 @@ public abstract class OutMessage {
     }
   }
 
-  protected final void writeIntArray(String name, int[] value) {
+  protected final void writeIntArray(@NotNull String name, @NotNull int[] value) {
     try {
       beginArguments();
       writer.name(name);
@@ -106,7 +109,32 @@ public abstract class OutMessage {
     }
   }
 
-  protected final void writeIntList(String name, TIntArrayList value) {
+  protected final void writeIntSet(@NotNull String name, @NotNull TIntHashSet value) {
+    try {
+      beginArguments();
+      writer.name(name);
+      writer.beginArray();
+      value.forEach(new TIntProcedure() {
+        @Override
+        public boolean execute(int value) {
+          try {
+            writer.value(value);
+          }
+          catch (IOException e) {
+            throw new RuntimeException(e);
+          }
+          return true;
+        }
+      });
+      writer.endArray();
+    }
+    catch (IOException e) {
+      throw new RuntimeException(e);
+    }
+
+  }
+
+  protected final void writeIntList(@NotNull String name, @NotNull TIntArrayList value) {
     try {
       beginArguments();
       writer.name(name);