diff options
author | Fan Zhang <zhfan@google.com> | 2018-11-27 13:06:44 -0800 |
---|---|---|
committer | Fan Zhang <zhfan@google.com> | 2018-11-29 11:37:00 -0800 |
commit | 8e14ca0058d65174de1c48e6c074dd3fc1e02fe1 (patch) | |
tree | 68e665b287fe83af0dddf1ee1dd1d752529d2a9e /src/com/android/settings/SettingsActivity.java | |
parent | 662b209b15151f3f79e9d25d55282c55fc210ba6 (diff) | |
download | packages_apps_Settings-8e14ca0058d65174de1c48e6c074dd3fc1e02fe1.tar.gz packages_apps_Settings-8e14ca0058d65174de1c48e6c074dd3fc1e02fe1.tar.bz2 packages_apps_Settings-8e14ca0058d65174de1c48e6c074dd3fc1e02fe1.zip |
Remove obsolete logic in SettingsActivity.
mIsShowingDashboard is always false, it used to be true when we are
displaying homepage, but now homepage is hosted in a entirely different
activity.
so all related logic can now be removed.
Test: robotests
Misc clean up: remove unused colors
Test: rebuild, color-lint
Change-Id: I1e1628c1e9606c2b7dc40ef3c21d4ed1391a8c03
Diffstat (limited to 'src/com/android/settings/SettingsActivity.java')
-rw-r--r-- | src/com/android/settings/SettingsActivity.java | 26 |
1 files changed, 5 insertions, 21 deletions
diff --git a/src/com/android/settings/SettingsActivity.java b/src/com/android/settings/SettingsActivity.java index 07f59e2b4d..222c18c77b 100644 --- a/src/com/android/settings/SettingsActivity.java +++ b/src/com/android/settings/SettingsActivity.java @@ -38,7 +38,6 @@ import android.view.View; import android.view.View.OnClickListener; import android.view.ViewGroup; import android.widget.Button; -import android.widget.Toolbar; import androidx.annotation.Nullable; import androidx.annotation.VisibleForTesting; @@ -169,8 +168,6 @@ public class SettingsActivity extends SettingsBaseActivity private Button mNextButton; - private boolean mIsShowingDashboard; - private ViewGroup mContent; // Categories @@ -241,9 +238,6 @@ public class SettingsActivity extends SettingsBaseActivity // Getting Intent properties can only be done after the super.onCreate(...) final String initialFragmentName = intent.getStringExtra(EXTRA_SHOW_FRAGMENT); - mIsShowingDashboard = TextUtils.equals( - SettingsActivity.class.getName(), intent.getComponent().getClassName()); - // This is a "Sub Settings" when: // - this is a real SubSettings // - or :settings:show_fragment_as_subsetting is passed to the Intent @@ -256,8 +250,7 @@ public class SettingsActivity extends SettingsBaseActivity setTheme(R.style.Theme_SubSettings); } - setContentView(mIsShowingDashboard ? - R.layout.settings_main_dashboard : R.layout.settings_main_prefs); + setContentView(R.layout.settings_main_prefs); mContent = findViewById(R.id.main_content); @@ -276,25 +269,16 @@ public class SettingsActivity extends SettingsBaseActivity setTitleFromBackStack(); } } else { - launchSettingFragment(initialFragmentName, isSubSettings, intent); + launchSettingFragment(initialFragmentName, intent); } final boolean deviceProvisioned = Utils.isDeviceProvisioned(this); - if (mIsShowingDashboard) { - findViewById(R.id.search_bar).setVisibility( - deviceProvisioned ? View.VISIBLE : View.INVISIBLE); - findViewById(R.id.action_bar).setVisibility(View.GONE); - final Toolbar toolbar = findViewById(R.id.search_action_bar); - setActionBar(toolbar); - FeatureFactory.getFactory(this).getSearchFeatureProvider() - .initSearchToolbar(this, toolbar); - } ActionBar actionBar = getActionBar(); if (actionBar != null) { actionBar.setDisplayHomeAsUpEnabled(deviceProvisioned); actionBar.setHomeButtonEnabled(deviceProvisioned); - actionBar.setDisplayShowTitleEnabled(!mIsShowingDashboard); + actionBar.setDisplayShowTitleEnabled(true); } mSwitchBar = findViewById(R.id.switch_bar); if (mSwitchBar != null) { @@ -372,8 +356,8 @@ public class SettingsActivity extends SettingsBaseActivity } @VisibleForTesting - void launchSettingFragment(String initialFragmentName, boolean isSubSettings, Intent intent) { - if (!mIsShowingDashboard && initialFragmentName != null) { + void launchSettingFragment(String initialFragmentName, Intent intent) { + if (initialFragmentName != null) { setTitleFromIntent(intent); Bundle initialArguments = intent.getBundleExtra(EXTRA_SHOW_FRAGMENT_ARGUMENTS); |