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

Unified Diff: chrome/test/android/javatests/src/org/chromium/chrome/test/BottomSheetTestCaseBase.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
« no previous file with comments | « chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapterTest.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/android/javatests/src/org/chromium/chrome/test/BottomSheetTestCaseBase.java
diff --git a/chrome/test/android/javatests/src/org/chromium/chrome/test/BottomSheetTestCaseBase.java b/chrome/test/android/javatests/src/org/chromium/chrome/test/BottomSheetTestCaseBase.java
index 480197bfd1c46649c89a832ca6e4b050562737a5..6a81592403e5b85097e68ee01ecfc4907f768ebe 100644
--- a/chrome/test/android/javatests/src/org/chromium/chrome/test/BottomSheetTestCaseBase.java
+++ b/chrome/test/android/javatests/src/org/chromium/chrome/test/BottomSheetTestCaseBase.java
@@ -26,8 +26,7 @@ public abstract class BottomSheetTestCaseBase extends ChromeTabbedActivityTestBa
// TODO(dgn,mdjones): Chrome restarts when the ChromeHome feature flag value changes. That
// crashes the test so we need to manually set the preference to match the flag before
// Chrome initialises via super.setUp()
- ChromePreferenceManager prefManager =
- ChromePreferenceManager.getInstance(getInstrumentation().getTargetContext());
+ ChromePreferenceManager prefManager = ChromePreferenceManager.getInstance();
mOldChromeHomeFlagValue = prefManager.isChromeHomeEnabled();
prefManager.setChromeHomeEnabled(true);
@@ -52,8 +51,7 @@ public abstract class BottomSheetTestCaseBase extends ChromeTabbedActivityTestBa
@Override
protected void tearDown() throws Exception {
super.tearDown();
- ChromePreferenceManager.getInstance(getInstrumentation().getTargetContext())
- .setChromeHomeEnabled(mOldChromeHomeFlagValue);
+ ChromePreferenceManager.getInstance().setChromeHomeEnabled(mOldChromeHomeFlagValue);
}
protected BottomSheetContent getBottomSheetContent() {
« no previous file with comments | « chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapterTest.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698