Merge pull request #13736 from aschackmull/dataflow/remove-superfluous-module-members

C#/Java/Ruby: Remove superfluous module members.

Author: aschackmull

csharp/ql/lib/semmle/code/csharp/dataflow/internal/ContentDataFlow.qll

@@ -107,8 +107,6 @@ module Global<ConfigSig ContentConfig> {
 
     predicate isBarrier = ContentConfig::isBarrier/1;
 
-    predicate isBarrier(DataFlow::Node node, FlowState state) { none() }
-
     DataFlow::FlowFeature getAFeature() { result = ContentConfig::getAFeature() }
 
     // needed to record reads/stores inside summarized callables

csharp/ql/src/utils/modelgenerator/internal/CaptureModels.qll

@@ -160,8 +160,6 @@ module ThroughFlowConfig implements DataFlow::StateConfigSig {
     exists(Type t | t = n.getType() and not isRelevantType(t))
   }
 
-  predicate isBarrier(DataFlow::Node node, FlowState state) { none() }
-
   DataFlow::FlowFeature getAFeature() {
     result instanceof DataFlow::FeatureEqualSourceSinkCallContext
   }

java/ql/lib/semmle/code/java/security/ArbitraryApkInstallationQuery.qll

@@ -68,8 +68,6 @@ private module InstallPackageActionConfig implements DataFlow::StateConfigSig {
   predicate isSink(DataFlow::Node node, FlowState state) {
     state instanceof HasInstallPackageAction and node.asExpr().getType() instanceof TypeIntent
   }
-
-  predicate isBarrier(DataFlow::Node node, FlowState state) { none() }
 }
 
 private module InstallPackageActionFlow =
@@ -113,8 +111,6 @@ private module PackageArchiveMimeTypeConfig implements DataFlow::StateConfigSig
     state instanceof HasPackageArchiveMimeType and
     node instanceof SetDataSink
   }
-
-  predicate isBarrier(DataFlow::Node node, FlowState state) { none() }
 }
 
 private module PackageArchiveMimeTypeFlow =

java/ql/lib/semmle/code/java/security/ImplicitPendingIntentsQuery.qll

@@ -72,8 +72,6 @@ module ImplicitPendingIntentStartConfig implements DataFlow::StateConfigSig {
 
   predicate isBarrier(DataFlow::Node sanitizer) { sanitizer instanceof ExplicitIntentSanitizer }
 
-  predicate isBarrier(DataFlow::Node node, FlowState state) { none() }
-
   predicate isAdditionalFlowStep(DataFlow::Node node1, DataFlow::Node node2) {
     any(ImplicitPendingIntentAdditionalTaintStep c).step(node1, node2)
   }

java/ql/src/Security/CWE/CWE-200/AndroidWebViewSettingsAllowsContentAccess.ql

@@ -97,8 +97,6 @@ module WebViewDisallowContentAccessConfig implements DataFlow::StateConfigSig {
     state instanceof IsSettings and
     node instanceof WebSettingsDisallowContentAccessSink
   }
-
-  predicate isBarrier(DataFlow::Node node, FlowState state) { none() }
 }
 
 module WebViewDisallowContentAccessFlow =

java/ql/src/utils/modelgenerator/internal/CaptureModels.qll

@@ -160,8 +160,6 @@ module ThroughFlowConfig implements DataFlow::StateConfigSig {
     exists(Type t | t = n.getType() and not isRelevantType(t))
   }
 
-  predicate isBarrier(DataFlow::Node node, FlowState state) { none() }
-
   DataFlow::FlowFeature getAFeature() {
     result instanceof DataFlow::FeatureEqualSourceSinkCallContext
   }

ruby/ql/lib/codeql/ruby/security/InsecureDownloadQuery.qll

@@ -45,14 +45,6 @@ module Config implements DataFlow::StateConfigSig {
   }
 
   predicate isBarrier(DataFlow::Node node) { node instanceof Sanitizer }
-
-  predicate isBarrier(DataFlow::Node node, FlowState state) { none() }
-
-  predicate isAdditionalFlowStep(
-    DataFlow::Node node1, FlowState state1, DataFlow::Node node2, FlowState state2
-  ) {
-    none()
-  }
 }
 
 module Flow = DataFlow::GlobalWithState<Config>;