Enable and add logging to ManageApplications

Bug: 77541894
Test: n/a
Change-Id: Ia6a26c97aa4cb7d81987a291d4b568e89087959b
This commit is contained in:
Beverly
2018-06-21 10:23:03 -04:00
parent 2f4355aa66
commit b3ba0ac11f
2 changed files with 31 additions and 8 deletions

View File

@@ -23,6 +23,7 @@ import android.os.UserHandle;
import android.os.UserManager; import android.os.UserManager;
import android.text.format.DateUtils; import android.text.format.DateUtils;
import android.util.ArrayMap; import android.util.ArrayMap;
import android.util.Log;
import android.view.View; import android.view.View;
import android.view.ViewGroup; import android.view.ViewGroup;
import android.widget.Switch; import android.widget.Switch;
@@ -46,6 +47,8 @@ import java.util.Map;
*/ */
public class AppStateNotificationBridge extends AppStateBaseBridge { public class AppStateNotificationBridge extends AppStateBaseBridge {
private final String TAG = "AppStateNotificationBridge";
private final boolean DEBUG = true;
private final Context mContext; private final Context mContext;
private IUsageStatsManager mUsageStatsManager; private IUsageStatsManager mUsageStatsManager;
protected List<Integer> mUserIds; protected List<Integer> mUserIds;
@@ -70,7 +73,12 @@ public class AppStateNotificationBridge extends AppStateBaseBridge {
@Override @Override
protected void loadAllExtraInfo() { protected void loadAllExtraInfo() {
ArrayList<AppEntry> apps = mAppSession.getAllApps(); ArrayList<AppEntry> apps = mAppSession.getAllApps();
if (apps == null) return; if (apps == null) {
if (DEBUG) {
Log.d(TAG, "No apps. No extra info loaded");
}
return;
}
final Map<String, NotificationsSentState> map = getAggregatedUsageEvents(); final Map<String, NotificationsSentState> map = getAggregatedUsageEvents();
for (AppEntry entry : apps) { for (AppEntry entry : apps) {

View File

@@ -16,7 +16,6 @@
package com.android.settings.applications.manageapplications; package com.android.settings.applications.manageapplications;
import static androidx.recyclerview.widget.RecyclerView.SCROLL_STATE_IDLE;
import static com.android.settings.applications.manageapplications.AppFilterRegistry import static com.android.settings.applications.manageapplications.AppFilterRegistry
.FILTER_APPS_ALL; .FILTER_APPS_ALL;
import static com.android.settings.applications.manageapplications.AppFilterRegistry import static com.android.settings.applications.manageapplications.AppFilterRegistry
@@ -38,6 +37,8 @@ import static com.android.settings.applications.manageapplications.AppFilterRegi
import static com.android.settings.applications.manageapplications.AppFilterRegistry import static com.android.settings.applications.manageapplications.AppFilterRegistry
.FILTER_APPS_WORK; .FILTER_APPS_WORK;
import static androidx.recyclerview.widget.RecyclerView.SCROLL_STATE_IDLE;
import android.annotation.Nullable; import android.annotation.Nullable;
import android.annotation.StringRes; import android.annotation.StringRes;
import android.app.Activity; import android.app.Activity;
@@ -141,7 +142,7 @@ public class ManageApplications extends InstrumentedFragment
implements View.OnClickListener, OnItemSelectedListener { implements View.OnClickListener, OnItemSelectedListener {
static final String TAG = "ManageApplications"; static final String TAG = "ManageApplications";
static final boolean DEBUG = Log.isLoggable(TAG, Log.DEBUG); static final boolean DEBUG = true;
// Intent extras. // Intent extras.
public static final String EXTRA_CLASSNAME = "classname"; public static final String EXTRA_CLASSNAME = "classname";
@@ -721,7 +722,9 @@ public class ManageApplications extends InstrumentedFragment
mFilter = mFilterAdapter.getFilter(position); mFilter = mFilterAdapter.getFilter(position);
mApplications.setFilter(mFilter); mApplications.setFilter(mFilter);
if (DEBUG) Log.d(TAG, "Selecting filter " + mFilter); if (DEBUG) {
Log.d(TAG, "Selecting filter " + getContext().getText(mFilter.getTitle()));
}
} }
@Override @Override
@@ -784,7 +787,7 @@ public class ManageApplications extends InstrumentedFragment
return; return;
} }
if (DEBUG) { if (DEBUG) {
Log.d(TAG, "Enabling filter " + filter); Log.d(TAG, "Enabling filter " + mContext.getText(filter.getTitle()));
} }
mFilterOptions.add(filter); mFilterOptions.add(filter);
Collections.sort(mFilterOptions); Collections.sort(mFilterOptions);
@@ -793,7 +796,8 @@ public class ManageApplications extends InstrumentedFragment
notifyDataSetChanged(); notifyDataSetChanged();
if (mFilterOptions.size() == 1) { if (mFilterOptions.size() == 1) {
if (DEBUG) { if (DEBUG) {
Log.d(TAG, "Auto selecting filter " + filter); Log.d(TAG, "Auto selecting filter " + filter + " " + mContext.getText(
filter.getTitle()));
} }
mManageApplications.mFilterSpinner.setSelection(0); mManageApplications.mFilterSpinner.setSelection(0);
mManageApplications.onItemSelected(null, null, 0, 0); mManageApplications.onItemSelected(null, null, 0, 0);
@@ -816,7 +820,8 @@ public class ManageApplications extends InstrumentedFragment
return; return;
} }
if (DEBUG) { if (DEBUG) {
Log.d(TAG, "Disabling filter " + filter); Log.d(TAG, "Disabling filter " + filter + " " + mContext.getText(
filter.getTitle()));
} }
Collections.sort(mFilterOptions); Collections.sort(mFilterOptions);
mManageApplications.mSpinnerHeader.setVisibility( mManageApplications.mSpinnerHeader.setVisibility(
@@ -825,7 +830,8 @@ public class ManageApplications extends InstrumentedFragment
if (mManageApplications.mFilter == filter) { if (mManageApplications.mFilter == filter) {
if (mFilterOptions.size() > 0) { if (mFilterOptions.size() > 0) {
if (DEBUG) { if (DEBUG) {
Log.d(TAG, "Auto selecting filter " + mFilterOptions.get(0)); Log.d(TAG, "Auto selecting filter " + mFilterOptions.get(0)
+ mContext.getText(mFilterOptions.get(0).getTitle()));
} }
mManageApplications.mFilterSpinner.setSelection(0); mManageApplications.mFilterSpinner.setSelection(0);
mManageApplications.onItemSelected(null, null, 0, 0); mManageApplications.onItemSelected(null, null, 0, 0);
@@ -1035,6 +1041,12 @@ public class ManageApplications extends InstrumentedFragment
if (!mHasReceivedLoadEntries if (!mHasReceivedLoadEntries
|| (mExtraInfoBridge != null && !mHasReceivedBridgeCallback)) { || (mExtraInfoBridge != null && !mHasReceivedBridgeCallback)) {
// Don't rebuild the list until all the app entries are loaded. // Don't rebuild the list until all the app entries are loaded.
if (DEBUG) {
Log.d(TAG, "Not rebuilding until all the app entries loaded."
+ " !mHasReceivedLoadEntries=" + !mHasReceivedLoadEntries
+ " !mExtraInfoBridgeNull=" + (mExtraInfoBridge != null)
+ " !mHasReceivedBridgeCallback=" + !mHasReceivedBridgeCallback);
}
return; return;
} }
ApplicationsState.AppFilter filterObj; ApplicationsState.AppFilter filterObj;
@@ -1140,6 +1152,9 @@ public class ManageApplications extends InstrumentedFragment
@Override @Override
public void onRebuildComplete(ArrayList<AppEntry> entries) { public void onRebuildComplete(ArrayList<AppEntry> entries) {
if (DEBUG) {
Log.d(TAG, "onRebuildComplete");
}
final int filterType = mAppFilter.getFilterType(); final int filterType = mAppFilter.getFilterType();
if (filterType == FILTER_APPS_POWER_WHITELIST || if (filterType == FILTER_APPS_POWER_WHITELIST ||
filterType == FILTER_APPS_POWER_WHITELIST_ALL) { filterType == FILTER_APPS_POWER_WHITELIST_ALL) {