Browse Source

Fix rare bug with domComplete never triggered

Gaël Métais 10 years ago
parent
commit
fe7de2db99
1 changed files with 5 additions and 1 deletions
  1. 5 1
      lib/tools/jsExecutionTransformer.js

+ 5 - 1
lib/tools/jsExecutionTransformer.js

@@ -38,7 +38,7 @@ var jsExecutionTransformer = function() {
             javascriptExecutionTree = JSON.parse(data.toolsResults.phantomas.offenders.javascriptExecutionTree[0]);
             javascriptExecutionTree = JSON.parse(data.toolsResults.phantomas.offenders.javascriptExecutionTree[0]);
         
         
             if (javascriptExecutionTree.children) {
             if (javascriptExecutionTree.children) {
-                javascriptExecutionTree.children.forEach(function(node) {
+                javascriptExecutionTree.children.forEach(function(node, index) {
                     
                     
                     var contextLength = (node.data.callDetails && node.data.callDetails.context) ? node.data.callDetails.context.length : null;
                     var contextLength = (node.data.callDetails && node.data.callDetails.context) ? node.data.callDetails.context.length : null;
 
 
@@ -90,6 +90,10 @@ var jsExecutionTransformer = function() {
                                 break;
                                 break;
                         }
                         }
                     }
                     }
+                    // Fix rare bug when domComplete was never triggered
+                    if (index === javascriptExecutionTree.children.length - 1 && !javascriptExecutionTree.data.domComplete) {
+                        javascriptExecutionTree.data.domComplete = node.data.timestamp + 1000;
+                    }
 
 
                     // Transform domPaths into objects
                     // Transform domPaths into objects
                     changeListOfDomPaths(node);
                     changeListOfDomPaths(node);