diff options
| author | John Reck <jreck@google.com> | 2010-11-24 17:01:43 -0800 |
|---|---|---|
| committer | John Reck <jreck@google.com> | 2010-11-24 17:01:43 -0800 |
| commit | 50438d9205f5bebc9f3cefd8e00ae37850e2eb4d (patch) | |
| tree | e4d9959ac49805cee924d843f6ca9dbaa8034f1d | |
| parent | c8490818ccb693ed2610366d2e91fb290e40c372 (diff) | |
| download | packages_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.java | 8 |
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; |
