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

Unified Diff: tools/linux-tick-processor

Issue 1179173009: Add support for running the profiler output processing scripts with node. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Passed a missing type arg through to CodeEntry Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/csvparser.js ('k') | tools/logreader.js » ('j') | tools/profile.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/linux-tick-processor
diff --git a/tools/linux-tick-processor b/tools/linux-tick-processor
index 7864fb405780569f136732f642b42c34d5e43bba..5695e79fe76e9143817f5510798cc2a3950c4425 100755
--- a/tools/linux-tick-processor
+++ b/tools/linux-tick-processor
@@ -27,15 +27,19 @@ if [ ! -x "$d8_exec" ]; then
fi
if [ ! -x "$d8_exec" ]; then
- echo "d8 shell not found in $D8_PATH"
- echo "To build, execute 'make native' from the V8 directory"
- exit 1
+ if [! -x "node"]; then
+ echo "d8 shell not found in $D8_PATH"
+ echo "To build, execute 'make native' from the V8 directory"
+ exit 1
+ else
+ node $tools_path/tickprocessor-driver.js $@
+ fi
+else
+ # nm spits out 'no symbols found' messages to stderr.
+ cat $log_file | $d8_exec $tools_path/splaytree.js $tools_path/codemap.js \
+ $tools_path/csvparser.js $tools_path/consarray.js \
+ $tools_path/profile.js $tools_path/profile_view.js \
+ $tools_path/logreader.js $tools_path/tickprocessor.js \
+ $tools_path/SourceMap.js \
+ $tools_path/tickprocessor-driver.js -- $@ 2>/dev/null
fi
-
-# nm spits out 'no symbols found' messages to stderr.
-cat $log_file | $d8_exec $tools_path/splaytree.js $tools_path/codemap.js \
- $tools_path/csvparser.js $tools_path/consarray.js \
- $tools_path/profile.js $tools_path/profile_view.js \
- $tools_path/logreader.js $tools_path/tickprocessor.js \
- $tools_path/SourceMap.js \
- $tools_path/tickprocessor-driver.js -- $@ 2>/dev/null
« no previous file with comments | « tools/csvparser.js ('k') | tools/logreader.js » ('j') | tools/profile.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698