chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageUma.java - Issue 1905013002: Refine tracking of snippet dismissals.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageUma.java

Issue 1905013002: Refine tracking of snippet dismissals. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Created 4 years, 8 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

« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java ('k') | chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetsBridge.java » ('j') | chrome/browser/android/ntp/ntp_snippets_bridge.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageUma.java

diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageUma.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageUma.java

index 245a3c456c67f607c92a2a1ddd71e317d91cb9db..f8b13ee56c920fad5d07ac99630fda1ff0023541 100644

--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageUma.java

+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageUma.java

@@ -62,7 +62,7 @@ public class NewTabPageUma {

/** Possible interactions with the snippets. */

@IntDef({SNIPPETS_ACTION_SHOWN, SNIPPETS_ACTION_SCROLLED, SNIPPETS_ACTION_CLICKED,

- SNIPPETS_ACTION_DISMISSED})

+ SNIPPETS_ACTION_DISMISSED_VISITED, SNIPPETS_ACTION_DISMISSED_UNVISITED})

@Retention(RetentionPolicy.SOURCE)

public @interface SnippetsAction {}

/** Snippets are enabled and are being shown to the user. */

@@ -71,10 +71,12 @@ public class NewTabPageUma {

public static final int SNIPPETS_ACTION_SCROLLED = 1;

/** A snippet has been clicked. */

public static final int SNIPPETS_ACTION_CLICKED = 2;

- /** A snippet has been swiped away. */

- public static final int SNIPPETS_ACTION_DISMISSED = 3;

+ /** A snippet has been swiped away, it had been viewed by the user (on this device). */

+ public static final int SNIPPETS_ACTION_DISMISSED_VISITED = 4;

+ /** A snippet has been swiped away, it had not been viewed by the user (on this device). */

+ public static final int SNIPPETS_ACTION_DISMISSED_UNVISITED = 5;

/** The number of possible actions. */

- private static final int NUM_SNIPPETS_ACTIONS = 4;

+ private static final int NUM_SNIPPETS_ACTIONS = 6;

/**

* Records an action taken by the user on the NTP.