summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJohn Reck <jreck@google.com>2010-11-24 17:01:43 -0800
committerJohn Reck <jreck@google.com>2010-11-24 17:01:43 -0800
commit50438d9205f5bebc9f3cefd8e00ae37850e2eb4d (patch)
treee4d9959ac49805cee924d843f6ca9dbaa8034f1d
parentc8490818ccb693ed2610366d2e91fb290e40c372 (diff)
downloadpackages_apps_Browser-50438d9205f5bebc9f3cefd8e00ae37850e2eb4d.tar.gz
packages_apps_Browser-50438d9205f5bebc9f3cefd8e00ae37850e2eb4d.tar.bz2
packages_apps_Browser-50438d9205f5bebc9f3cefd8e00ae37850e2eb4d.zip
Fix NPE from monkey test
Bug: 3224296 Change-Id: I3bfbb096bc29e0c6cbc6cbaad36ff0e7135b9a7c
-rw-r--r--src/com/android/browser/preferences/PageContentPreferencesFragment.java8
1 files changed, 8 insertions, 0 deletions
diff --git a/src/com/android/browser/preferences/PageContentPreferencesFragment.java b/src/com/android/browser/preferences/PageContentPreferencesFragment.java
index 4bb2fabbf..1b5d0feee 100644
--- a/src/com/android/browser/preferences/PageContentPreferencesFragment.java
+++ b/src/com/android/browser/preferences/PageContentPreferencesFragment.java
@@ -27,6 +27,7 @@ import android.os.Bundle;
import android.preference.EditTextPreference;
import android.preference.Preference;
import android.preference.PreferenceFragment;
+import android.util.Log;
public class PageContentPreferencesFragment extends PreferenceFragment
implements Preference.OnPreferenceChangeListener {
@@ -63,6 +64,13 @@ public class PageContentPreferencesFragment extends PreferenceFragment
@Override
public boolean onPreferenceChange(Preference pref, Object objValue) {
+ if (getActivity() == null) {
+ // We aren't attached, so don't accept preferences changes from the
+ // invisible UI.
+ Log.w("PageContentPreferencesFragment", "onPreferenceChange called from detached fragment!");
+ return false;
+ }
+
if (pref.getKey().equals(BrowserSettings.PREF_HOMEPAGE)) {
String value = (String) objValue;
boolean needUpdate = value.indexOf(' ') != -1;