add foreground service to keep App in memory (otherwise rate-limiting won't work) and provide notification

src/org/envaya/kalsms/App.java

@@ -58,6 +58,9 @@ public final class App extends Application {
     
     // intent to signal to Main activity (if open) that log has changed
     public static final String LOG_INTENT = "org.envaya.kalsms.LOG";
+    
+    public static final String START_INTENT = "org.envaya.kalsms.START";
+    public static final String STOP_INTENT = "org.envaya.kalsms.STOP";
             
     public static final String QUERY_EXPANSION_PACKS_INTENT = "org.envaya.kalsms.QUERY_EXPANSION_PACKS";
     public static final String QUERY_EXPANSION_PACKS_EXTRA_PACKAGES = "packages";
@@ -141,9 +144,15 @@ public final class App extends Application {
         mmsObserver = new MmsObserver(this);
         mmsObserver.register();
         
-        setOutgoingMessageAlarm();
+        setOutgoingMessageAlarm();        
+        updateEnabledNotification();
     }   
     
+    public void updateEnabledNotification()
+    {     
+        startService(new Intent(this, ForegroundService.class));        
+    }
+    
     public synchronized String chooseOutgoingSmsPackage()
     {
         outgoingMessageCount++;

src/org/envaya/kalsms/CheckMmsInboxService.java

@@ -42,7 +42,14 @@ public class CheckMmsInboxService extends IntentService
                 // times if we don't delete them.
                 mmsUtils.markOldMms(mms);                 
 
-                app.forwardToServer(mms);                                    
+                if (mms.isForwardable())
+                {                
+                    app.forwardToServer(mms);                                    
+                }
+                else
+                {
+                    app.log("Ignoring incoming MMS from " + mms.getFrom());
+                }
             }
         }
     }

src/org/envaya/kalsms/ForegroundService.java

@@ -0,0 +1,188 @@
+/*
+ * Copyright (C) 2009 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.envaya.kalsms;
+
+import android.app.Notification;
+import android.app.NotificationManager;
+import android.app.PendingIntent;
+import android.app.Service;
+import android.content.Intent;
+import android.os.IBinder;
+import android.util.Log;
+
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import org.envaya.kalsms.ui.Main;
+
+/*
+ *  Service running in foreground to make sure App instance stays 
+ *  in memory (otherwise we could lose timestamps of sent messages
+ *  which could cause us to exceed Android's SMS sending limit)
+ * 
+ *  Also adds notification to status bar. 
+ */
+public class ForegroundService extends Service {
+
+    private App app;
+    
+    private static final Class<?>[] mSetForegroundSignature = new Class[] {
+        boolean.class};
+    private static final Class<?>[] mStartForegroundSignature = new Class[] {
+        int.class, Notification.class};
+    private static final Class<?>[] mStopForegroundSignature = new Class[] {
+        boolean.class};
+
+    private NotificationManager mNM;
+    private Method mSetForeground;
+    private Method mStartForeground;
+    private Method mStopForeground;
+    private Object[] mSetForegroundArgs = new Object[1];
+    private Object[] mStartForegroundArgs = new Object[2];
+    private Object[] mStopForegroundArgs = new Object[1];
+
+    void invokeMethod(Method method, Object[] args) {
+        try {
+            mStartForeground.invoke(this, mStartForegroundArgs);
+        } catch (InvocationTargetException e) {
+            // Should not happen.
+            Log.w("ApiDemos", "Unable to invoke method", e);
+        } catch (IllegalAccessException e) {
+            // Should not happen.
+            Log.w("ApiDemos", "Unable to invoke method", e);
+        }
+    }
+
+    /**
+     * This is a wrapper around the new startForeground method, using the older
+     * APIs if it is not available.
+     */
+    void startForegroundCompat(int id, Notification notification) {
+        // If we have the new startForeground API, then use it.
+        if (mStartForeground != null) {
+            mStartForegroundArgs[0] = Integer.valueOf(id);
+            mStartForegroundArgs[1] = notification;
+            invokeMethod(mStartForeground, mStartForegroundArgs);
+            return;
+        }
+
+        // Fall back on the old API.
+        mSetForegroundArgs[0] = Boolean.TRUE;
+        invokeMethod(mSetForeground, mSetForegroundArgs);
+        mNM.notify(id, notification);
+    }
+
+    /**
+     * This is a wrapper around the new stopForeground method, using the older
+     * APIs if it is not available.
+     */
+    void stopForegroundCompat(int id) {
+        // If we have the new stopForeground API, then use it.
+        if (mStopForeground != null) {
+            mStopForegroundArgs[0] = Boolean.TRUE;
+            try {
+                mStopForeground.invoke(this, mStopForegroundArgs);
+            } catch (InvocationTargetException e) {
+                // Should not happen.
+                Log.w("ApiDemos", "Unable to invoke stopForeground", e);
+            } catch (IllegalAccessException e) {
+                // Should not happen.
+                Log.w("ApiDemos", "Unable to invoke stopForeground", e);
+            }
+            return;
+        }
+
+        // Fall back on the old API.  Note to cancel BEFORE changing the
+        // foreground state, since we could be killed at that point.
+        mNM.cancel(id);
+        mSetForegroundArgs[0] = Boolean.FALSE;
+        invokeMethod(mSetForeground, mSetForegroundArgs);
+    }
+
+    @Override
+    public void onCreate() {
+        mNM = (NotificationManager)getSystemService(NOTIFICATION_SERVICE);
+        app = (App)getApplication();
+        try {
+            mStartForeground = getClass().getMethod("startForeground",
+                    mStartForegroundSignature);
+            mStopForeground = getClass().getMethod("stopForeground",
+                    mStopForegroundSignature);
+        } catch (NoSuchMethodException e) {
+            // Running on an older platform.
+            mStartForeground = mStopForeground = null;
+            return;
+        }
+        try {
+            mSetForeground = getClass().getMethod("setForeground",
+                    mSetForegroundSignature);
+        } catch (NoSuchMethodException e) {
+            throw new IllegalStateException(
+                    "OS doesn't have Service.startForeground OR Service.setForeground!");
+        }
+    }
+
+    @Override
+    public void onDestroy() {
+        // Make sure our notification is gone.
+        stopForegroundCompat(R.string.service_started);
+    }
+
+    // This is the old onStart method that will be called on the pre-2.0
+    // platform.  On 2.0 or later we override onStartCommand() so this
+    // method will not be called.
+    @Override
+    public void onStart(Intent intent, int startId) {
+        handleCommand(intent);
+    }
+
+    //@Override
+    public int onStartCommand(Intent intent, int flags, int startId) {
+        handleCommand(intent);
+        // We want this service to continue running until it is explicitly
+        // stopped, so return sticky.
+        return 1; //START_STICKY;
+    }
+
+    void handleCommand(Intent intent) 
+    {        
+        if (app.isEnabled())           
+        {
+            CharSequence text = getText(R.string.service_started);
+            
+            Notification notification = new Notification(R.drawable.icon, text,
+                    System.currentTimeMillis());
+
+            PendingIntent contentIntent = PendingIntent.getActivity(this, 0,
+                    new Intent(this, Main.class), 0);
+
+            notification.setLatestEventInfo(this, 
+                    "KalSMS running",
+                   text, contentIntent);
+
+            startForegroundCompat(R.string.service_started, notification);
+        }
+        else
+        {
+            this.stopForegroundCompat(R.string.service_started);
+        }
+    }
+
+    @Override
+    public IBinder onBind(Intent intent) {
+        return null;
+    }
+}

