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

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

Issue 2522593010: Remove warnings in org.chromium.chrome.browser.ntp.* code and JUnit tests. (Closed)
Patch Set: review Created 4 years, 1 month 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/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerView.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerView.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerView.java
index 0395613912f5c56c2d26cccdb9fcd03a4d9904f4..75010922759c694c42e4334015cc99657cae037c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerView.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerView.java
@@ -49,8 +49,6 @@ public class NewTabPageRecyclerView extends RecyclerView implements TouchDisable
private static final int PEEKING_CARD_ANIMATION_TIME_MS = 1000;
private static final int PEEKING_CARD_ANIMATION_START_DELAY_MS = 300;
- private static final String PREF_ANIMATION_RUN_COUNT = "ntp_recycler_view_animation_run_count";
-
private final GestureDetector mGestureDetector;
private final LinearLayoutManager mLayoutManager;
private final int mToolbarHeight;
@@ -481,9 +479,8 @@ public class NewTabPageRecyclerView extends RecyclerView implements TouchDisable
/**
* Animates the card being swiped to the right as if the user had dismissed it. Any changes to
- * the animation here should be reflected also in
- * {@link #updateViewStateForDismiss(float, ViewHolder)} and reset in
- * {@link CardViewHolder#onBindViewHolder()}.
+ * the animation here should be reflected also in {@link #updateViewStateForDismiss} and reset
+ * in {@link CardViewHolder#onBindViewHolder()}.
*/
public void dismissItemWithAnimation(final ViewHolder viewHolder) {
// We need to check the position, as the view holder might have been removed.

Powered by Google App Engine
This is Rietveld 408576698