Remove drawer from settings
Change-Id: Ib11376206cf8366c86c427104c47740c3ca8b0d0
Fix: 35809663
Test: make RunSettingsRoboTests
diff --git a/src/com/android/settings/SettingsActivity.java b/src/com/android/settings/SettingsActivity.java
index 00bdf57..84901fb 100644
--- a/src/com/android/settings/SettingsActivity.java
+++ b/src/com/android/settings/SettingsActivity.java
@@ -58,7 +58,6 @@
import com.android.settings.core.gateway.SettingsGateway;
import com.android.settings.core.instrumentation.MetricsFeatureProvider;
import com.android.settings.core.instrumentation.SharedPreferencesLogger;
-import com.android.settings.dashboard.DashboardContainerFragment;
import com.android.settings.dashboard.DashboardFeatureProvider;
import com.android.settings.dashboard.DashboardSummary;
import com.android.settings.dashboard.SearchResultsSummary;
@@ -154,6 +153,7 @@
public static final String EXTRA_SHOW_FRAGMENT_AS_SUBSETTING =
":settings:show_fragment_as_subsetting";
+ @Deprecated
public static final String EXTRA_HIDE_DRAWER = ":settings:hide_drawer";
public static final String META_DATA_KEY_FRAGMENT_CLASS =
@@ -367,9 +367,6 @@
if (intent.hasExtra(EXTRA_UI_OPTIONS)) {
getWindow().setUiOptions(intent.getIntExtra(EXTRA_UI_OPTIONS, 0));
}
- if (intent.getBooleanExtra(EXTRA_HIDE_DRAWER, false)) {
- setIsDrawerPresent(false);
- }
mDevelopmentPreferences = getSharedPreferences(DevelopmentSettings.PREF_FILE,
Context.MODE_PRIVATE);
@@ -461,17 +458,8 @@
mDisplaySearch = true;
mInitialTitleResId = R.string.dashboard_title;
- // add argument to indicate which settings tab should be initially selected
- final Bundle args = new Bundle();
- final String extraName = DashboardContainerFragment.EXTRA_SELECT_SETTINGS_TAB;
- args.putString(extraName, intent.getStringExtra(extraName));
- if (isDashboardFeatureEnabled()) {
- switchToFragment(DashboardSummary.class.getName(), args, false, false,
+ switchToFragment(DashboardSummary.class.getName(), null /* args */, false, false,
mInitialTitleResId, mInitialTitle, false);
- } else {
- switchToFragment(DashboardContainerFragment.class.getName(), args, false, false,
- mInitialTitleResId, mInitialTitle, false);
- }
}
}
@@ -957,12 +945,7 @@
if (UserHandle.MU_ENABLED && !isAdmin) {
// When on restricted users, disable all extra categories (but only the settings ones).
- List<DashboardCategory> categories;
- if (isDashboardFeatureEnabled()) {
- categories = mDashboardFeatureProvider.getAllCategories();
- } else {
- categories = getDashboardCategories();
- }
+ final List<DashboardCategory> categories = mDashboardFeatureProvider.getAllCategories();
for (DashboardCategory category : categories) {
for (Tile tile : category.tiles) {
@@ -1017,15 +1000,6 @@
}
}
- @Override
- protected boolean isDashboardFeatureEnabled() {
- if (mDashboardFeatureProvider == null) {
- mDashboardFeatureProvider =
- FeatureFactory.getFactory(this).getDashboardFeatureProvider(this);
- }
- return mDashboardFeatureProvider.isEnabled();
- }
-
// give subclasses access to the Next button
public boolean hasNextButton() {
return mNextButton != null;
@@ -1085,16 +1059,6 @@
}
@Override
- protected void onTileClicked(Tile tile) {
- if (mIsShowingDashboard) {
- // If on dashboard, don't finish so the back comes back to here.
- openTile(tile);
- } else {
- super.onTileClicked(tile);
- }
- }
-
- @Override
public void onProfileTileOpen() {
if (!mIsShowingDashboard) {
finish();
diff --git a/src/com/android/settings/accessibility/AccessibilitySettingsForSetupWizardActivity.java b/src/com/android/settings/accessibility/AccessibilitySettingsForSetupWizardActivity.java
index cf72383..b820cc2 100644
--- a/src/com/android/settings/accessibility/AccessibilitySettingsForSetupWizardActivity.java
+++ b/src/com/android/settings/accessibility/AccessibilitySettingsForSetupWizardActivity.java
@@ -46,7 +46,6 @@
// Finish configuring the content view.
getActionBar().setDisplayHomeAsUpEnabled(true);
- setIsDrawerPresent(false);
}
@Override
diff --git a/src/com/android/settings/dashboard/DashboardContainerFragment.java b/src/com/android/settings/dashboard/DashboardContainerFragment.java
deleted file mode 100644
index b3ee808..0000000
--- a/src/com/android/settings/dashboard/DashboardContainerFragment.java
+++ /dev/null
@@ -1,194 +0,0 @@
-/*
- * Copyright (C) 2016 The Android Open Source Project
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package com.android.settings.dashboard;
-
-import android.app.Activity;
-import android.app.Fragment;
-import android.app.FragmentManager;
-import android.content.Context;
-import android.os.Bundle;
-import android.support.v13.app.FragmentPagerAdapter;
-import android.text.TextUtils;
-import android.view.LayoutInflater;
-import android.view.View;
-import android.view.ViewGroup;
-
-import com.android.internal.logging.nano.MetricsProto;
-import com.android.settings.R;
-import com.android.settings.SettingsActivity;
-import com.android.settings.core.InstrumentedFragment;
-import com.android.settings.core.instrumentation.MetricsFeatureProvider;
-import com.android.settings.overlay.FeatureFactory;
-import com.android.settings.overlay.SupportFeatureProvider;
-import com.android.settings.widget.RtlCompatibleViewPager;
-import com.android.settings.widget.SlidingTabLayout;
-import com.android.settingslib.drawer.SettingsDrawerActivity;
-
-/**
- * Container for Dashboard fragments.
- */
-public final class DashboardContainerFragment extends InstrumentedFragment {
-
- public static final String EXTRA_SELECT_SETTINGS_TAB = ":settings:select_settings_tab";
-
- private static final String ARG_SUPPORT_TAB = "SUPPORT";
- private static final String ARG_SUMMARY_TAB = "SUMMARY";
- private static final int INDEX_SUMMARY_FRAGMENT = 0;
- private static final int INDEX_SUPPORT_FRAGMENT = 1;
-
- private RtlCompatibleViewPager mViewPager;
- private View mHeaderView;
- private DashboardViewPagerAdapter mPagerAdapter;
-
- @Override
- public int getMetricsCategory() {
- return MetricsProto.MetricsEvent.DASHBOARD_CONTAINER;
- }
-
- @Override
- public void onCreate(Bundle savedInstanceState) {
- super.onCreate(savedInstanceState);
- setHasOptionsMenu(true);
- }
-
- @Override
- public View onCreateView(LayoutInflater inflater, ViewGroup parent, Bundle savedInstanceState) {
- final View content = inflater.inflate(R.layout.dashboard_container, parent, false);
- mViewPager = (RtlCompatibleViewPager) content.findViewById(R.id.pager);
- mPagerAdapter = new DashboardViewPagerAdapter(getContext(),
- getChildFragmentManager(), mViewPager);
- mViewPager.setAdapter(mPagerAdapter);
- mViewPager.addOnPageChangeListener(
- new TabChangeListener((SettingsActivity) getActivity(), mMetricsFeatureProvider));
-
- // check if support tab needs to be selected
- final String selectedTab = getArguments().
- getString(EXTRA_SELECT_SETTINGS_TAB, ARG_SUMMARY_TAB);
- if (TextUtils.equals(selectedTab, ARG_SUPPORT_TAB)) {
- mViewPager.setCurrentItem(INDEX_SUPPORT_FRAGMENT);
- } else {
- mViewPager.setCurrentItem(INDEX_SUMMARY_FRAGMENT);
- }
-
- mHeaderView = inflater.inflate(R.layout.dashboard_container_header, parent, false);
- ((SlidingTabLayout) mHeaderView).setViewPager(mViewPager);
- return content;
- }
-
- @Override
- public void onResume() {
- super.onResume();
- if (mPagerAdapter.getCount() > 1) {
- final Activity activity = getActivity();
- if (activity instanceof SettingsDrawerActivity) {
- ((SettingsDrawerActivity) getActivity()).setContentHeaderView(mHeaderView);
- }
- }
- }
-
- private static final class DashboardViewPagerAdapter extends FragmentPagerAdapter {
-
- private final Context mContext;
- private final SupportFeatureProvider mSupportFeatureProvider;
- private final RtlCompatibleViewPager mViewPager;
-
- public DashboardViewPagerAdapter(Context context, FragmentManager fragmentManager,
- RtlCompatibleViewPager viewPager) {
- super(fragmentManager);
- mContext = context;
- mSupportFeatureProvider =
- FeatureFactory.getFactory(context).getSupportFeatureProvider(context);
- mViewPager = viewPager;
- }
-
- @Override
- public CharSequence getPageTitle(int position) {
- switch (position) {
- case INDEX_SUMMARY_FRAGMENT:
- return mContext.getString(R.string.page_tab_title_summary);
- case INDEX_SUPPORT_FRAGMENT:
- return mContext.getString(R.string.page_tab_title_support);
- }
- return super.getPageTitle(position);
- }
-
- @Override
- public Fragment getItem(int position) {
- switch (position) {
- case INDEX_SUMMARY_FRAGMENT:
- return new DashboardSummary();
- case INDEX_SUPPORT_FRAGMENT:
- return new SupportFragment();
- default:
- throw new IllegalArgumentException(
- String.format(
- "Position %d does not map to a valid dashboard fragment",
- position));
- }
- }
-
- @Override
- public Object instantiateItem(ViewGroup container, int position) {
- return super.instantiateItem(container,
- mViewPager.getRtlAwareIndex(position));
- }
-
- @Override
- public int getCount() {
- return mSupportFeatureProvider == null ? 1 : 2;
- }
- }
-
- private static final class TabChangeListener
- implements RtlCompatibleViewPager.OnPageChangeListener {
-
- private final SettingsActivity mActivity;
- private final MetricsFeatureProvider mMetricsFeatureProvider;
-
- public TabChangeListener(SettingsActivity activity,
- MetricsFeatureProvider metricsFeatureProvider) {
- mActivity = activity;
- mMetricsFeatureProvider = metricsFeatureProvider;
- }
-
- @Override
- public void onPageScrolled(int position, float positionOffset, int positionOffsetPixels) {
- // Do nothing.
- }
-
- @Override
- public void onPageScrollStateChanged(int state) {
- // Do nothing
- }
-
- @Override
- public void onPageSelected(int position) {
- switch (position) {
- case INDEX_SUMMARY_FRAGMENT:
- mMetricsFeatureProvider.action(
- mActivity, MetricsProto.MetricsEvent.ACTION_SELECT_SUMMARY);
- mActivity.setDisplaySearchMenu(true);
- break;
- case INDEX_SUPPORT_FRAGMENT:
- mMetricsFeatureProvider.action(
- mActivity, MetricsProto.MetricsEvent.ACTION_SELECT_SUPPORT_FRAGMENT);
- mActivity.setDisplaySearchMenu(false);
- break;
- }
- }
- }
-}