[Contextual Search] Exclude suppressed taps from aggregate suppression heuristics logging

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

Index: chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchHeuristic.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchHeuristic.java b/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchHeuristic.java
index 0882d3d1082d0584c1cd4b8c59238a2f4bdaaf99..8bed11b96c3a00fddcfc538ae11d3e760066b30c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchHeuristic.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchHeuristic.java
@@ -44,9 +44,9 @@ abstract class ContextualSearchHeuristic {
     }
 
     /**
-     * @return Whether this heuristic's condition would have been satisfied if it were enabled
-     *         through VariationsAssociatedData. When logging aggregate metrics for Tap suppression,
-     *         the condition may be considered satisfied even if the tap wasn't suppresed.
+     * @return Whether this heuristic's condition would have been satisfied, causing a tap
+     *         suppression, if it were enabled through VariationsAssociatedData. This method should
+     *         return false if the feature is enabled through VariationsAssociatedData.

[Donn Denman, 2016/09/23 20:40:13]

Nit: please rephrase this second sentence to better connect with the previous
regarding enabling, e.g. "If the feature is enabled through
VariationsAssociatedData then this method should return false."

[Theresa, 2016/09/23 20:57:30]

Done.

      */
     protected abstract boolean isConditionSatisfiedForAggregateLogging();
 }