[Contextual Search] Do not include tap suppression in aggregate heuristic logging

chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/NearTopTapSuppression.java

Index: chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/NearTopTapSuppression.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/NearTopTapSuppression.java b/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/NearTopTapSuppression.java
index 8606ba4928d9cd06d45bf0cd0e2a5d9da902362c..39bf83f7362fd71d3b7be2698bf430aa7d93c779 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/NearTopTapSuppression.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/NearTopTapSuppression.java
@@ -46,10 +46,6 @@ public class NearTopTapSuppression extends ContextualSearchHeuristic {
         }
     }
 
-    @Override
-    protected boolean isConditionSatisfiedForAggregateLogging() {
-        // TODO(twellington): Define a default value to use when determining if the condition is
-        // satisfied for logging.
-        return false;
-    }
+    // TODO(twellington): Define a default value to use when determining if the condition is

[Donn Denman, 2016/10/04 20:55:29]

Let's just do this so we can get rid of the TODO?

I think this would work:
protected boolean isConditionSatisfiedForAggregateLogging() {
  return (mIsConditionSatisfied && mExperiementThresholdDps == 0); 
}

[Theresa, 2016/10/04 21:10:37]

If mExperimentThreshold == 0 then mIsConditionSatisfied will always be false. I
think we need to do some analysis on the histograms we're collecting for near
top tap suppression to figure out a good default value. This is something simple
we may be able to merge back after we analyze.

+    // satisfied for logging.
 }