Micro cleanup and refactoring

- remove unused dimen resources
- remove unused and refactor codes

Test: make RunSettingsRoboTests -j
Change-Id: Ic2a5b16323667741d048e20e3db8da3fb4dc348a
This commit is contained in:
Jason Chiu
2019-04-03 13:49:26 +08:00
parent 8cc371e643
commit b335e96d2a
3 changed files with 1 additions and 24 deletions

View File

@@ -319,14 +319,6 @@
<dimen name="screen_pinning_padding_start">64dp</dimen> <dimen name="screen_pinning_padding_start">64dp</dimen>
<dimen name="screen_pinning_padding_end">64dp</dimen> <dimen name="screen_pinning_padding_end">64dp</dimen>
<!-- Suggestion/condition header padding -->
<dimen name="suggestion_condition_header_padding_collapsed">10dp</dimen>
<dimen name="suggestion_condition_header_padding_expanded">5dp</dimen>
<dimen name="condition_header_height">36dp</dimen>
<!-- Condition cards size and padding -->
<dimen name="condition_card_elevation">2dp</dimen>
<!-- Padding for the reset screens --> <!-- Padding for the reset screens -->
<dimen name="reset_checkbox_padding_end">8dp</dimen> <dimen name="reset_checkbox_padding_end">8dp</dimen>
<dimen name="reset_checkbox_title_padding_top">12dp</dimen> <dimen name="reset_checkbox_title_padding_top">12dp</dimen>
@@ -334,12 +326,6 @@
<dimen name="reset_checkbox_title_text_size">18sp</dimen> <dimen name="reset_checkbox_title_text_size">18sp</dimen>
<dimen name="reset_checkbox_summary_text_size">14sp</dimen> <dimen name="reset_checkbox_summary_text_size">14sp</dimen>
<!-- Bottombar size and padding -->
<dimen name="homepage_bottomsheet_height">90dp</dimen>
<dimen name="homepage_bottombar_height">56dp</dimen>
<dimen name="homepage_bottombar_top_margin">34dp</dimen>
<dimen name="homepage_bottombar_fab_cradle">68dp</dimen>
<!-- Homepage cards size and padding --> <!-- Homepage cards size and padding -->
<dimen name="homepage_card_icon_size">24dp</dimen> <dimen name="homepage_card_icon_size">24dp</dimen>
<dimen name="homepage_card_vertical_margin">8dp</dimen> <dimen name="homepage_card_vertical_margin">8dp</dimen>
@@ -369,8 +355,6 @@
<dimen name="homepage_card_dismissal_button_margin_end">6dp</dimen> <dimen name="homepage_card_dismissal_button_margin_end">6dp</dimen>
<!-- Horizontal divider size and margin --> <!-- Horizontal divider size and margin -->
<dimen name="horizontal_divider_margin_top">4dp</dimen>
<dimen name="horizontal_divider_margin_bottom">8dp</dimen>
<dimen name="horizontal_divider_height">.75dp</dimen> <dimen name="horizontal_divider_height">.75dp</dimen>
<!-- Vertical divider size --> <!-- Vertical divider size -->

View File

@@ -84,7 +84,7 @@ public class SettingsHomepageActivity extends SettingsBaseActivity {
View.SYSTEM_UI_FLAG_LAYOUT_FULLSCREEN View.SYSTEM_UI_FLAG_LAYOUT_FULLSCREEN
); );
getWindow().setStatusBarColor(getResources().getColor(R.color.homepage_status_bar_color)); getWindow().setStatusBarColor(getColor(R.color.homepage_status_bar_color));
} }
@VisibleForTesting @VisibleForTesting

View File

@@ -17,14 +17,10 @@
package com.android.settings.homepage.contextualcards.slices; package com.android.settings.homepage.contextualcards.slices;
import android.content.Context; import android.content.Context;
import android.util.Log;
import android.view.View; import android.view.View;
import androidx.annotation.NonNull;
import androidx.recyclerview.widget.RecyclerView; import androidx.recyclerview.widget.RecyclerView;
import androidx.slice.Slice; import androidx.slice.Slice;
import androidx.slice.SliceItem;
import androidx.slice.widget.EventInfo;
import androidx.slice.widget.SliceView; import androidx.slice.widget.SliceView;
import com.android.settings.R; import com.android.settings.R;
@@ -32,13 +28,10 @@ import com.android.settings.homepage.contextualcards.ContextualCard;
import com.android.settings.homepage.contextualcards.ContextualCardFeatureProvider; import com.android.settings.homepage.contextualcards.ContextualCardFeatureProvider;
import com.android.settings.overlay.FeatureFactory; import com.android.settings.overlay.FeatureFactory;
import java.util.Set;
/** /**
* Card renderer helper for {@link ContextualCard} built as slice full card. * Card renderer helper for {@link ContextualCard} built as slice full card.
*/ */
class SliceFullCardRendererHelper { class SliceFullCardRendererHelper {
private static final String TAG = "SliceFCRendererHelper";
private final Context mContext; private final Context mContext;