src/org/envaya/kalsms/MmsObserver.java

@@ -1,12 +1,9 @@
 package org.envaya.kalsms;
 
-import android.app.IntentService;
 import android.content.Intent;
 import android.database.ContentObserver;
 import android.os.Handler;
-import java.util.HashSet;
 import java.util.List;
-import java.util.Set;
 
 final class MmsObserver extends ContentObserver {
 

src/org/envaya/kalsms/task/HttpTask.java

@@ -26,7 +26,6 @@ import org.apache.http.entity.mime.FormBodyPart;
 import org.apache.http.entity.mime.MultipartEntity;
 import org.apache.http.entity.mime.content.StringBody;
 import org.apache.http.message.BasicNameValuePair;
-import org.apache.http.util.EntityUtils;
 import org.envaya.kalsms.App;
 import org.envaya.kalsms.Base64Coder;
 import org.envaya.kalsms.OutgoingMessage;
@@ -210,6 +209,7 @@ public class HttpTask extends AsyncTask<String, Void, HttpResponse> {
             try
             {
                 handleResponse(response);            
+                response.getEntity().consumeContent();
             }
             catch (Throwable ex)
             {
@@ -226,10 +226,6 @@ public class HttpTask extends AsyncTask<String, Void, HttpResponse> {
     
     protected void handleResponse(HttpResponse response) throws Exception
     {
-        if (response != null)
-        {
-            response.getEntity().consumeContent();
-        }
     }    
     
     protected void handleFailure()

src/org/envaya/kalsms/ui/Prefs.java

@@ -102,6 +102,7 @@ public class Prefs extends PreferenceActivity implements OnSharedPreferenceChang
         {
             app.log(app.isEnabled() ? "SMS Gateway started." : "SMS Gateway stopped.");
             app.setOutgoingMessageAlarm();
+            app.updateEnabledNotification();
         }
         
         updatePrefSummary(findPreference(key));