JS: remove duplicate ES2015PromiseDefinition

asger-semmle · asger-semmle · commit 66901dc7b89a · 2019-01-17T12:55:38.000Z
diff --git a/javascript/ql/src/semmle/javascript/Promises.qll b/javascript/ql/src/semmle/javascript/Promises.qll
@@ -4,15 +4,6 @@
 
 import javascript
 
-/**
- * A promise object created by the standard ECMAScript 2015 `Promise` constructor.
- */
-private class ES2015PromiseDefinition extends PromiseDefinition, DataFlow::NewNode {
-  ES2015PromiseDefinition() { this = DataFlow::globalVarRef("Promise").getAnInstantiation() }
-
-  override DataFlow::FunctionNode getExecutor() { result = getCallback(0) }
-}
-
 /**
  * A data flow edge from a promise reaction to the corresponding handler.
  */
diff --git a/javascript/ql/src/semmle/javascript/StandardLibrary.qll b/javascript/ql/src/semmle/javascript/StandardLibrary.qll
@@ -133,7 +133,7 @@ class PromiseCandidate extends DataFlow::InvokeNode {
 /**
  * A promise object created by the standard ECMAScript 2015 `Promise` constructor.
  */
-private class ES2015PromiseDefinition extends PromiseDefinition, DataFlow::NewNode {
+private class ES2015PromiseDefinition extends PromiseDefinition, DataFlow::NewNode {  
   ES2015PromiseDefinition() { this = DataFlow::globalVarRef("Promise").getAnInstantiation() }
 
   override DataFlow::FunctionNode getExecutor() { result = getCallback(0) }
