Bug 1560667 - Ignore weird destructor cases rather than asserting. r=jonco

This is correct; if it's one of the odd cases, then it is not going to be matching the variable we care about, so it's ok to return false.

Differential Revision: https://phabricator.services.mozilla.com/D46564

--HG--
extra : moz-landing-system : lando

js/src/devtools/rootAnalysis/CFG.js

@@ -125,7 +125,8 @@ function isMatchingConstructor(destructor, edge)
         return false;
 
     var destructExp = destructor.PEdgeCallInstance.Exp;
-    assert(destructExp.Kind == "Var");
+    if (destructExp.Kind != "Var")
+        return false;
 
     var constructExp = edge.PEdgeCallInstance.Exp;
     if (constructExp.Kind != "Var")