Bug 1479672 - Use template string for compact code, r=me
authorSteve Fink <sfink@mozilla.com>
Tue, 26 Sep 2017 18:19:38 -0700
changeset 500011 a38d3c565e75d226b168deb110b127640d29cb91
parent 500010 de0d7eda05f72b832218f73a06ef5134f5536076
child 500012 47d9c24f99d74b69396b503ba00b36e15dd9ef3c
push id1864
push userffxbld-merge
push dateMon, 03 Dec 2018 15:51:40 +0000
treeherdermozilla-release@f040763d99ad [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs1479672
milestone64.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1479672 - Use template string for compact code, r=me
js/src/devtools/rootAnalysis/computeCallgraph.js
--- a/js/src/devtools/rootAnalysis/computeCallgraph.js
+++ b/js/src/devtools/rootAnalysis/computeCallgraph.js
@@ -116,17 +116,18 @@ function processBody(functionName, body)
                 if (prev_limits & ~limits) {
                     // Only output an edge if it loosens a limit.
                     seen.set(callee.name, prev_limits & limits);
                     printOnce("D " + prologue + memo(callee.name));
                 }
             } else if (callee.kind == 'field') {
                 var { csu, field, isVirtual } = callee;
                 const tag = isVirtual ? 'V' : 'F';
-                printOnce(tag + " " + prologue + memo(`${csu}.${field}`) + " CLASS " + csu + " FIELD " + field);
+                const fullfield = `${csu}.${field}`;
+                printOnce(`${tag} ${prologue}${memo(fullfield)} CLASS ${csu} FIELD ${field}`);
             } else if (callee.kind == 'resolved-field') {
                 // Fully-resolved field (virtual method) call. Record the
                 // callgraph edges. Do not consider limits, since they are
                 // local to this callsite and we are writing out a global
                 // record here.
                 //
                 // Any field call that does *not* have an R entry must be
                 // assumed to call anything.