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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java

Issue 2865963003: [Suggestions UI] Drop Bitmap references from articles under memory pressure. (Closed)
Patch Set: remove annotation Created 3 years, 7 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/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java
index 8fd2267f9f0d8d30231c021e0306bfcdc49178fe..cb7a4410a90300953aa1c3bfd636f85d492f855c 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java
@@ -4,6 +4,7 @@
package org.chromium.chrome.browser.ntp.snippets;
+import android.graphics.Bitmap;
import android.graphics.BitmapFactory;
import android.support.test.InstrumentationRegistry;
import android.support.test.filters.MediumTest;
@@ -16,6 +17,7 @@ import org.junit.Rule;
import org.junit.Test;
import org.junit.runner.RunWith;
+import org.chromium.base.DiscardableReferencePool;
import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.base.test.util.Feature;
@@ -146,9 +148,11 @@ public class ArticleSnippetsTest {
1466614774, // Publish timestamp
10f, // Score
1466634774); // Fetch timestamp
- shortSnippet.setThumbnailBitmap(
+
+ Bitmap thumbnail =
BitmapFactory.decodeResource(mActivityTestRule.getActivity().getResources(),
- R.drawable.signin_promo_illustration));
+ R.drawable.signin_promo_illustration);
+ shortSnippet.setThumbnailBitmap(mUiDelegate.getReferencePool().put(thumbnail));
SnippetArticle longSnippet = new SnippetArticle(fullCategory, "id2",
new String(new char[20]).replace("\0", "Snippet "),
@@ -206,6 +210,7 @@ public class ArticleSnippetsTest {
private SuggestionsEventReporter mSuggestionsEventReporter =
new DummySuggestionsEventReporter();
private SuggestionsRanker mSuggestionsRanker = new SuggestionsRanker();
+ private final DiscardableReferencePool mReferencePool = new DiscardableReferencePool();
@Override
public void getLocalFaviconImageForURL(
@@ -245,6 +250,11 @@ public class ArticleSnippetsTest {
return mSuggestionsRanker;
}
+ @Override
+ public DiscardableReferencePool getReferencePool() {
+ return mReferencePool;
+ }
+
@Override
public void addDestructionObserver(DestructionObserver destructionObserver) {}

Powered by Google App Engine
This is Rietveld 408576698