service should be unbound when activity is destroyed

also some other cleanup of unused code
This commit is contained in:
Nathan Freitas 2013-12-28 00:43:36 -05:00
parent 867ff53b8b
commit 5891feee9a
1 changed files with 12 additions and 30 deletions

View File

@ -34,7 +34,6 @@ import android.os.Handler;
import android.os.IBinder; import android.os.IBinder;
import android.os.Message; import android.os.Message;
import android.os.RemoteException; import android.os.RemoteException;
import android.preference.PreferenceManager;
import android.text.ClipboardManager; import android.text.ClipboardManager;
import android.text.Layout; import android.text.Layout;
import android.text.method.ScrollingMovementMethod; import android.text.method.ScrollingMovementMethod;
@ -47,7 +46,6 @@ import android.view.View;
import android.view.View.OnLongClickListener; import android.view.View.OnLongClickListener;
import android.view.View.OnTouchListener; import android.view.View.OnTouchListener;
import android.view.animation.AccelerateInterpolator; import android.view.animation.AccelerateInterpolator;
import android.view.animation.Animation;
import android.widget.Button; import android.widget.Button;
import android.widget.SlidingDrawer; import android.widget.SlidingDrawer;
import android.widget.TextView; import android.widget.TextView;
@ -97,6 +95,8 @@ public class Orbot extends SherlockActivity implements TorConstants, OnLongClick
StrictMode.setVmPolicy(vmpolicy); StrictMode.setVmPolicy(vmpolicy);
} }
*/ */
//Kill tor if the button stop tor (in the notification) was clicked!
//Same code needs to be executed in onResume
mPrefs = getPrefs(); mPrefs = getPrefs();
mPrefs.registerOnSharedPreferenceChangeListener(this); mPrefs.registerOnSharedPreferenceChangeListener(this);
@ -342,10 +342,6 @@ public class Orbot extends SherlockActivity implements TorConstants, OnLongClick
protected void onPause() { protected void onPause() {
super.onPause(); super.onPause();
//unbindService();
//hideProgressDialog();
if (aDialog != null) if (aDialog != null)
aDialog.dismiss(); aDialog.dismiss();
} }
@ -427,7 +423,6 @@ public class Orbot extends SherlockActivity implements TorConstants, OnLongClick
super.onResume(); super.onResume();
bindService(); bindService();
updateStatus(""); updateStatus("");
} }
@ -511,26 +506,6 @@ public class Orbot extends SherlockActivity implements TorConstants, OnLongClick
} }
/*
else if (Intent.ACTION_SEND.equals(action))
{
Uri dataUri = (Uri) intent.getParcelableExtra(Intent.EXTRA_STREAM);
try {
String shareUrl = mService.addOnionShare(dataUri, type);
Toast.makeText(this, "Share available at: " + shareUrl, Toast.LENGTH_LONG).show();
ClipboardManager cm = (ClipboardManager)getSystemService(Context.CLIPBOARD_SERVICE);
cm.setText(shareUrl);
intent.setAction(null);
} catch (RemoteException e) {
// TODO Auto-generated catch block
e.printStackTrace();
}
}*/
else else
{ {
@ -574,7 +549,6 @@ public class Orbot extends SherlockActivity implements TorConstants, OnLongClick
protected void onStop() { protected void onStop() {
super.onStop(); super.onStop();
//unbindService();
} }
@ -1157,6 +1131,7 @@ public class Orbot extends SherlockActivity implements TorConstants, OnLongClick
} }
*/ */
private void setLocale () private void setLocale ()
{ {
@ -1174,6 +1149,13 @@ public class Orbot extends SherlockActivity implements TorConstants, OnLongClick
} }
} }
@Override
protected void onDestroy() {
super.onDestroy();
unbindService();
}
public class DataCount { public class DataCount {
// data uploaded // data uploaded
public long Upload; public long Upload;