[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[tor-commits] [orbot/master] Removes the refresh button on the VPN Selection screen
commit cde49d1a715f6afca07d234eaea3637e0aacd0af
Author: bim <dsnake@xxxxxxxxxxxxxx>
Date: Wed Jun 26 22:30:51 2019 -0400
Removes the refresh button on the VPN Selection screen
Since this operaiton happens onResume() in the
activity, there's not really any need for the user to invoke it and from
a user's perspective it isnt entirely clear what it does.
Additionally, this commit removes unused code and addresses IDE
warnings in AppManagerActivity
---
.../torproject/android/ui/AppManagerActivity.java | 87 +++------------------
.../main/res/drawable/ic_autorenew_black_36dp.png | Bin 546 -> 0 bytes
app/src/main/res/menu/orbot_apps.xml | 30 -------
3 files changed, 13 insertions(+), 104 deletions(-)
diff --git a/app/src/main/java/org/torproject/android/ui/AppManagerActivity.java b/app/src/main/java/org/torproject/android/ui/AppManagerActivity.java
index 2f60a40c..071788fc 100644
--- a/app/src/main/java/org/torproject/android/ui/AppManagerActivity.java
+++ b/app/src/main/java/org/torproject/android/ui/AppManagerActivity.java
@@ -16,20 +16,16 @@ import org.torproject.android.R;
import org.torproject.android.service.util.TorServiceUtils;
import org.torproject.android.service.vpn.TorifiedApp;
-import android.content.Context;
import android.content.Intent;
import android.content.SharedPreferences;
import android.content.SharedPreferences.Editor;
import android.content.pm.ApplicationInfo;
import android.content.pm.PackageInfo;
import android.content.pm.PackageManager;
-import android.graphics.Color;
import android.os.AsyncTask;
import android.os.Bundle;
import android.support.v7.app.AppCompatActivity;
import android.view.LayoutInflater;
-import android.view.Menu;
-import android.view.MenuInflater;
import android.view.MenuItem;
import android.view.View;
import android.view.View.OnClickListener;
@@ -47,7 +43,6 @@ public class AppManagerActivity extends AppCompatActivity implements OnClickList
private GridView listApps;
private ListAdapter adapterApps;
private ProgressBar progressBar;
- private final static String TAG = "Orbot";
PackageManager pMgr = null;
protected void onCreate(Bundle savedInstanceState) {
@@ -62,34 +57,12 @@ public class AppManagerActivity extends AppCompatActivity implements OnClickList
progressBar = findViewById(R.id.progressBar);
}
- /*
- * Create the UI Options Menu (non-Javadoc)
- * @see android.app.Activity#onCreateOptionsMenu(android.view.Menu)
- */
- @Override
- public boolean onCreateOptionsMenu(Menu menu) {
- super.onCreateOptionsMenu(menu);
- MenuInflater inflater = getMenuInflater();
- inflater.inflate(R.menu.orbot_apps, menu);
-
- return true;
- }
-
-
@Override
public boolean onOptionsItemSelected(MenuItem item) {
- if (item.getItemId() == android.R.id.home)
- {
+ if (item.getItemId() == android.R.id.home) {
finish();
return true;
}
- else if (item.getItemId() == R.id.menu_apps_refresh)
- {
- mApps = null;
- reloadApps();
- return true;
- }
-
return super.onOptionsItemSelected(item);
}
@@ -122,11 +95,9 @@ public class AppManagerActivity extends AppCompatActivity implements OnClickList
SharedPreferences mPrefs = null;
ArrayList<TorifiedApp> mApps = null;
- private void loadApps (SharedPreferences prefs)
- {
-
+ private void loadApps (SharedPreferences prefs) {
if (mApps == null)
- mApps = getApps(getApplicationContext(), prefs);
+ mApps = getApps(prefs);
Collections.sort(mApps,new Comparator<TorifiedApp>() {
public int compare(TorifiedApp o1, TorifiedApp o2) {
@@ -145,6 +116,7 @@ public class AppManagerActivity extends AppCompatActivity implements OnClickList
adapterApps = new ArrayAdapter<TorifiedApp>(this, R.layout.layout_apps_item, R.id.itemtext,mApps) {
+ @Override
public View getView(int position, View convertView, ViewGroup parent) {
ListEntry entry = null;
@@ -157,9 +129,9 @@ public class AppManagerActivity extends AppCompatActivity implements OnClickList
if (entry == null) {
// Inflate a new view
entry = new ListEntry();
- entry.icon = (ImageView) convertView.findViewById(R.id.itemicon);
- entry.box = (CheckBox) convertView.findViewById(R.id.itemcheck);
- entry.text = (TextView) convertView.findViewById(R.id.itemtext);
+ entry.icon = convertView.findViewById(R.id.itemicon);
+ entry.box = convertView.findViewById(R.id.itemcheck);
+ entry.text = convertView.findViewById(R.id.itemtext);
convertView.setTag(entry);
}
@@ -203,18 +175,7 @@ public class AppManagerActivity extends AppCompatActivity implements OnClickList
private ImageView icon;
}
- /* (non-Javadoc)
- * @see android.app.Activity#onStop()
- */
- @Override
- protected void onStop() {
- super.onStop();
-
- }
-
-
- public ArrayList<TorifiedApp> getApps (Context context, SharedPreferences prefs)
- {
+ public ArrayList<TorifiedApp> getApps(SharedPreferences prefs) {
String tordAppString = prefs.getString(PREFS_KEY_TORIFIED, "");
String[] tordApps;
@@ -232,11 +193,11 @@ public class AppManagerActivity extends AppCompatActivity implements OnClickList
Iterator<ApplicationInfo> itAppInfo = lAppInfo.iterator();
- ArrayList<TorifiedApp> apps = new ArrayList<TorifiedApp>();
+ ArrayList<TorifiedApp> apps = new ArrayList<>();
- ApplicationInfo aInfo = null;
+ ApplicationInfo aInfo;
- TorifiedApp app = null;
+ TorifiedApp app;
while (itAppInfo.hasNext())
{
@@ -316,8 +277,7 @@ public class AppManagerActivity extends AppCompatActivity implements OnClickList
}
- public void saveAppSettings (Context context)
- {
+ public void saveAppSettings() {
StringBuilder tordApps = new StringBuilder();
Intent response = new Intent();
@@ -340,23 +300,6 @@ public class AppManagerActivity extends AppCompatActivity implements OnClickList
}
- /**
- * Called an application is check/unchecked
- */
- /**
- public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
- final TorifiedApp app = (TorifiedApp) buttonView.getTag();
- if (app != null) {
- app.setTorified(isChecked);
- }
-
- saveAppSettings(this);
-
- }**/
-
-
-
-
public void onClick(View v) {
CheckBox cbox = null;
@@ -373,11 +316,7 @@ public class AppManagerActivity extends AppCompatActivity implements OnClickList
cbox.setChecked(app.isTorified());
}
- saveAppSettings(this);
+ saveAppSettings();
}
}
-
-
-
-
}
diff --git a/app/src/main/res/drawable/ic_autorenew_black_36dp.png b/app/src/main/res/drawable/ic_autorenew_black_36dp.png
deleted file mode 100644
index 6ae7ac8a..00000000
Binary files a/app/src/main/res/drawable/ic_autorenew_black_36dp.png and /dev/null differ
diff --git a/app/src/main/res/menu/orbot_apps.xml b/app/src/main/res/menu/orbot_apps.xml
deleted file mode 100644
index 12bcbc3c..00000000
--- a/app/src/main/res/menu/orbot_apps.xml
+++ /dev/null
@@ -1,30 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!--
-/*
- * Copyright (C) 2008 Esmertec AG.
- * Copyright (C) 2008 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.
- */
--->
-<menu xmlns:android="http://schemas.android.com/apk/res/android";
- xmlns:yourapp="http://schemas.android.com/apk/res-auto";
- >
-
- <item android:id="@+id/menu_apps_refresh"
- android:title=""
- android:icon="@drawable/ic_autorenew_black_36dp"
- yourapp:showAsAction="always"
- />
-
-</menu>
_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits