From 23176395a5413ad55eeaff780e60f36e7d70977e Mon Sep 17 00:00:00 2001 From: "Philip P. Moltmann" Date: Mon, 27 Jun 2016 13:45:25 -0700 Subject: [PATCH] Setup up print settings fragments in onCreateView instead of onCreate. Otherwise some of the layout is not yet set up. Fixes: 29764546 Change-Id: Ia0716dd0603d66e8bf68494e426756bd553f1afd --- .../android/settings/print/PrintJobSettingsFragment.java | 9 +++++++-- .../settings/print/PrintServiceSettingsFragment.java | 8 ++++++-- .../android/settings/print/PrintSettingsFragment.java | 8 ++++++-- 3 files changed, 19 insertions(+), 6 deletions(-) diff --git a/src/com/android/settings/print/PrintJobSettingsFragment.java b/src/com/android/settings/print/PrintJobSettingsFragment.java index 08bbd9781aa..dc7df52fc11 100644 --- a/src/com/android/settings/print/PrintJobSettingsFragment.java +++ b/src/com/android/settings/print/PrintJobSettingsFragment.java @@ -26,11 +26,13 @@ import android.print.PrintManager.PrintJobStateChangeListener; import android.support.v7.preference.Preference; import android.text.TextUtils; import android.text.format.DateUtils; +import android.view.LayoutInflater; import android.view.Menu; import android.view.MenuInflater; import android.view.MenuItem; import android.view.View; +import android.view.ViewGroup; import com.android.internal.logging.MetricsProto.MetricsEvent; import com.android.settings.R; import com.android.settings.SettingsPreferenceFragment; @@ -70,8 +72,9 @@ public class PrintJobSettingsFragment extends SettingsPreferenceFragment { } @Override - public void onCreate(Bundle icicle) { - super.onCreate(icicle); + public View onCreateView(LayoutInflater inflater, ViewGroup container, + Bundle savedInstanceState) { + View view = super.onCreateView(inflater, container, savedInstanceState); addPreferencesFromResource(R.xml.print_job_settings); mPrintJobPreference = findPreference(PRINT_JOB_PREFERENCE); @@ -86,6 +89,8 @@ public class PrintJobSettingsFragment extends SettingsPreferenceFragment { processArguments(); setHasOptionsMenu(true); + + return view; } @Override diff --git a/src/com/android/settings/print/PrintServiceSettingsFragment.java b/src/com/android/settings/print/PrintServiceSettingsFragment.java index 42d8a8d3465..6ef900ba37e 100644 --- a/src/com/android/settings/print/PrintServiceSettingsFragment.java +++ b/src/com/android/settings/print/PrintServiceSettingsFragment.java @@ -39,6 +39,7 @@ import android.printservice.PrintServiceInfo; import android.text.TextUtils; import android.util.Log; import android.util.TypedValue; +import android.view.LayoutInflater; import android.view.Menu; import android.view.MenuInflater; import android.view.MenuItem; @@ -132,8 +133,9 @@ public class PrintServiceSettingsFragment extends SettingsPreferenceFragment } @Override - public void onCreate(Bundle icicle) { - super.onCreate(icicle); + public View onCreateView(LayoutInflater inflater, ViewGroup container, + Bundle savedInstanceState) { + View root = super.onCreateView(inflater, container, savedInstanceState); mServiceEnabled = getArguments().getBoolean(PrintSettingsFragment.EXTRA_CHECKED); @@ -141,6 +143,8 @@ public class PrintServiceSettingsFragment extends SettingsPreferenceFragment if (!TextUtils.isEmpty(title)) { getActivity().setTitle(title); } + + return root; } @Override diff --git a/src/com/android/settings/print/PrintSettingsFragment.java b/src/com/android/settings/print/PrintSettingsFragment.java index 82a18ee80a5..49d254c41e1 100644 --- a/src/com/android/settings/print/PrintSettingsFragment.java +++ b/src/com/android/settings/print/PrintSettingsFragment.java @@ -43,6 +43,7 @@ import android.support.v7.preference.PreferenceScreen; import android.text.TextUtils; import android.text.format.DateUtils; import android.util.Log; +import android.view.LayoutInflater; import android.view.View; import android.view.View.OnClickListener; import android.view.ViewGroup; @@ -104,8 +105,9 @@ public class PrintSettingsFragment extends ProfileSettingsPreferenceFragment } @Override - public void onCreate(Bundle icicle) { - super.onCreate(icicle); + public View onCreateView(LayoutInflater inflater, ViewGroup container, + Bundle savedInstanceState) { + View root = super.onCreateView(inflater, container, savedInstanceState); addPreferencesFromResource(R.xml.print_settings); mActivePrintJobsCategory = (PreferenceCategory) findPreference( @@ -119,6 +121,8 @@ public class PrintSettingsFragment extends ProfileSettingsPreferenceFragment mPrintServicesController = new PrintServicesController(); getLoaderManager().initLoader(LOADER_ID_PRINT_SERVICES, null, mPrintServicesController); + + return root; } @Override