Stop showing notifications if they're ignored

chrome/browser/android/ntp/content_suggestions_notification_helper.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/ntp/content_suggestions_notification_helper.h"
 
 #include <limits>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
+#include "base/metrics/histogram_macros.h"
 #include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/ntp_snippets/ntp_snippets_features.h"
+#include "chrome/browser/ntp_snippets/ntp_snippets_metrics.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/common/pref_names.h"
+#include "components/prefs/pref_service.h"
+#include "components/variations/variations_associated_data.h"
 #include "jni/ContentSuggestionsNotificationHelper_jni.h"
 #include "ui/gfx/android/java_bitmap.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/image/image_skia.h"
 
+using base::android::JavaParamRef;
+
 namespace ntp_snippets {
 
-void ContentSuggestionsNotificationHelper::OpenURL(const GURL& url) {
-  JNIEnv* env = base::android::AttachCurrentThread();
-  Java_ContentSuggestionsNotificationHelper_openUrl(
-      env, base::android::ConvertUTF8ToJavaString(env, url.spec()));
+namespace {
+
+bool IsDisabledForProfile(Profile* profile) {
+  PrefService* prefs = profile->GetPrefs();
+  int current =
+      prefs->GetInteger(prefs::kContentSuggestionsConsecutiveIgnoredPrefName);
+  int limit = variations::GetVariationParamByFeatureAsInt(
+      kContentSuggestionsNotificationsFeature,
+      kContentSuggestionsNotificationsIgnoredLimitParam,
+      kContentSuggestionsNotificationsIgnoredDefaultLimit);
+  return current >= limit;
 }
 
+}  // namespace
+
 void ContentSuggestionsNotificationHelper::SendNotification(
     const GURL& url,
     const base::string16& title,
     const base::string16& text,
     const gfx::Image& image,
     base::Time timeout_at) {
   JNIEnv* env = base::android::AttachCurrentThread();
   SkBitmap skimage = image.AsImageSkia().GetRepresentation(1.0f).sk_bitmap();
   if (skimage.empty())
     return;
 
   jint timeout_at_millis = timeout_at.ToJavaTime();
   if (timeout_at == base::Time::Max()) {
     timeout_at_millis = std::numeric_limits<jint>::max();
   }
 
   Java_ContentSuggestionsNotificationHelper_showNotification(
       env, base::android::ConvertUTF8ToJavaString(env, url.spec()),
       base::android::ConvertUTF16ToJavaString(env, title),
       base::android::ConvertUTF16ToJavaString(env, text),
       gfx::ConvertToJavaBitmap(&skimage), timeout_at_millis);
 }
 
 void ContentSuggestionsNotificationHelper::HideAllNotifications() {
   JNIEnv* env = base::android::AttachCurrentThread();
   Java_ContentSuggestionsNotificationHelper_hideAllNotifications(env);
 }
 
+bool ContentSuggestionsNotificationHelper::IsDisabledForProfile(
+    Profile* profile) {
+  return ntp_snippets::IsDisabledForProfile(profile);
+}
+
+// static
+bool ContentSuggestionsNotificationHelper::Register(JNIEnv* env) {
+  return RegisterNativesImpl(env);
+}
+
+static void NotificationTapped(JNIEnv* env,
+                               const JavaParamRef<jclass> /* class_object */&) {
+  Profile* profile = ProfileManager::GetLastUsedProfile()->GetOriginalProfile();
+  PrefService* prefs = profile->GetPrefs();
+  prefs->SetInteger(prefs::kContentSuggestionsConsecutiveIgnoredPrefName, 0);
+  RecordContentSuggestionsNotificationAction(CONTENT_SUGGESTIONS_TAP);
+}
+
+static void NotificationDismissed(
+    JNIEnv* env,
+    const JavaParamRef<jclass> /* class_object */&) {

[Bernhard Bauer, 2017/01/12 12:13:52]

Put the ampersand before the variable name?

+  Profile* profile = ProfileManager::GetLastUsedProfile()->GetOriginalProfile();
+  PrefService* prefs = profile->GetPrefs();
+  const bool was_disabled = IsDisabledForProfile(profile);
+  int ignored =
+      prefs->GetInteger(prefs::kContentSuggestionsConsecutiveIgnoredPrefName);
+  prefs->SetInteger(prefs::kContentSuggestionsConsecutiveIgnoredPrefName,
+                    ignored + 1);
+  RecordContentSuggestionsNotificationAction(CONTENT_SUGGESTIONS_DISMISSAL);
+  const bool is_disabled = IsDisabledForProfile(profile);
+  if (!was_disabled && is_disabled) {
+    RecordContentSuggestionsNotificationOptOut(CONTENT_SUGGESTIONS_IMPLICIT);
+  }
+}
+
+static void NotificationExpired(
+    JNIEnv* env,
+    const JavaParamRef<jclass>& /* class_object */) {
+  Profile* profile = ProfileManager::GetLastUsedProfile()->GetOriginalProfile();
+  PrefService* prefs = profile->GetPrefs();
+  const bool was_disabled = IsDisabledForProfile(profile);
+  int ignored =
+      prefs->GetInteger(prefs::kContentSuggestionsConsecutiveIgnoredPrefName);
+  prefs->SetInteger(prefs::kContentSuggestionsConsecutiveIgnoredPrefName,
+                    ignored + 1);
+  RecordContentSuggestionsNotificationAction(CONTENT_SUGGESTIONS_HIDE_DEADLINE);
+  const bool is_disabled = IsDisabledForProfile(profile);
+  if (!was_disabled && is_disabled) {
+    RecordContentSuggestionsNotificationOptOut(CONTENT_SUGGESTIONS_IMPLICIT);
+  }
+}
+
 }  // namespace ntp_snippets