Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(129)

Unified Diff: chrome/browser/android/ntp/content_suggestions_notification_helper.h

Issue 2639553002: Track active notifications in Java (Closed)
Patch Set: FindBugs 2 Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/ntp/content_suggestions_notification_helper.h
diff --git a/chrome/browser/android/ntp/content_suggestions_notification_helper.h b/chrome/browser/android/ntp/content_suggestions_notification_helper.h
index 05f54b1e73ef53cc196ce92251293e96427ab703..df59e762ddc86cc6d39273c85f4a088f11ecd54d 100644
--- a/chrome/browser/android/ntp/content_suggestions_notification_helper.h
+++ b/chrome/browser/android/ntp/content_suggestions_notification_helper.h
@@ -11,6 +11,8 @@
#include "base/macros.h"
#include "base/strings/string16.h"
#include "base/time/time.h"
+#include "chrome/browser/ntp_snippets/ntp_snippets_metrics.h"
+#include "components/ntp_snippets/content_suggestion.h"
#include "url/gurl.h"
class Profile;
@@ -23,12 +25,15 @@ namespace ntp_snippets {
class ContentSuggestionsNotificationHelper {
public:
- static void SendNotification(const GURL& url,
+ static bool SendNotification(const ContentSuggestion::ID& id,
+ const GURL& url,
const base::string16& title,
const base::string16& text,
const gfx::Image& image,
base::Time timeout_at);
- static void HideAllNotifications();
+ static void HideNotification(const ContentSuggestion::ID& id,
+ ContentSuggestionsNotificationAction why);
+ static void HideAllNotifications(ContentSuggestionsNotificationAction why);
// Moves metrics tracked in Java into native histograms. Should be called when
// the native library starts up, to capture any actions that were taken since

Powered by Google App Engine
This is Rietveld 408576698