Revision: 5754
Author: [email protected]
Date: Tue Nov 2 05:08:58 2010
Log: Remove all remaining code related to exposing C-frames to the
JavaScript debugger
BUG=906
Review URL: http://codereview.chromium.org/4104009
http://code.google.com/p/v8/source/detail?r=5754
Modified:
/branches/bleeding_edge/src/debug-debugger.js
/branches/bleeding_edge/src/runtime.cc
/branches/bleeding_edge/src/runtime.h
=======================================
--- /branches/bleeding_edge/src/debug-debugger.js Wed Oct 27 02:19:43 2010
+++ /branches/bleeding_edge/src/debug-debugger.js Tue Nov 2 05:08:58 2010
@@ -897,10 +897,6 @@
return new FrameMirror(this.break_id, opt_index);
};
-ExecutionState.prototype.cframesValue = function(opt_from_index,
opt_to_index) {
- return %GetCFrames(this.break_id);
-};
-
ExecutionState.prototype.setSelectedFrame = function(index) {
var i = %ToNumber(index);
if (i < 0 || i >= this.frameCount()) throw new Error('Illegal frame
index.');
@@ -1751,11 +1747,6 @@
};
-DebugCommandProcessor.prototype.backtracec = function(cmd, args) {
- return this.exec_state_.cframesValue();
-};
-
-
DebugCommandProcessor.prototype.frameRequest_ = function(request,
response) {
// No frames no source.
if (this.exec_state_.frameCount() == 0) {
@@ -2205,29 +2196,6 @@
return r;
};
-DebugCommandProcessor.prototype.formatCFrames = function(cframes_value) {
- var result = "";
- if (cframes_value == null || cframes_value.length == 0) {
- result += "(stack empty)";
- } else {
- for (var i = 0; i < cframes_value.length; ++i) {
- if (i != 0) result += "\n";
- result += this.formatCFrame(cframes_value[i]);
- }
- }
- return result;
-};
-
-
-DebugCommandProcessor.prototype.formatCFrame = function(cframe_value) {
- var result = "";
- result += "0x" + NumberToHex8Str(cframe_value.address);
- if (!IS_UNDEFINED(cframe_value.text)) {
- result += " " + cframe_value.text;
- }
- return result;
-}
-
/**
* Convert an Object to its debugger protocol representation. The
representation
=======================================
--- /branches/bleeding_edge/src/runtime.cc Wed Oct 27 02:19:43 2010
+++ /branches/bleeding_edge/src/runtime.cc Tue Nov 2 05:08:58 2010
@@ -8889,12 +8889,6 @@
#endif
return Heap::undefined_value();
}
-
-
-static MaybeObject* Runtime_GetCFrames(Arguments args) {
- // See bug 906.
- return Heap::undefined_value();
-}
static MaybeObject* Runtime_GetThreadCount(Arguments args) {
=======================================
--- /branches/bleeding_edge/src/runtime.h Wed Oct 27 02:19:43 2010
+++ /branches/bleeding_edge/src/runtime.h Tue Nov 2 05:08:58 2010
@@ -325,7 +325,6 @@
F(GetScopeCount, 2, 1) \
F(GetScopeDetails, 3, 1) \
F(DebugPrintScopes, 0, 1) \
- F(GetCFrames, 1, 1) \
F(GetThreadCount, 1, 1) \
F(GetThreadDetails, 2, 1) \
F(SetDisableBreak, 1, 1) \
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
