Author: [email protected]
Date: Tue Apr 14 18:17:00 2009
New Revision: 1708

Modified:
    branches/bleeding_edge/src/mirror-delay.js

Log:
To resolve functions from the call stack we need at least ids for the  
scripts of those functions. The information on the scripts is now included  
into the response.
Review URL: http://codereview.chromium.org/65006

Modified: branches/bleeding_edge/src/mirror-delay.js
==============================================================================
--- branches/bleeding_edge/src/mirror-delay.js  (original)
+++ branches/bleeding_edge/src/mirror-delay.js  Tue Apr 14 18:17:00 2009
@@ -1715,8 +1715,8 @@
    // Collect the JSON property/value pairs in an array.
    var content = new Array();

-  // Add the handle for value mirrors.
-  if (mirror.isValue()) {
+  // Add the mirror handle.
+  if (mirror.isValue() || mirror.isScript()) {
      content.push(MakeJSONPair_('handle', NumberToJSON_(mirror.handle())));
    }

@@ -1771,10 +1771,11 @@
        break;

      case SCRIPT_TYPE:
-      // Script is represented by name and source attributes.
+      // Script is represented by id, name and source attributes.
        if (mirror.name()) {
          content.push(MakeJSONPair_('name', StringToJSON_(mirror.name())));
        }
+      content.push(MakeJSONPair_('id', NumberToJSON_(mirror.id())));
        content.push(MakeJSONPair_('lineOffset',
                                   NumberToJSON_(mirror.lineOffset())));
        content.push(MakeJSONPair_('columnOffset',
@@ -1908,7 +1909,12 @@
    content.push(MakeJSONPair_('index', NumberToJSON_(mirror.index())));
    content.push(MakeJSONPair_('receiver',
                               this.serializeReference(mirror.receiver())));
-  content.push(MakeJSONPair_('func',  
this.serializeReference(mirror.func())));
+  var func = mirror.func();
+  content.push(MakeJSONPair_('func', this.serializeReference(func)));
+  if (func.script()) {
+    content.push(MakeJSONPair_('script',
+                               this.serializeReference(func.script())));
+  }
    content.push(MakeJSONPair_('constructCall',
                               BooleanToJSON_(mirror.isConstructCall())));
    content.push(MakeJSONPair_('debuggerFrame',

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to