Backed out changeset a76a4f921987 (bug 964700) for XPC Bustage on a CLOSED TREE

browser/devtools/shared/Parser.jsm

@@ -85,7 +85,7 @@ Parser.prototype = {
       }
     }
 
-    let pool = new SyntaxTreesPool(syntaxTrees, aUrl);
+    let pool = new SyntaxTreesPool(syntaxTrees);
 
     // Cache the syntax trees pool by the specified url. This is entirely
     // optional, but it's strongly encouraged to cache ASTs because
@@ -123,12 +123,9 @@ Parser.prototype = {
  *
  * @param object aSyntaxTrees
  *        A collection of AST nodes generated for a source.
- * @param string aUrl [optional]
- *        The source url.
  */
-function SyntaxTreesPool(aSyntaxTrees, aUrl = "<unknown>") {
+function SyntaxTreesPool(aSyntaxTrees) {
   this._trees = aSyntaxTrees;
-  this._url = aUrl;
   this._cache = new Map();
 }
 
@@ -218,7 +215,7 @@ SyntaxTreesPool.prototype = {
         // Can't guarantee that the tree traversal logic is forever perfect :)
         // Language features may be added, in which case the recursive methods
         // need to be updated. If an exception is thrown here, file a bug.
-        DevToolsUtils.reportException("Syntax tree visitor for " + aUrl, e);
+        DevToolsUtils.reportException("syntax tree", e);
       }
     }
     this._cache.set(requestId, results);

toolkit/devtools/DevToolsUtils.js

@@ -29,7 +29,7 @@ this.safeErrorString = function safeErrorString(aError) {
       } catch (ee) { }
 
       if (typeof aError.lineNumber == "number" && typeof aError.columnNumber == "number") {
-        errorString += "Line: " + aError.lineNumber + ", column: " + aError.columnNumber;
+        errorString += ", line: " + aError.lineNumber + ", column: " + aError.columnNumber;
       }
 
       return errorString;