[debugger] noRefs option for fixing blowing array refs

src/debug-debugger.js

Index: src/debug-debugger.js
diff --git a/src/debug-debugger.js b/src/debug-debugger.js
index 8cbe0b362cf423f7c8c70a94094aace081dd5b8f..ff66b6618c4d1d8091dcd8a40064bd13e7ed390f 100644
--- a/src/debug-debugger.js
+++ b/src/debug-debugger.js
@@ -1349,7 +1349,9 @@ ProtocolMessage.prototype.toJSONProtocol = function() {
       bodyJson = ObjectToProtocolObject_(this.body, serializer);
     }
     json.body = bodyJson;
-    json.refs = serializer.serializeReferencedObjects();
+    if (!this.options_ || !this.options_.noRefs) {

[Peter Rybin, 2011/12/14 12:22:22]

What happens if you use includeRefs options here instead of introducing your
own? Have you tried it?
In your client you set protocol argument "includeRefs": false
Will this work?

[indutny, 2011/12/14 12:33:08]

You mean includeRefs: true, right?

Looks like this works for me, you can close this CL, thanks!

[Peter Rybin, 2011/12/14 18:13:55]

Your change allows user to reject "refs" from the response. I meant that you
could keep it, but try to re-use already existing option called "includeRefs"
instead of introducing your own.

However if the old options already fully works for you it's cool.

+      json.refs = serializer.serializeReferencedObjects();
+    }
   }
   if (this.message) {
     json.message = this.message;
@@ -1395,6 +1397,9 @@ DebugCommandProcessor.prototype.processDebugJSONRequest = function(
         if (args.inlineRefs || args.compactFormat) {
           response.setOption('inlineRefs', true);
         }
+        if (args.noRefs) {
+          response.setOption('noRefs', true);
+        }
         if (!IS_UNDEFINED(args.maxStringLength)) {
           response.setOption('maxStringLength', args.maxStringLength);
         }