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

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

Issue 2692203005: Remove unused context in ChromePreferenceManager. (Closed)
Patch Set: Update new use Created 3 years, 10 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/junit/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapterTest.java
diff --git a/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapterTest.java b/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapterTest.java
index 1726bfd2befa093d383e73404d99313a6839f3be..f63ea838bf37f5cbec41128152f01233b20c766b 100644
--- a/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapterTest.java
+++ b/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapterTest.java
@@ -198,8 +198,7 @@ public class NewTabPageAdapterTest {
ChromeFeatureList.setTestEnabledFeatures(Collections.<String>emptySet());
// Initialise the sign in state. We will be signed in by default in the tests.
- assertFalse(ChromePreferenceManager.getInstance(RuntimeEnvironment.application)
- .getNewTabPageSigninPromoDismissed());
+ assertFalse(ChromePreferenceManager.getInstance().getNewTabPageSigninPromoDismissed());
SigninManager.setInstanceForTesting(mMockSigninManager);
when(mMockSigninManager.isSignedInOnNative()).thenReturn(true);
when(mMockSigninManager.isSignInAllowed()).thenReturn(true);
@@ -220,8 +219,7 @@ public class NewTabPageAdapterTest {
CardsVariationParameters.setTestVariationParams(null);
ChromeFeatureList.setTestEnabledFeatures(null);
SigninManager.setInstanceForTesting(null);
- ChromePreferenceManager.getInstance(RuntimeEnvironment.application)
- .setNewTabPageSigninPromoDismissed(false);
+ ChromePreferenceManager.getInstance().setNewTabPageSigninPromoDismissed(false);
}
/**
@@ -791,8 +789,7 @@ public class NewTabPageAdapterTest {
when(mMockSigninManager.isSignInAllowed()).thenReturn(true);
when(mMockSigninManager.isSignedInOnNative()).thenReturn(false);
- ChromePreferenceManager.getInstance(RuntimeEnvironment.application)
- .setNewTabPageSigninPromoDismissed(false);
+ ChromePreferenceManager.getInstance().setNewTabPageSigninPromoDismissed(false);
reloadNtp();
final int signInPromoPosition = mAdapter.getFirstPositionForType(ItemViewType.PROMO);
@@ -803,8 +800,7 @@ public class NewTabPageAdapterTest {
verify(itemDismissedCallback).onResult(anyString());
assertFalse(isSignInPromoVisible());
- assertTrue(ChromePreferenceManager.getInstance(RuntimeEnvironment.application)
- .getNewTabPageSigninPromoDismissed());
+ assertTrue(ChromePreferenceManager.getInstance().getNewTabPageSigninPromoDismissed());
reloadNtp();
assertFalse(isSignInPromoVisible());

Powered by Google App Engine
This is Rietveld 408576698