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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/LocationProvider.java

Issue 11419287: android: Improve ActivityStatus and add ChromiumActivity. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: fix "if" indentations Created 8 years 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: content/public/android/java/src/org/chromium/content/browser/LocationProvider.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/LocationProvider.java b/content/public/android/java/src/org/chromium/content/browser/LocationProvider.java
index 75be3ddb625b3792c7f49878279ca4a49edb5863..6efcb95f0f8ffe138bd4e89e897302637afd8b5d 100644
--- a/content/public/android/java/src/org/chromium/content/browser/LocationProvider.java
+++ b/content/public/android/java/src/org/chromium/content/browser/LocationProvider.java
@@ -37,7 +37,7 @@ class LocationProvider {
* ensures that the start/stop calls into this class are done in the UI thread.
*/
private static class LocationProviderImpl
- implements LocationListener, ActivityStatus.Listener {
+ implements LocationListener, ActivityStatus.StateListener {
private Context mContext;
private LocationManager mLocationManager;
@@ -49,11 +49,11 @@ class LocationProvider {
mContext = context;
}
- public void onActivityStatusChanged(boolean isPaused) {
- if (isPaused) {
+ public void onActivityStateChange(int state) {
+ if (state == ActivityStatus.PAUSED) {
mShouldRunAfterActivityResume = mIsRunning;
unregisterFromLocationUpdates();
- } else {
+ } else if (state == ActivityStatus.RESUMED) {
assert !mIsRunning;
if (mShouldRunAfterActivityResume) {
registerForLocationUpdates();
@@ -68,7 +68,7 @@ class LocationProvider {
private void start(boolean gpsEnabled) {
if (!mIsRunning && !mShouldRunAfterActivityResume) {
// Currently idle so start listening to activity status changes.
- ActivityStatus.getInstance().registerListener(this);
+ ActivityStatus.registerStateListener(this);
}
mIsGpsEnabled = gpsEnabled;
if (ActivityStatus.getInstance().isPaused()) {
@@ -84,7 +84,7 @@ class LocationProvider {
*/
private void stop() {
unregisterFromLocationUpdates();
- ActivityStatus.getInstance().unregisterListener(this);
+ ActivityStatus.unregisterStateListener(this);
mShouldRunAfterActivityResume = false;
}

Powered by Google App Engine
This is Rietveld 408576698