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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivity.java

Issue 2108983002: Use ContextUtils in PhysicalWeb (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update tests to use new methods Created 4 years, 5 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/java/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivity.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivity.java
index 9900cd90ca00ce633d7a00368d9e3ac07489d2cc..22f951a0a96a4e3a0d0fce8b815d73cf1baa441d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivity.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivity.java
@@ -32,7 +32,6 @@ import org.chromium.base.ContextUtils;
import org.chromium.base.Log;
import org.chromium.base.VisibleForTesting;
import org.chromium.chrome.R;
-import org.chromium.chrome.browser.ChromeApplication;
import org.chromium.chrome.browser.widget.FadingShadow;
import org.chromium.chrome.browser.widget.FadingShadowView;
@@ -122,8 +121,7 @@ public class ListUrlsActivity extends AppCompatActivity implements AdapterView.O
mIsInitialDisplayRecorded = false;
mIsRefreshing = false;
mIsRefreshUserInitiated = false;
- mPhysicalWebBleClient =
- PhysicalWebBleClient.getInstance((ChromeApplication) getApplicationContext());
+ mPhysicalWebBleClient = PhysicalWebBleClient.getInstance();
}
@Override
@@ -169,7 +167,7 @@ public class ListUrlsActivity extends AppCompatActivity implements AdapterView.O
@Override
protected void onStart() {
super.onStart();
- UrlManager.getInstance(this).addObserver(this);
+ UrlManager.getInstance().addObserver(this);
}
@Override
@@ -198,7 +196,7 @@ public class ListUrlsActivity extends AppCompatActivity implements AdapterView.O
@Override
protected void onStop() {
- UrlManager.getInstance(this).removeObserver(this);
+ UrlManager.getInstance().removeObserver(this);
super.onStop();
}
@@ -267,13 +265,13 @@ public class ListUrlsActivity extends AppCompatActivity implements AdapterView.O
// Clear the list adapter to trigger the empty list display.
mAdapter.clear();
- Collection<UrlInfo> urls = UrlManager.getInstance(this).getUrls(true);
+ Collection<UrlInfo> urls = UrlManager.getInstance().getUrls(true);
// Check the Physical Web preference to ensure we do not resolve URLs when Physical Web is
// off or onboarding. Normally the user will not reach this activity unless the preference
// is explicitly enabled, but there is a button on the diagnostics page that launches into
// the activity without checking the preference state.
- if (urls.isEmpty() || !PhysicalWeb.isPhysicalWebPreferenceEnabled(this)) {
+ if (urls.isEmpty() || !PhysicalWeb.isPhysicalWebPreferenceEnabled()) {
finishRefresh();
} else {
// Show the swipe-to-refresh busy indicator for refreshes initiated by a swipe.

Powered by Google App Engine
This is Rietveld 408576698