Index: LayoutTests/http/tests/inspector/debugger-test.js |
diff --git a/LayoutTests/http/tests/inspector/debugger-test.js b/LayoutTests/http/tests/inspector/debugger-test.js |
index f50be12d443acebe220d4598b3e740cc5bab13f2..b5cc3deabf1ecc2f911b9305ce9099a58290a0b7 100644 |
--- a/LayoutTests/http/tests/inspector/debugger-test.js |
+++ b/LayoutTests/http/tests/inspector/debugger-test.js |
@@ -110,7 +110,10 @@ InspectorTest.resumeExecution = function(callback) |
InspectorTest.waitUntilResumed(callback); |
}; |
-InspectorTest.captureStackTrace = function(callFrames, dropLineNumbers) |
+// FIXME: upstream showColumnNumbers field perhaps upstreaming as |
+// dropColumnNumbers instead. JavaScript and Dart both support column numbers |
+// so upstreaming this change wis very reasonable. dartbug.com/13378 |
+InspectorTest.captureStackTrace = function(callFrames, dropLineNumbers, showColumnNumbers) |
{ |
InspectorTest.addResult("Call stack:"); |
for (var i = 0; i < callFrames.length; i++) { |
@@ -118,14 +121,17 @@ InspectorTest.captureStackTrace = function(callFrames, dropLineNumbers) |
var script = WebInspector.debuggerModel.scriptForId(frame.location.scriptId); |
var url; |
var lineNumber; |
+ var columnNumber; |
if (script) { |
url = WebInspector.displayNameForURL(script.sourceURL); |
lineNumber = frame.location.lineNumber + 1; |
+ columnNumber = frame.location.columnNumber + 1; |
} else { |
url = "(internal script)"; |
lineNumber = "(line number)"; |
+ columnNumber = "(column number)"; |
} |
- var s = " " + i + ") " + frame.functionName + " (" + url + (dropLineNumbers ? "" : ":" + lineNumber) + ")"; |
+ var s = " " + i + ") " + frame.functionName + " (" + url + (dropLineNumbers ? "" : ":" + lineNumber + (showColumnNumbers ? ":" + columnNumber : "")) + ")"; |
InspectorTest.addResult(s); |
} |
}; |