Chromium Code Reviews

Issue 3444011: Use //@ sourceURL when formatting stack trace (Closed)

Created:
10 years, 3 months ago by yurys
Modified:
9 years, 7 months ago
Reviewers:
mnaganov (inactive), pfeldman, Mads Ager (chromium)
CC:
v8-dev
Visibility:
Public.

Description

Use //@ sourceURL when formatting stack trace BUG=672 Committed: http://code.google.com/p/v8/source/detail?r=5498

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Stats (+43 lines, -14 lines)
M src/messages.js View 4 chunks +27 lines, -13 lines 0 comments
M test/mjsunit/stack-traces.js View 3 chunks +16 lines, -1 line 0 comments

Messages

Total messages: 4 (0 generated)
yurys
10 years, 3 months ago (2010-09-20 15:35:57 UTC) #1
mnaganov (inactive)
On 2010/09/20 15:35:57, Yury Semikhatsky wrote: > LGTM
10 years, 3 months ago (2010-09-20 15:43:47 UTC) #2
pfeldman
I don't see it working for the following change: <html> <script> function foo() { function ...
10 years, 3 months ago (2010-09-20 16:34:19 UTC) #3
pfeldman
10 years, 3 months ago (2010-09-20 16:35:02 UTC) #4 
s/following change/following case/

Powered by Google App Engine