From 466e665b3cfdaee0f6149d16d5244b54ddb86838 Mon Sep 17 00:00:00 2001 From: Kurt Partridge Date: Wed, 27 Feb 2013 12:13:42 -0800 Subject: [PATCH] [FileEncap2] Remove mCanUpload instance variable Change-Id: I62a100c4d73bb64a14a7994f7dd9af7b6946d064 --- .../android/inputmethod/research/UploaderService.java | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/java/src/com/android/inputmethod/research/UploaderService.java b/java/src/com/android/inputmethod/research/UploaderService.java index 7acd79d63..ed195ba4b 100644 --- a/java/src/com/android/inputmethod/research/UploaderService.java +++ b/java/src/com/android/inputmethod/research/UploaderService.java @@ -56,7 +56,6 @@ public final class UploaderService extends IntentService { private static final int BUF_SIZE = 1024 * 8; protected static final int TIMEOUT_IN_MS = 1000 * 4; - private boolean mCanUpload; private File mFilesDir; private URL mUrl; @@ -68,7 +67,6 @@ public final class UploaderService extends IntentService { public void onCreate() { super.onCreate(); - mCanUpload = false; mFilesDir = null; mUrl = null; @@ -83,12 +81,15 @@ public final class UploaderService extends IntentService { } mFilesDir = getFilesDir(); mUrl = new URL(urlString); - mCanUpload = true; } catch (MalformedURLException e) { e.printStackTrace(); } } + public boolean isPossibleToUpload() { + return hasUploadingPermission() && mUrl != null && !IS_INHIBITING_AUTO_UPLOAD; + } + private boolean hasUploadingPermission() { final PackageManager packageManager = getPackageManager(); return packageManager.checkPermission(Manifest.permission.INTERNET, @@ -97,9 +98,7 @@ public final class UploaderService extends IntentService { @Override protected void onHandleIntent(Intent intent) { - if (!mCanUpload) { - return; - } + if (!isPossibleToUpload()) return; boolean isUploadingUnconditionally = false; Bundle bundle = intent.getExtras(); if (bundle != null && bundle.containsKey(EXTRA_UPLOAD_UNCONDITIONALLY)) {