From d92e0419b10d378c1f480be7088c469e1d1dd5f8 Mon Sep 17 00:00:00 2001 From: Jeff Sharkey Date: Tue, 24 Apr 2012 11:35:43 -0700 Subject: [PATCH] Fix data usage settings button visibility. Also check that fragment is attached before launching intent. Bug: 6385040 Change-Id: If857758acbf6fe17c8ffbd75226114346ffe6c62 --- src/com/android/settings/DataUsageSummary.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/com/android/settings/DataUsageSummary.java b/src/com/android/settings/DataUsageSummary.java index 0d003b33645..4a2c2625590 100644 --- a/src/com/android/settings/DataUsageSummary.java +++ b/src/com/android/settings/DataUsageSummary.java @@ -781,6 +781,7 @@ public class DataUsageSummary extends Fragment { final boolean matchFound = pm.resolveActivity(mAppSettingsIntent, 0) != null; mAppSettings.setEnabled(matchFound); + mAppSettings.setVisibility(View.VISIBLE); } else { mAppSettingsIntent = null; @@ -1047,6 +1048,8 @@ public class DataUsageSummary extends Fragment { private OnClickListener mAppSettingsListener = new OnClickListener() { @Override public void onClick(View v) { + if (!isAdded()) return; + // TODO: target torwards entire UID instead of just first package startActivity(mAppSettingsIntent); }