diff options
author | Andrew Lee <anwlee@google.com> | 2015-04-03 16:48:00 -0700 |
---|---|---|
committer | Andrew Lee <anwlee@google.com> | 2015-04-03 17:16:02 -0700 |
commit | f652ab09390ecc5f0932cc5f5e7328395f3fd93d (patch) | |
tree | d9b59b2fc6ab6f25af0ac6c39180ef99e6d71ca4 /src | |
parent | 0d27e1d231226e6ca5e7ed51225f7886598b05b0 (diff) |
Don't track ReportDialog state in CallLogFragment.
This was implemented formerly to prevent a crash on rotation, but
this can be done by specifying that the Report fragment should be
retained in GoogleDialer.
This means that the fragment will not be retained now if the activity
is killed off by the system, but I think this should be okay since
it is a dialog.
Bug: 19949319
Change-Id: If3ebca4625fce93d9d9ce3c9bde6d92729f21c40
Diffstat (limited to 'src')
-rw-r--r-- | src/com/android/dialer/calllog/CallLogFragment.java | 20 |
1 files changed, 0 insertions, 20 deletions
diff --git a/src/com/android/dialer/calllog/CallLogFragment.java b/src/com/android/dialer/calllog/CallLogFragment.java index 7b5907c1f..c1c976bb7 100644 --- a/src/com/android/dialer/calllog/CallLogFragment.java +++ b/src/com/android/dialer/calllog/CallLogFragment.java @@ -67,8 +67,6 @@ public class CallLogFragment extends ListFragment private static final String TAG = "CallLogFragment"; private static final String REPORT_DIALOG_TAG = "report_dialog"; - private String mReportDialogNumber; - private boolean mIsReportDialogShowing; /** * ID of the empty loader to defer other fragments. @@ -79,8 +77,6 @@ public class CallLogFragment extends ListFragment private static final String KEY_LOG_LIMIT = "log_limit"; private static final String KEY_DATE_LIMIT = "date_limit"; private static final String KEY_SHOW_FOOTER = "show_footer"; - private static final String KEY_IS_REPORT_DIALOG_SHOWING = "is_report_dialog_showing"; - private static final String KEY_REPORT_DIALOG_NUMBER = "report_dialog_number"; private CallLogAdapter mAdapter; private CallLogQueryHandler mCallLogQueryHandler; @@ -185,9 +181,6 @@ public class CallLogFragment extends ListFragment mLogLimit = state.getInt(KEY_LOG_LIMIT, mLogLimit); mDateLimit = state.getLong(KEY_DATE_LIMIT, mDateLimit); mHasFooterView = state.getBoolean(KEY_SHOW_FOOTER, mHasFooterView); - mIsReportDialogShowing = state.getBoolean(KEY_IS_REPORT_DIALOG_SHOWING, - mIsReportDialogShowing); - mReportDialogNumber = state.getString(KEY_REPORT_DIALOG_NUMBER, mReportDialogNumber); } String currentCountryIso = GeoUtil.getCurrentCountryIso(getActivity()); @@ -214,14 +207,6 @@ public class CallLogFragment extends ListFragment mFadeOutDuration = getResources().getInteger(R.integer.call_log_actions_fade_out_duration); mExpandCollapseDuration = getResources().getInteger( R.integer.call_log_expand_collapse_duration); - - if (mIsReportDialogShowing) { - DialogFragment df = ObjectFactory.getReportDialogFragment(mReportDialogNumber); - if (df != null) { - df.setTargetFragment(this, 0); - df.show(getActivity().getFragmentManager(), REPORT_DIALOG_TAG); - } - } } /** Called by the CallLogQueryHandler when the list of calls has been fetched or updated. */ @@ -406,8 +391,6 @@ public class CallLogFragment extends ListFragment outState.putInt(KEY_LOG_LIMIT, mLogLimit); outState.putLong(KEY_DATE_LIMIT, mDateLimit); outState.putBoolean(KEY_SHOW_FOOTER, mHasFooterView); - outState.putBoolean(KEY_IS_REPORT_DIALOG_SHOWING, mIsReportDialogShowing); - outState.putString(KEY_REPORT_DIALOG_NUMBER, mReportDialogNumber); } @Override @@ -668,7 +651,6 @@ public class CallLogFragment extends ListFragment } public void onBadDataReported(String number) { - mIsReportDialogShowing = false; if (number == null) { return; } @@ -681,8 +663,6 @@ public class CallLogFragment extends ListFragment if (df != null) { df.setTargetFragment(this, 0); df.show(getActivity().getFragmentManager(), REPORT_DIALOG_TAG); - mReportDialogNumber = number; - mIsReportDialogShowing = true; } } } |