Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(262)

Issue 6456025: Shorten constructor names in JS tickprocessor. (Closed)

Created:
9 years, 10 months ago by mnaganov (inactive)
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

Shorten constructor names in JS tickprocessor. As they are no more used in DevTools profiler, there is no need to prefix them with "devtools.profiler" namespace. Committed: http://code.google.com/p/v8/source/detail?r=6712

Patch Set 1 #

Patch Set 2 : Also for SplayTree #

Patch Set 3 : and CallTree #

Unified diffs Side-by-side diffs Delta from patch set Stats (+219 lines, -249 lines) Patch
M test/mjsunit/tools/codemap.js View 7 chunks +7 lines, -7 lines 0 comments Download
M test/mjsunit/tools/csvparser.js View 1 chunk +1 line, -1 line 0 comments Download
M test/mjsunit/tools/profile.js View 1 chunk +1 line, -1 line 0 comments Download
M test/mjsunit/tools/profile_view.js View 2 chunks +2 lines, -2 lines 0 comments Download
M test/mjsunit/tools/splaytree.js View 1 9 chunks +9 lines, -9 lines 0 comments Download
M tools/codemap.js View 1 15 chunks +32 lines, -37 lines 0 comments Download
M tools/csvparser.js View 1 3 chunks +6 lines, -11 lines 0 comments Download
M tools/logreader.js View 1 10 chunks +10 lines, -14 lines 0 comments Download
M tools/profile.js View 1 2 43 chunks +74 lines, -79 lines 0 comments Download
M tools/profile_view.js View 1 13 chunks +28 lines, -33 lines 0 comments Download
M tools/splaytree.js View 1 14 chunks +27 lines, -33 lines 0 comments Download
M tools/tickprocessor.js View 8 chunks +22 lines, -22 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
mnaganov (inactive)
9 years, 10 months ago (2011-02-09 15:51:11 UTC) #1
Søren Thygesen Gjesse
9 years, 10 months ago (2011-02-10 07:14:09 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698