Snap for 5214914 from 1517be0b7f to qt-release
Change-Id: Id6d30655ddcaaea2e367020891f1205889a2de96
This commit is contained in:
@@ -139,7 +139,11 @@
|
||||
android:label="@string/network_settings_title"
|
||||
android:theme="@style/Theme.Settings.Home"
|
||||
android:launchMode="singleTask">
|
||||
<!-- TODO(b/114749736): add intent filter here and disable the one in telephony -->
|
||||
<intent-filter android:priority="1">
|
||||
<action android:name="android.settings.NETWORK_OPERATOR_SETTINGS" />
|
||||
<action android:name="android.settings.DATA_ROAMING_SETTINGS" />
|
||||
<category android:name="android.intent.category.DEFAULT" />
|
||||
</intent-filter>
|
||||
</activity>
|
||||
|
||||
<!-- Alias for launcher activity only, as this belongs to each profile. -->
|
||||
|
||||
@@ -14,60 +14,73 @@
|
||||
limitations under the License.
|
||||
-->
|
||||
|
||||
<LinearLayout
|
||||
<androidx.coordinatorlayout.widget.CoordinatorLayout
|
||||
xmlns:android="http://schemas.android.com/apk/res/android"
|
||||
xmlns:settings="http://schemas.android.com/apk/res-auto"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="match_parent"
|
||||
android:orientation="vertical">
|
||||
|
||||
<FrameLayout
|
||||
<androidx.core.widget.NestedScrollView
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="0dp"
|
||||
android:layout_weight="1">
|
||||
android:layout_height="match_parent"
|
||||
android:fillViewport="true"
|
||||
settings:layout_behavior="com.android.settings.widget.FloatingAppBarScrollingViewBehavior">
|
||||
|
||||
<LinearLayout
|
||||
android:id="@+id/list_container"
|
||||
<FrameLayout
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="match_parent"
|
||||
android:orientation="vertical"
|
||||
android:visibility="gone">
|
||||
android:paddingTop="@dimen/app_bar_height">
|
||||
|
||||
<FrameLayout
|
||||
android:id="@+id/pinned_header"
|
||||
<LinearLayout
|
||||
android:id="@+id/list_container"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content" />
|
||||
android:layout_height="match_parent"
|
||||
android:orientation="vertical"
|
||||
android:visibility="gone">
|
||||
|
||||
<FrameLayout
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="0px"
|
||||
android:layout_weight="1">
|
||||
|
||||
<androidx.recyclerview.widget.RecyclerView
|
||||
android:id="@+id/apps_list"
|
||||
<FrameLayout
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="match_parent"
|
||||
settings:fastScrollEnabled="true"
|
||||
settings:fastScrollHorizontalThumbDrawable="@drawable/thumb_drawable"
|
||||
settings:fastScrollHorizontalTrackDrawable="@drawable/line_drawable"
|
||||
settings:fastScrollVerticalThumbDrawable="@drawable/thumb_drawable"
|
||||
settings:fastScrollVerticalTrackDrawable="@drawable/line_drawable"/>
|
||||
android:layout_height="match_parent">
|
||||
|
||||
<TextView
|
||||
android:id="@android:id/empty"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="match_parent"
|
||||
android:gravity="center"
|
||||
android:text="@string/no_applications"
|
||||
android:textAppearance="?android:attr/textAppearanceLarge"
|
||||
android:visibility="invisible" />
|
||||
<androidx.recyclerview.widget.RecyclerView
|
||||
android:id="@+id/apps_list"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="match_parent"
|
||||
settings:fastScrollEnabled="true"
|
||||
settings:fastScrollHorizontalThumbDrawable="@drawable/thumb_drawable"
|
||||
settings:fastScrollHorizontalTrackDrawable="@drawable/line_drawable"
|
||||
settings:fastScrollVerticalThumbDrawable="@drawable/thumb_drawable"
|
||||
settings:fastScrollVerticalTrackDrawable="@drawable/line_drawable"/>
|
||||
|
||||
</FrameLayout>
|
||||
<TextView
|
||||
android:id="@android:id/empty"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="match_parent"
|
||||
android:gravity="center"
|
||||
android:layout_gravity="center"
|
||||
android:text="@string/no_applications"
|
||||
android:textAppearance="?android:attr/textAppearanceLarge"
|
||||
android:visibility="invisible"/>
|
||||
|
||||
</LinearLayout>
|
||||
</FrameLayout>
|
||||
|
||||
<include layout="@layout/loading_container" />
|
||||
</LinearLayout>
|
||||
|
||||
</FrameLayout>
|
||||
<include layout="@layout/loading_container"/>
|
||||
|
||||
</LinearLayout>
|
||||
</FrameLayout>
|
||||
|
||||
</androidx.core.widget.NestedScrollView>
|
||||
|
||||
<com.google.android.material.appbar.AppBarLayout
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content">
|
||||
<FrameLayout
|
||||
android:id="@+id/pinned_header"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
settings:layout_scrollFlags="scroll|enterAlways"/>
|
||||
</com.google.android.material.appbar.AppBarLayout>
|
||||
|
||||
</androidx.coordinatorlayout.widget.CoordinatorLayout>
|
||||
|
||||
@@ -114,6 +114,8 @@
|
||||
<dimen name="wifi_assistant_height">182dp</dimen>
|
||||
<dimen name="wifi_assistant_image_top">32dp</dimen>
|
||||
<dimen name="wifi_assistant_image_start">24dp</dimen>
|
||||
<!-- appbar height is equal search bar height (48dp) plus search bar top and bottom margin -->
|
||||
<dimen name="app_bar_height">80dp</dimen>
|
||||
|
||||
<!-- CryptKeeper top margin for password/pin screen -->
|
||||
<dimen name="crypt_keeper_password_top_margin">88dip</dimen>
|
||||
@@ -354,7 +356,7 @@
|
||||
<dimen name="homepage_condition_full_card_padding_bottom">12dp</dimen>
|
||||
<dimen name="homepage_condition_header_padding_top">10dp</dimen>
|
||||
<dimen name="homepage_condition_header_padding_bottom">10dp</dimen>
|
||||
<dimen name="homepage_condition_header_icons_margin_start">24dp</dimen>
|
||||
<dimen name="homepage_condition_header_icons_margin_start">16dp</dimen>
|
||||
<dimen name="homepage_condition_header_indicator_padding_top">4dp</dimen>
|
||||
<dimen name="homepage_condition_header_indicator_padding_start">16dp</dimen>
|
||||
<dimen name="homepage_condition_header_indicator_padding_end">16dp</dimen>
|
||||
|
||||
@@ -454,7 +454,8 @@
|
||||
|
||||
<style name="ConditionFullCardBorderlessButton"
|
||||
parent="@style/ConditionCardBorderlessButton">
|
||||
<item name="android:textAlignment">viewEnd</item>
|
||||
<item name="android:minWidth">24dp</item>
|
||||
<item name="android:layout_marginStart">20dp</item>
|
||||
</style>
|
||||
|
||||
<style name="ContextualCardDismissalButton"
|
||||
|
||||
@@ -107,12 +107,12 @@ public class ContextualCardLoader extends AsyncLoaderCompat<List<ContextualCard>
|
||||
}
|
||||
}
|
||||
}
|
||||
return getFinalDisplayableCards(result);
|
||||
return getDisplayableCards(result);
|
||||
}
|
||||
|
||||
// Get final displayed cards and log what cards will be displayed/hidden
|
||||
@VisibleForTesting
|
||||
List<ContextualCard> getFinalDisplayableCards(List<ContextualCard> candidates) {
|
||||
List<ContextualCard> getDisplayableCards(List<ContextualCard> candidates) {
|
||||
final List<ContextualCard> eligibleCards = filterEligibleCards(candidates);
|
||||
final List<ContextualCard> visibleCards = new ArrayList<>();
|
||||
final List<ContextualCard> hiddenCards = new ArrayList<>();
|
||||
|
||||
@@ -84,7 +84,11 @@ public class ContextualCardLookupTable {
|
||||
LegacySuggestionContextualCardController.class,
|
||||
LegacySuggestionContextualCardRenderer.class));
|
||||
add(new ControllerRendererMapping(CardType.SLICE,
|
||||
SliceContextualCardRenderer.VIEW_TYPE,
|
||||
SliceContextualCardRenderer.VIEW_TYPE_FULL_WIDTH,
|
||||
SliceContextualCardController.class,
|
||||
SliceContextualCardRenderer.class));
|
||||
add(new ControllerRendererMapping(CardType.SLICE,
|
||||
SliceContextualCardRenderer.VIEW_TYPE_HALF_WIDTH,
|
||||
SliceContextualCardController.class,
|
||||
SliceContextualCardRenderer.class));
|
||||
add(new ControllerRendererMapping(CardType.CONDITIONAL_FOOTER,
|
||||
|
||||
@@ -17,6 +17,7 @@
|
||||
package com.android.settings.homepage.contextualcards;
|
||||
|
||||
import static com.android.settings.homepage.contextualcards.ContextualCardLoader.CARD_CONTENT_LOADER_ID;
|
||||
import static com.android.settings.intelligence.ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE;
|
||||
|
||||
import static java.util.stream.Collectors.groupingBy;
|
||||
|
||||
@@ -172,7 +173,8 @@ public class ContextualCardManager implements ContextualCardLoader.CardContentLo
|
||||
|
||||
//replace with the new data
|
||||
mContextualCards.clear();
|
||||
mContextualCards.addAll(sortCards(allCards));
|
||||
final List<ContextualCard> sortedCards = sortCards(allCards);
|
||||
mContextualCards.addAll(assignCardWidth(sortedCards));
|
||||
|
||||
loadCardControllers();
|
||||
|
||||
@@ -224,6 +226,24 @@ public class ContextualCardManager implements ContextualCardLoader.CardContentLo
|
||||
mListener = listener;
|
||||
}
|
||||
|
||||
@VisibleForTesting
|
||||
List<ContextualCard> assignCardWidth(List<ContextualCard> cards) {
|
||||
final List<ContextualCard> result = new ArrayList<>(cards);
|
||||
// Shows as half cards if 2 suggestion type of cards are next to each other.
|
||||
// Shows as full card if 1 suggestion type of card lives alone.
|
||||
for (int index = 1; index < result.size(); index++) {
|
||||
final ContextualCard previous = result.get(index - 1);
|
||||
final ContextualCard current = result.get(index);
|
||||
if (current.getCategory() == SUGGESTION_VALUE
|
||||
&& previous.getCategory() == SUGGESTION_VALUE) {
|
||||
result.set(index - 1, previous.mutate().setIsHalfWidth(true).build());
|
||||
result.set(index, current.mutate().setIsHalfWidth(true).build());
|
||||
index++;
|
||||
}
|
||||
}
|
||||
return result;
|
||||
}
|
||||
|
||||
private List<ContextualCard> getCardsToKeep(List<ContextualCard> cards) {
|
||||
if (mSavedCards != null) {
|
||||
//screen rotate
|
||||
|
||||
@@ -40,7 +40,7 @@ import java.util.stream.Collectors;
|
||||
*/
|
||||
public class ConditionContextualCardController implements ContextualCardController,
|
||||
ConditionListener, LifecycleObserver, OnStart, OnStop {
|
||||
public static final int EXPANDING_THRESHOLD = 2;
|
||||
public static final int EXPANDING_THRESHOLD = 0;
|
||||
|
||||
private static final double UNSUPPORTED_RANKING = -99999.0;
|
||||
private static final String TAG = "ConditionCtxCardCtrl";
|
||||
|
||||
@@ -82,8 +82,8 @@ public class BatteryFixSlice implements CustomSliceable {
|
||||
return buildBatteryGoodSlice(sliceBuilder, true);
|
||||
}
|
||||
|
||||
final List<BatteryTip> batteryTips = SliceBackgroundWorker.getInstance(mContext,
|
||||
this).getResults();
|
||||
final SliceBackgroundWorker worker = SliceBackgroundWorker.getInstance(getUri());
|
||||
final List<BatteryTip> batteryTips = worker != null ? worker.getResults() : null;
|
||||
|
||||
if (batteryTips == null) {
|
||||
// Because we need wait slice background worker return data
|
||||
|
||||
@@ -26,7 +26,6 @@ import android.view.View;
|
||||
import android.widget.Button;
|
||||
import android.widget.ViewFlipper;
|
||||
|
||||
import androidx.annotation.NonNull;
|
||||
import androidx.annotation.VisibleForTesting;
|
||||
import androidx.lifecycle.Lifecycle;
|
||||
import androidx.lifecycle.LifecycleObserver;
|
||||
@@ -35,40 +34,40 @@ import androidx.lifecycle.LiveData;
|
||||
import androidx.lifecycle.OnLifecycleEvent;
|
||||
import androidx.recyclerview.widget.RecyclerView;
|
||||
import androidx.slice.Slice;
|
||||
import androidx.slice.SliceItem;
|
||||
import androidx.slice.widget.EventInfo;
|
||||
import androidx.slice.widget.SliceLiveData;
|
||||
import androidx.slice.widget.SliceView;
|
||||
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.homepage.contextualcards.CardContentProvider;
|
||||
import com.android.settings.homepage.contextualcards.ContextualCard;
|
||||
import com.android.settings.homepage.contextualcards.ContextualCardFeatureProvider;
|
||||
import com.android.settings.homepage.contextualcards.ContextualCardRenderer;
|
||||
import com.android.settings.homepage.contextualcards.ControllerRendererPool;
|
||||
import com.android.settings.overlay.FeatureFactory;
|
||||
|
||||
import java.util.Map;
|
||||
import java.util.Set;
|
||||
|
||||
/**
|
||||
* Card renderer for {@link ContextualCard} built as slices.
|
||||
* Card renderer for {@link ContextualCard} built as slice full card or slice half card.
|
||||
*/
|
||||
public class SliceContextualCardRenderer implements ContextualCardRenderer,
|
||||
SliceView.OnSliceActionListener, LifecycleObserver {
|
||||
public static final int VIEW_TYPE = R.layout.homepage_slice_tile;
|
||||
public class SliceContextualCardRenderer implements ContextualCardRenderer, LifecycleObserver {
|
||||
public static final int VIEW_TYPE_FULL_WIDTH = R.layout.homepage_slice_tile;
|
||||
public static final int VIEW_TYPE_HALF_WIDTH = R.layout.homepage_slice_half_tile;
|
||||
|
||||
private static final String TAG = "SliceCardRenderer";
|
||||
|
||||
@VisibleForTesting
|
||||
final Map<Uri, LiveData<Slice>> mSliceLiveDataMap;
|
||||
@VisibleForTesting
|
||||
final Set<SliceViewHolder> mFlippedCardSet;
|
||||
final Set<RecyclerView.ViewHolder> mFlippedCardSet;
|
||||
|
||||
private final Context mContext;
|
||||
private final LifecycleOwner mLifecycleOwner;
|
||||
private final ControllerRendererPool mControllerRendererPool;
|
||||
private final Set<ContextualCard> mCardSet;
|
||||
private final SliceFullCardRendererHelper mFullCardHelper;
|
||||
private final SliceHalfCardRendererHelper mHalfCardHelper;
|
||||
|
||||
//TODO(b/121303357): Remove isHalfWidth field from SliceContextualCardRenderer class.
|
||||
private boolean mIsHalfWidth;
|
||||
|
||||
public SliceContextualCardRenderer(Context context, LifecycleOwner lifecycleOwner,
|
||||
ControllerRendererPool controllerRendererPool) {
|
||||
@@ -79,21 +78,26 @@ public class SliceContextualCardRenderer implements ContextualCardRenderer,
|
||||
mCardSet = new ArraySet<>();
|
||||
mFlippedCardSet = new ArraySet<>();
|
||||
mLifecycleOwner.getLifecycle().addObserver(this);
|
||||
mFullCardHelper = new SliceFullCardRendererHelper(context);
|
||||
mHalfCardHelper = new SliceHalfCardRendererHelper(context);
|
||||
}
|
||||
|
||||
@Override
|
||||
public int getViewType(boolean isHalfWidth) {
|
||||
return VIEW_TYPE;
|
||||
mIsHalfWidth = isHalfWidth;
|
||||
return isHalfWidth? VIEW_TYPE_HALF_WIDTH : VIEW_TYPE_FULL_WIDTH;
|
||||
}
|
||||
|
||||
@Override
|
||||
public RecyclerView.ViewHolder createViewHolder(View view) {
|
||||
return new SliceViewHolder(view);
|
||||
if (mIsHalfWidth) {
|
||||
return mHalfCardHelper.createViewHolder(view);
|
||||
}
|
||||
return mFullCardHelper.createViewHolder(view);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void bindView(RecyclerView.ViewHolder holder, ContextualCard card) {
|
||||
final SliceViewHolder cardHolder = (SliceViewHolder) holder;
|
||||
final Uri uri = card.getSliceUri();
|
||||
//TODO(b/120629936): Take this out once blank card issue is fixed.
|
||||
Log.d(TAG, "bindView - uri = " + uri);
|
||||
@@ -103,10 +107,6 @@ public class SliceContextualCardRenderer implements ContextualCardRenderer,
|
||||
return;
|
||||
}
|
||||
|
||||
cardHolder.sliceView.setScrollable(false);
|
||||
cardHolder.sliceView.setTag(uri);
|
||||
//TODO(b/114009676): We will soon have a field to decide what slice mode we should set.
|
||||
cardHolder.sliceView.setMode(SliceView.MODE_LARGE);
|
||||
LiveData<Slice> sliceLiveData = mSliceLiveDataMap.get(uri);
|
||||
|
||||
if (sliceLiveData == null) {
|
||||
@@ -125,82 +125,58 @@ public class SliceContextualCardRenderer implements ContextualCardRenderer,
|
||||
//TODO(b/120629936): Take this out once blank card issue is fixed.
|
||||
Log.d(TAG, "Slice callback - uri = " + slice.getUri());
|
||||
}
|
||||
cardHolder.sliceView.setSlice(slice);
|
||||
if (holder.getItemViewType() == VIEW_TYPE_HALF_WIDTH) {
|
||||
mHalfCardHelper.bindView(holder, card, slice);
|
||||
} else {
|
||||
mFullCardHelper.bindView(holder, card, slice, mCardSet);
|
||||
}
|
||||
});
|
||||
|
||||
// Set this listener so we can log the interaction users make on the slice
|
||||
cardHolder.sliceView.setOnSliceActionListener(this);
|
||||
|
||||
// Customize slice view for Settings
|
||||
cardHolder.sliceView.showTitleItems(true);
|
||||
if (card.isLargeCard()) {
|
||||
cardHolder.sliceView.showHeaderDivider(true);
|
||||
cardHolder.sliceView.showActionDividers(true);
|
||||
if (holder.getItemViewType() == VIEW_TYPE_HALF_WIDTH) {
|
||||
initDismissalActions(holder, card, R.id.content);
|
||||
} else {
|
||||
initDismissalActions(holder, card, R.id.slice_view);
|
||||
}
|
||||
|
||||
initDismissalActions(cardHolder, card);
|
||||
}
|
||||
|
||||
private void initDismissalActions(SliceViewHolder cardHolder, ContextualCard card) {
|
||||
cardHolder.sliceView.setOnLongClickListener(v -> {
|
||||
cardHolder.viewFlipper.showNext();
|
||||
mFlippedCardSet.add(cardHolder);
|
||||
private void initDismissalActions(RecyclerView.ViewHolder holder, ContextualCard card,
|
||||
int initialViewId) {
|
||||
// initialView is the first view in the ViewFlipper.
|
||||
final View initialView = holder.itemView.findViewById(initialViewId);
|
||||
initialView.setOnLongClickListener(v -> {
|
||||
flipCardToDismissalView(holder);
|
||||
mFlippedCardSet.add(holder);
|
||||
return true;
|
||||
});
|
||||
|
||||
final Button btnKeep = cardHolder.itemView.findViewById(R.id.keep);
|
||||
final Button btnKeep = holder.itemView.findViewById(R.id.keep);
|
||||
btnKeep.setOnClickListener(v -> {
|
||||
cardHolder.resetCard();
|
||||
mFlippedCardSet.remove(cardHolder);
|
||||
mFlippedCardSet.remove(holder);
|
||||
resetCardView(holder);
|
||||
});
|
||||
|
||||
final Button btnRemove = cardHolder.itemView.findViewById(R.id.remove);
|
||||
final Button btnRemove = holder.itemView.findViewById(R.id.remove);
|
||||
btnRemove.setOnClickListener(v -> {
|
||||
mControllerRendererPool.getController(mContext, card.getCardType()).onDismissed(card);
|
||||
cardHolder.resetCard();
|
||||
mFlippedCardSet.remove(cardHolder);
|
||||
mFlippedCardSet.remove(holder);
|
||||
resetCardView(holder);
|
||||
mSliceLiveDataMap.get(card.getSliceUri()).removeObservers(mLifecycleOwner);
|
||||
});
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onSliceAction(@NonNull EventInfo eventInfo, @NonNull SliceItem sliceItem) {
|
||||
//TODO(b/79698338): Log user interaction
|
||||
|
||||
// sliceItem.getSlice().getUri() is like
|
||||
// content://android.settings.slices/action/wifi/_gen/0/_gen/0
|
||||
// contextualCard.getSliceUri() is prefix of sliceItem.getSlice().getUri()
|
||||
for (ContextualCard card : mCardSet) {
|
||||
if (sliceItem.getSlice().getUri().toString().startsWith(
|
||||
card.getSliceUri().toString())) {
|
||||
ContextualCardFeatureProvider contexualCardFeatureProvider =
|
||||
FeatureFactory.getFactory(mContext)
|
||||
.getContextualCardFeatureProvider(mContext);
|
||||
contexualCardFeatureProvider.logContextualCardClick(card,
|
||||
eventInfo.rowIndex, eventInfo.actionType);
|
||||
break;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@OnLifecycleEvent(Lifecycle.Event.ON_STOP)
|
||||
public void onStop() {
|
||||
mFlippedCardSet.stream().forEach(holder -> holder.resetCard());
|
||||
mFlippedCardSet.stream().forEach(holder -> resetCardView(holder));
|
||||
mFlippedCardSet.clear();
|
||||
}
|
||||
|
||||
public static class SliceViewHolder extends RecyclerView.ViewHolder {
|
||||
public final SliceView sliceView;
|
||||
public final ViewFlipper viewFlipper;
|
||||
private void resetCardView(RecyclerView.ViewHolder holder) {
|
||||
final ViewFlipper viewFlipper = holder.itemView.findViewById(R.id.view_flipper);
|
||||
viewFlipper.setDisplayedChild(0 /* whichChild */);
|
||||
}
|
||||
|
||||
public SliceViewHolder(View view) {
|
||||
super(view);
|
||||
sliceView = view.findViewById(R.id.slice_view);
|
||||
viewFlipper = view.findViewById(R.id.view_flipper);
|
||||
}
|
||||
|
||||
public void resetCard() {
|
||||
viewFlipper.setDisplayedChild(0);
|
||||
}
|
||||
private void flipCardToDismissalView(RecyclerView.ViewHolder holder) {
|
||||
final ViewFlipper viewFlipper = holder.itemView.findViewById(R.id.view_flipper);
|
||||
viewFlipper.showNext();
|
||||
}
|
||||
}
|
||||
|
||||
@@ -0,0 +1,99 @@
|
||||
/*
|
||||
* Copyright (C) 2019 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.homepage.contextualcards.slices;
|
||||
|
||||
import android.content.Context;
|
||||
import android.view.View;
|
||||
|
||||
import androidx.annotation.NonNull;
|
||||
import androidx.recyclerview.widget.RecyclerView;
|
||||
import androidx.slice.Slice;
|
||||
import androidx.slice.SliceItem;
|
||||
import androidx.slice.widget.EventInfo;
|
||||
import androidx.slice.widget.SliceView;
|
||||
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.homepage.contextualcards.ContextualCard;
|
||||
import com.android.settings.homepage.contextualcards.ContextualCardFeatureProvider;
|
||||
import com.android.settings.overlay.FeatureFactory;
|
||||
|
||||
import java.util.Set;
|
||||
|
||||
/**
|
||||
* Card renderer helper for {@link ContextualCard} built as slice full card.
|
||||
*/
|
||||
class SliceFullCardRendererHelper implements SliceView.OnSliceActionListener {
|
||||
private static final String TAG = "SliceFCRendererHelper";
|
||||
|
||||
private final Context mContext;
|
||||
|
||||
private Set<ContextualCard> mCardSet;
|
||||
|
||||
SliceFullCardRendererHelper(Context context) {
|
||||
mContext = context;
|
||||
}
|
||||
|
||||
RecyclerView.ViewHolder createViewHolder(View view) {
|
||||
return new SliceViewHolder(view);
|
||||
}
|
||||
|
||||
void bindView(RecyclerView.ViewHolder holder, ContextualCard card, Slice slice,
|
||||
Set<ContextualCard> cardSet) {
|
||||
final SliceViewHolder cardHolder = (SliceViewHolder) holder;
|
||||
cardHolder.sliceView.setScrollable(false);
|
||||
cardHolder.sliceView.setTag(card.getSliceUri());
|
||||
//TODO(b/114009676): We will soon have a field to decide what slice mode we should set.
|
||||
cardHolder.sliceView.setMode(SliceView.MODE_LARGE);
|
||||
cardHolder.sliceView.setSlice(slice);
|
||||
mCardSet = cardSet;
|
||||
// Set this listener so we can log the interaction users make on the slice
|
||||
cardHolder.sliceView.setOnSliceActionListener(this);
|
||||
|
||||
// Customize slice view for Settings
|
||||
cardHolder.sliceView.showTitleItems(true);
|
||||
if (card.isLargeCard()) {
|
||||
cardHolder.sliceView.showHeaderDivider(true);
|
||||
cardHolder.sliceView.showActionDividers(true);
|
||||
}
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onSliceAction(@NonNull EventInfo eventInfo, @NonNull SliceItem sliceItem) {
|
||||
// sliceItem.getSlice().getUri() is like
|
||||
// content://android.settings.slices/action/wifi/_gen/0/_gen/0
|
||||
// contextualCard.getSliceUri() is prefix of sliceItem.getSlice().getUri()
|
||||
final ContextualCardFeatureProvider contextualCardFeatureProvider =
|
||||
FeatureFactory.getFactory(mContext).getContextualCardFeatureProvider(mContext);
|
||||
for (ContextualCard card : mCardSet) {
|
||||
if (sliceItem.getSlice().getUri().toString().startsWith(
|
||||
card.getSliceUri().toString())) {
|
||||
contextualCardFeatureProvider.logContextualCardClick(card, eventInfo.rowIndex,
|
||||
eventInfo.actionType);
|
||||
break;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
static class SliceViewHolder extends RecyclerView.ViewHolder {
|
||||
public final SliceView sliceView;
|
||||
|
||||
public SliceViewHolder(View view) {
|
||||
super(view);
|
||||
sliceView = view.findViewById(R.id.slice_view);
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,85 @@
|
||||
/*
|
||||
* Copyright (C) 2019 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.homepage.contextualcards.slices;
|
||||
|
||||
import android.app.PendingIntent;
|
||||
import android.content.Context;
|
||||
import android.util.Log;
|
||||
import android.view.View;
|
||||
import android.widget.ImageView;
|
||||
import android.widget.LinearLayout;
|
||||
import android.widget.TextView;
|
||||
|
||||
import androidx.recyclerview.widget.RecyclerView;
|
||||
import androidx.slice.Slice;
|
||||
import androidx.slice.SliceMetadata;
|
||||
import androidx.slice.core.SliceAction;
|
||||
import androidx.slice.widget.EventInfo;
|
||||
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.homepage.contextualcards.ContextualCard;
|
||||
import com.android.settings.homepage.contextualcards.ContextualCardFeatureProvider;
|
||||
import com.android.settings.overlay.FeatureFactory;
|
||||
|
||||
/**
|
||||
* Card renderer helper for {@link ContextualCard} built as slice half card.
|
||||
*/
|
||||
class SliceHalfCardRendererHelper {
|
||||
private static final String TAG = "SliceHCRendererHelper";
|
||||
|
||||
private final Context mContext;
|
||||
|
||||
SliceHalfCardRendererHelper(Context context) {
|
||||
mContext = context;
|
||||
}
|
||||
|
||||
RecyclerView.ViewHolder createViewHolder(View view) {
|
||||
return new HalfCardViewHolder(view);
|
||||
}
|
||||
|
||||
void bindView(RecyclerView.ViewHolder holder, ContextualCard card, Slice slice) {
|
||||
final HalfCardViewHolder view = (HalfCardViewHolder) holder;
|
||||
final SliceMetadata sliceMetadata = SliceMetadata.from(mContext, slice);
|
||||
final SliceAction primaryAction = sliceMetadata.getPrimaryAction();
|
||||
view.icon.setImageDrawable(primaryAction.getIcon().loadDrawable(mContext));
|
||||
view.title.setText(primaryAction.getTitle());
|
||||
view.content.setOnClickListener(v -> {
|
||||
try {
|
||||
primaryAction.getAction().send();
|
||||
} catch (PendingIntent.CanceledException e) {
|
||||
Log.w(TAG, "Failed to start intent " + primaryAction.getTitle());
|
||||
}
|
||||
final ContextualCardFeatureProvider contextualCardFeatureProvider =
|
||||
FeatureFactory.getFactory(mContext).getContextualCardFeatureProvider(mContext);
|
||||
contextualCardFeatureProvider.logContextualCardClick(card, 0 /* row */,
|
||||
EventInfo.ACTION_TYPE_CONTENT);
|
||||
});
|
||||
}
|
||||
|
||||
static class HalfCardViewHolder extends RecyclerView.ViewHolder {
|
||||
public final LinearLayout content;
|
||||
public final ImageView icon;
|
||||
public final TextView title;
|
||||
|
||||
public HalfCardViewHolder(View itemView) {
|
||||
super(itemView);
|
||||
content = itemView.findViewById(R.id.content);
|
||||
icon = itemView.findViewById(android.R.id.icon);
|
||||
title = itemView.findViewById(android.R.id.title);
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -17,6 +17,7 @@
|
||||
package com.android.settings.network.telephony;
|
||||
|
||||
import android.app.Dialog;
|
||||
import android.app.settings.SettingsEnums;
|
||||
import android.content.Context;
|
||||
import android.content.DialogInterface;
|
||||
import android.os.Bundle;
|
||||
@@ -109,8 +110,7 @@ public class MobileDataDialogFragment extends InstrumentedDialogFragment impleme
|
||||
|
||||
@Override
|
||||
public int getMetricsCategory() {
|
||||
//TODO(b/114749736): add metric id for this fragment
|
||||
return 0;
|
||||
return SettingsEnums.MOBILE_DATA_DIALOG;
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
@@ -196,7 +196,6 @@ public class MobileNetworkSettings extends RestrictedDashboardFragment {
|
||||
}
|
||||
}
|
||||
|
||||
//TODO(b/114749736): update search provider
|
||||
public static final Indexable.SearchIndexProvider SEARCH_INDEX_DATA_PROVIDER =
|
||||
new BaseSearchIndexProvider() {
|
||||
@Override
|
||||
|
||||
@@ -116,8 +116,6 @@ public class PreferredNetworkModePreferenceController extends BasePreferenceCont
|
||||
}
|
||||
|
||||
private int getPreferredNetworkModeSummaryResId(int NetworkMode) {
|
||||
//TODO(b/114749736): refactor it to "Preferred network mode: <Mode>", instead of building
|
||||
// string for each type...
|
||||
switch (NetworkMode) {
|
||||
case TelephonyManager.NETWORK_MODE_TDSCDMA_GSM_WCDMA:
|
||||
return R.string.preferred_network_mode_tdscdma_gsm_wcdma_summary;
|
||||
|
||||
@@ -17,6 +17,7 @@ package com.android.settings.network.telephony;
|
||||
|
||||
import android.app.AlertDialog;
|
||||
import android.app.Dialog;
|
||||
import android.app.settings.SettingsEnums;
|
||||
import android.content.Context;
|
||||
import android.content.DialogInterface;
|
||||
import android.content.DialogInterface.OnClickListener;
|
||||
@@ -78,8 +79,7 @@ public class RoamingDialogFragment extends InstrumentedDialogFragment implements
|
||||
|
||||
@Override
|
||||
public int getMetricsCategory() {
|
||||
//TODO(b/114749736): add category for roaming dialog
|
||||
return 0;
|
||||
return SettingsEnums.MOBILE_ROAMING_DIALOG;
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
@@ -17,6 +17,7 @@
|
||||
package com.android.settings.slices;
|
||||
|
||||
import android.annotation.MainThread;
|
||||
import android.annotation.Nullable;
|
||||
import android.content.Context;
|
||||
import android.net.Uri;
|
||||
import android.util.ArrayMap;
|
||||
@@ -58,20 +59,29 @@ public abstract class SliceBackgroundWorker<E> implements Closeable {
|
||||
mUri = uri;
|
||||
}
|
||||
|
||||
public Uri getUri() {
|
||||
protected Uri getUri() {
|
||||
return mUri;
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns the singleton instance of the {@link SliceBackgroundWorker} for specified {@link Uri}
|
||||
* if exists
|
||||
*/
|
||||
@Nullable
|
||||
public static SliceBackgroundWorker getInstance(Uri uri) {
|
||||
return LIVE_WORKERS.get(uri);
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns the singleton instance of the {@link SliceBackgroundWorker} for specified {@link
|
||||
* CustomSliceable}
|
||||
*/
|
||||
public static SliceBackgroundWorker getInstance(Context context, CustomSliceable sliceable) {
|
||||
static SliceBackgroundWorker getInstance(Context context, CustomSliceable sliceable) {
|
||||
final Uri uri = sliceable.getUri();
|
||||
final Class<? extends SliceBackgroundWorker> workerClass =
|
||||
sliceable.getBackgroundWorkerClass();
|
||||
SliceBackgroundWorker worker = LIVE_WORKERS.get(uri);
|
||||
SliceBackgroundWorker worker = getInstance(uri);
|
||||
if (worker == null) {
|
||||
final Class<? extends SliceBackgroundWorker> workerClass =
|
||||
sliceable.getBackgroundWorkerClass();
|
||||
worker = createInstance(context, uri, workerClass);
|
||||
LIVE_WORKERS.put(uri, worker);
|
||||
}
|
||||
|
||||
@@ -105,8 +105,8 @@ public class WifiSlice implements CustomSliceable {
|
||||
return listBuilder.build();
|
||||
}
|
||||
|
||||
final List<AccessPoint> results =
|
||||
SliceBackgroundWorker.getInstance(mContext, this).getResults();
|
||||
final SliceBackgroundWorker worker = SliceBackgroundWorker.getInstance(getUri());
|
||||
final List<AccessPoint> results = worker != null ? worker.getResults() : null;
|
||||
|
||||
// Need a loading text when results are not ready.
|
||||
boolean needLoadingRow = results == null;
|
||||
|
||||
@@ -53,10 +53,8 @@ public class WifiTetherSSIDPreferenceController extends WifiTetherBasePreference
|
||||
final WifiConfiguration config = mWifiManager.getWifiApConfiguration();
|
||||
if (config != null) {
|
||||
mSSID = config.SSID;
|
||||
Log.d(TAG, "Updating SSID in Preference, " + mSSID);
|
||||
} else {
|
||||
mSSID = DEFAULT_SSID;
|
||||
Log.d(TAG, "Updating to default SSID in Preference, " + mSSID);
|
||||
}
|
||||
((ValidatedEditTextPreference) mPreference).setValidator(this);
|
||||
updateSsidDisplay((EditTextPreference) mPreference);
|
||||
|
||||
@@ -29,16 +29,16 @@ import android.net.Uri;
|
||||
|
||||
import com.android.settings.slices.CustomSliceRegistry;
|
||||
|
||||
import java.util.ArrayList;
|
||||
import java.util.List;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import org.junit.Before;
|
||||
import org.junit.Test;
|
||||
import org.junit.runner.RunWith;
|
||||
import org.robolectric.RobolectricTestRunner;
|
||||
import org.robolectric.RuntimeEnvironment;
|
||||
|
||||
import java.util.ArrayList;
|
||||
import java.util.List;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
@RunWith(RobolectricTestRunner.class)
|
||||
public class ContextualCardLoaderTest {
|
||||
|
||||
@@ -82,29 +82,29 @@ public class ContextualCardLoaderTest {
|
||||
}
|
||||
|
||||
@Test
|
||||
public void getFinalDisplayableCards_twoEligibleCards_shouldShowAll() {
|
||||
public void getDisplayableCards_twoEligibleCards_shouldShowAll() {
|
||||
final List<ContextualCard> cards = getContextualCardList().stream().limit(2)
|
||||
.collect(Collectors.toList());
|
||||
doReturn(cards).when(mContextualCardLoader).filterEligibleCards(any(List.class));
|
||||
|
||||
final List<ContextualCard> result = mContextualCardLoader.getFinalDisplayableCards(cards);
|
||||
final List<ContextualCard> result = mContextualCardLoader.getDisplayableCards(cards);
|
||||
|
||||
assertThat(result).hasSize(cards.size());
|
||||
}
|
||||
|
||||
@Test
|
||||
public void getFinalDisplayableCards_fiveEligibleCardsNoLarge_shouldShowDefaultCardCount() {
|
||||
public void getDisplayableCards_fiveEligibleCardsNoLarge_shouldShowDefaultCardCount() {
|
||||
final List<ContextualCard> fiveCards = getContextualCardListWithNoLargeCard();
|
||||
doReturn(fiveCards).when(mContextualCardLoader).filterEligibleCards(any(List.class));
|
||||
|
||||
final List<ContextualCard> result = mContextualCardLoader.getFinalDisplayableCards(
|
||||
final List<ContextualCard> result = mContextualCardLoader.getDisplayableCards(
|
||||
fiveCards);
|
||||
|
||||
assertThat(result).hasSize(DEFAULT_CARD_COUNT);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void getFinalDisplayableCards_threeEligibleCardsOneLarge_shouldShowThreeCards() {
|
||||
public void getDisplayableCards_threeEligibleCardsOneLarge_shouldShowThreeCards() {
|
||||
final List<ContextualCard> cards = getContextualCardList().stream().limit(2)
|
||||
.collect(Collectors.toList());
|
||||
cards.add(new ContextualCard.Builder()
|
||||
@@ -115,18 +115,18 @@ public class ContextualCardLoaderTest {
|
||||
.build());
|
||||
doReturn(cards).when(mContextualCardLoader).filterEligibleCards(any(List.class));
|
||||
|
||||
final List<ContextualCard> result = mContextualCardLoader.getFinalDisplayableCards(cards);
|
||||
final List<ContextualCard> result = mContextualCardLoader.getDisplayableCards(cards);
|
||||
|
||||
assertThat(result).hasSize(3);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void getFinalDisplayableCards_threeEligibleCardsTwoLarge_shouldShowTwoCards() {
|
||||
public void getDisplayableCards_threeEligibleCardsTwoLarge_shouldShowTwoCards() {
|
||||
final List<ContextualCard> threeCards = getContextualCardList().stream().limit(3)
|
||||
.collect(Collectors.toList());
|
||||
doReturn(threeCards).when(mContextualCardLoader).filterEligibleCards(any(List.class));
|
||||
|
||||
final List<ContextualCard> result = mContextualCardLoader.getFinalDisplayableCards(
|
||||
final List<ContextualCard> result = mContextualCardLoader.getDisplayableCards(
|
||||
threeCards);
|
||||
|
||||
assertThat(result).hasSize(2);
|
||||
|
||||
@@ -32,6 +32,8 @@ import android.util.ArrayMap;
|
||||
import com.android.settings.homepage.contextualcards.conditional.ConditionFooterContextualCard;
|
||||
import com.android.settings.homepage.contextualcards.conditional.ConditionHeaderContextualCard;
|
||||
import com.android.settings.homepage.contextualcards.conditional.ConditionalContextualCard;
|
||||
import com.android.settings.intelligence.ContextualCardProto;
|
||||
import com.android.settings.slices.CustomSliceRegistry;
|
||||
|
||||
import org.junit.Before;
|
||||
import org.junit.Test;
|
||||
@@ -203,6 +205,134 @@ public class ContextualCardManagerTest {
|
||||
assertThat(actualCards).containsExactlyElementsIn(expectedCards);
|
||||
}
|
||||
|
||||
|
||||
@Test
|
||||
public void assignCardWidth_noSuggestionCards_shouldNotHaveHalfCards() {
|
||||
final List<Integer> categories = Arrays.asList(
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE
|
||||
);
|
||||
final List<ContextualCard> noSuggestionCards = buildCategoriedCards(getContextualCardList(),
|
||||
categories);
|
||||
|
||||
final List<ContextualCard> result = mManager.assignCardWidth(noSuggestionCards);
|
||||
|
||||
assertThat(result).hasSize(5);
|
||||
for (ContextualCard card : result) {
|
||||
assertThat(card.isHalfWidth()).isFalse();
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
public void assignCardWidth_oneSuggestionCards_shouldNotHaveHalfCards() {
|
||||
final List<Integer> categories = Arrays.asList(
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE
|
||||
);
|
||||
final List<ContextualCard> oneSuggestionCards = buildCategoriedCards(
|
||||
getContextualCardList(), categories);
|
||||
|
||||
final List<ContextualCard> result = mManager.assignCardWidth(oneSuggestionCards);
|
||||
|
||||
assertThat(result).hasSize(5);
|
||||
for (ContextualCard card : result) {
|
||||
assertThat(card.isHalfWidth()).isFalse();
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
public void assignCardWidth_twoConsecutiveSuggestionCards_shouldHaveTwoHalfCards() {
|
||||
final List<Integer> categories = Arrays.asList(
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE
|
||||
);
|
||||
final List<ContextualCard> twoConsecutiveSuggestionCards = buildCategoriedCards(
|
||||
getContextualCardList(), categories);
|
||||
final List<Boolean> expectedValues = Arrays.asList(false, false, true, true, false);
|
||||
|
||||
final List<ContextualCard> result = mManager.assignCardWidth(
|
||||
twoConsecutiveSuggestionCards);
|
||||
|
||||
assertThat(result).hasSize(5);
|
||||
for (int i = 0; i < result.size(); i++) {
|
||||
assertThat(result.get(i).isHalfWidth()).isEqualTo(expectedValues.get(i));
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
public void assignCardWidth_twoNonConsecutiveSuggestionCards_shouldNotHaveHalfCards() {
|
||||
final List<Integer> categories = Arrays.asList(
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE
|
||||
);
|
||||
final List<ContextualCard> twoNonConsecutiveSuggestionCards = buildCategoriedCards(
|
||||
getContextualCardList(), categories);
|
||||
|
||||
final List<ContextualCard> result = mManager.assignCardWidth(
|
||||
twoNonConsecutiveSuggestionCards);
|
||||
|
||||
assertThat(result).hasSize(5);
|
||||
for (ContextualCard card : result) {
|
||||
assertThat(card.isHalfWidth()).isFalse();
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
public void assignCardWidth_threeConsecutiveSuggestionCards_shouldHaveTwoHalfCards() {
|
||||
final List<Integer> categories = Arrays.asList(
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE
|
||||
);
|
||||
final List<ContextualCard> threeConsecutiveSuggestionCards = buildCategoriedCards(
|
||||
getContextualCardList(), categories);
|
||||
final List<Boolean> expectedValues = Arrays.asList(false, true, true, false, false);
|
||||
|
||||
final List<ContextualCard> result = mManager.assignCardWidth(
|
||||
threeConsecutiveSuggestionCards);
|
||||
|
||||
assertThat(result).hasSize(5);
|
||||
for (int i = 0; i < result.size(); i++) {
|
||||
assertThat(result.get(i).isHalfWidth()).isEqualTo(expectedValues.get(i));
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
public void assignCardWidth_fourConsecutiveSuggestionCards_shouldHaveFourHalfCards() {
|
||||
final List<Integer> categories = Arrays.asList(
|
||||
ContextualCardProto.ContextualCard.Category.IMPORTANT_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE,
|
||||
ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE
|
||||
);
|
||||
final List<ContextualCard> fourConsecutiveSuggestionCards = buildCategoriedCards(
|
||||
getContextualCardList(), categories);
|
||||
final List<Boolean> expectedValues = Arrays.asList(false, true, true, true, true);
|
||||
|
||||
final List<ContextualCard> result = mManager.assignCardWidth(
|
||||
fourConsecutiveSuggestionCards);
|
||||
|
||||
assertThat(result).hasSize(5);
|
||||
for (int i = 0; i < result.size(); i++) {
|
||||
assertThat(result.get(i).isHalfWidth()).isEqualTo(expectedValues.get(i));
|
||||
}
|
||||
}
|
||||
|
||||
private ContextualCard buildContextualCard(String sliceUri) {
|
||||
return new ContextualCard.Builder()
|
||||
.setName(TEST_SLICE_NAME)
|
||||
@@ -210,4 +340,45 @@ public class ContextualCardManagerTest {
|
||||
.setSliceUri(Uri.parse(sliceUri))
|
||||
.build();
|
||||
}
|
||||
|
||||
private List<ContextualCard> buildCategoriedCards(List<ContextualCard> cards,
|
||||
List<Integer> categories) {
|
||||
final List<ContextualCard> result = new ArrayList<>();
|
||||
for (int i = 0; i < cards.size(); i++) {
|
||||
result.add(cards.get(i).mutate().setCategory(categories.get(i)).build());
|
||||
}
|
||||
return result;
|
||||
}
|
||||
|
||||
private List<ContextualCard> getContextualCardList() {
|
||||
final List<ContextualCard> cards = new ArrayList<>();
|
||||
cards.add(new ContextualCard.Builder()
|
||||
.setName("test_wifi")
|
||||
.setCardType(ContextualCard.CardType.SLICE)
|
||||
.setSliceUri(CustomSliceRegistry.CONTEXTUAL_WIFI_SLICE_URI)
|
||||
.build());
|
||||
cards.add(new ContextualCard.Builder()
|
||||
.setName("test_flashlight")
|
||||
.setCardType(ContextualCard.CardType.SLICE)
|
||||
.setSliceUri(
|
||||
Uri.parse("content://com.android.settings.test.slices/action/flashlight"))
|
||||
.build());
|
||||
cards.add(new ContextualCard.Builder()
|
||||
.setName("test_connected")
|
||||
.setCardType(ContextualCard.CardType.SLICE)
|
||||
.setSliceUri(CustomSliceRegistry.BLUETOOTH_DEVICES_SLICE_URI)
|
||||
.build());
|
||||
cards.add(new ContextualCard.Builder()
|
||||
.setName("test_gesture")
|
||||
.setCardType(ContextualCard.CardType.SLICE)
|
||||
.setSliceUri(Uri.parse(
|
||||
"content://com.android.settings.test.slices/action/gesture_pick_up"))
|
||||
.build());
|
||||
cards.add(new ContextualCard.Builder()
|
||||
.setName("test_battery")
|
||||
.setCardType(ContextualCard.CardType.SLICE)
|
||||
.setSliceUri(CustomSliceRegistry.BATTERY_INFO_SLICE_URI)
|
||||
.build());
|
||||
return cards;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -112,7 +112,8 @@ public class ConditionContextualCardControllerTest {
|
||||
}
|
||||
|
||||
@Test
|
||||
public void getConditionalCards_hasOneConditionCard_shouldGetOneFullWidthCard() {
|
||||
public void getConditionalCards_hasOneConditionCardAndExpanded_shouldGetOneFullWidthCard() {
|
||||
mController.setIsExpanded(true);
|
||||
final Map<Integer, List<ContextualCard>> conditionalCards =
|
||||
mController.buildConditionalCardsWithFooterOrHeader(generateConditionCards(1));
|
||||
|
||||
@@ -120,11 +121,24 @@ public class ConditionContextualCardControllerTest {
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL)).hasSize(1);
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL).get(0).isHalfWidth()).isFalse();
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL_HEADER)).isEmpty();
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL_FOOTER)).isNotEmpty();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void getConditionalCards_hasOneConditionCardAndCollapsed_shouldGetConditionalHeader() {
|
||||
mController.setIsExpanded(false);
|
||||
final Map<Integer, List<ContextualCard>> conditionalCards =
|
||||
mController.buildConditionalCardsWithFooterOrHeader(generateConditionCards(1));
|
||||
|
||||
assertThat(conditionalCards).hasSize(3);
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL)).isEmpty();
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL_HEADER)).isNotEmpty();
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL_FOOTER)).isEmpty();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void getConditionalCards_hasTwoConditionCards_shouldGetTwoHalfWidthCards() {
|
||||
public void getConditionalCards_hasTwoConditionCardsAndExpanded_shouldGetTwoHalfWidthCards() {
|
||||
mController.setIsExpanded(true);
|
||||
final Map<Integer, List<ContextualCard>> conditionalCards =
|
||||
mController.buildConditionalCardsWithFooterOrHeader(generateConditionCards(2));
|
||||
|
||||
@@ -134,6 +148,18 @@ public class ConditionContextualCardControllerTest {
|
||||
assertThat(card.isHalfWidth()).isTrue();
|
||||
}
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL_HEADER)).isEmpty();
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL_FOOTER)).isNotEmpty();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void getConditionalCards_hasTwoConditionCardsAndCollapsed_shouldGetConditionalHeader() {
|
||||
mController.setIsExpanded(false);
|
||||
final Map<Integer, List<ContextualCard>> conditionalCards =
|
||||
mController.buildConditionalCardsWithFooterOrHeader(generateConditionCards(2));
|
||||
|
||||
assertThat(conditionalCards).hasSize(3);
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL)).isEmpty();
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL_HEADER)).isNotEmpty();
|
||||
assertThat(conditionalCards.get(CardType.CONDITIONAL_FOOTER)).isEmpty();
|
||||
}
|
||||
|
||||
|
||||
@@ -78,17 +78,6 @@ public class SliceContextualCardRendererTest {
|
||||
mControllerRendererPool);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void bindView_shouldSetScrollableToFalse() {
|
||||
RecyclerView.ViewHolder viewHolder = getSliceViewHolder();
|
||||
|
||||
mRenderer.bindView(viewHolder, buildContextualCard(TEST_SLICE_URI));
|
||||
|
||||
assertThat(
|
||||
((SliceContextualCardRenderer.SliceViewHolder) viewHolder).sliceView.isScrollable
|
||||
()).isFalse();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void bindView_invalidScheme_sliceShouldBeNull() {
|
||||
final Uri sliceUri = Uri.parse("contet://com.android.settings.slices/action/flashlight");
|
||||
@@ -97,7 +86,7 @@ public class SliceContextualCardRendererTest {
|
||||
mRenderer.bindView(viewHolder, buildContextualCard(sliceUri));
|
||||
|
||||
assertThat(
|
||||
((SliceContextualCardRenderer.SliceViewHolder) viewHolder).sliceView.getSlice())
|
||||
((SliceFullCardRendererHelper.SliceViewHolder) viewHolder).sliceView.getSlice())
|
||||
.isNull();
|
||||
}
|
||||
|
||||
|
||||
@@ -0,0 +1,151 @@
|
||||
/*
|
||||
* Copyright (C) 2019 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.homepage.contextualcards.slices;
|
||||
|
||||
import static com.android.settings.homepage.contextualcards.slices.SliceContextualCardRenderer.VIEW_TYPE_FULL_WIDTH;
|
||||
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
|
||||
import android.app.Activity;
|
||||
import android.app.PendingIntent;
|
||||
import android.content.Intent;
|
||||
import android.net.Uri;
|
||||
import android.view.LayoutInflater;
|
||||
import android.view.View;
|
||||
|
||||
import androidx.core.graphics.drawable.IconCompat;
|
||||
import androidx.recyclerview.widget.LinearLayoutManager;
|
||||
import androidx.recyclerview.widget.RecyclerView;
|
||||
import androidx.slice.Slice;
|
||||
import androidx.slice.SliceProvider;
|
||||
import androidx.slice.builders.ListBuilder;
|
||||
import androidx.slice.builders.SliceAction;
|
||||
import androidx.slice.widget.SliceLiveData;
|
||||
import androidx.slice.widget.SliceView;
|
||||
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.homepage.contextualcards.ContextualCard;
|
||||
import com.android.settings.homepage.contextualcards.slices.SliceFullCardRendererHelper.SliceViewHolder;
|
||||
import com.android.settings.intelligence.ContextualCardProto;
|
||||
|
||||
import org.junit.Before;
|
||||
import org.junit.Test;
|
||||
import org.junit.runner.RunWith;
|
||||
import org.robolectric.Robolectric;
|
||||
import org.robolectric.RobolectricTestRunner;
|
||||
|
||||
import java.util.Collections;
|
||||
|
||||
@RunWith(RobolectricTestRunner.class)
|
||||
public class SliceFullCardRendererHelperTest {
|
||||
|
||||
private static final Uri TEST_SLICE_URI = Uri.parse("content://test/test");
|
||||
|
||||
private Activity mActivity;
|
||||
private SliceFullCardRendererHelper mHelper;
|
||||
|
||||
@Before
|
||||
public void setUp() {
|
||||
// Set-up specs for SliceMetadata.
|
||||
SliceProvider.setSpecs(SliceLiveData.SUPPORTED_SPECS);
|
||||
mActivity = Robolectric.buildActivity(Activity.class).create().get();
|
||||
mActivity.setTheme(R.style.Theme_Settings_Home);
|
||||
mHelper = new SliceFullCardRendererHelper(mActivity);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void createViewHolder_shouldAlwaysReturnSliceViewHolder() {
|
||||
final RecyclerView.ViewHolder viewHolder = getSliceViewHolder();
|
||||
|
||||
assertThat(viewHolder).isInstanceOf(SliceViewHolder.class);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void bindView_shouldSetScrollableToFalse() {
|
||||
final RecyclerView.ViewHolder viewHolder = getSliceViewHolder();
|
||||
|
||||
mHelper.bindView(viewHolder, buildContextualCard(), buildSlice(), Collections.emptySet());
|
||||
|
||||
assertThat(((SliceViewHolder) viewHolder).sliceView.isScrollable()).isFalse();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void bindView_shouldSetTagToSliceUri() {
|
||||
final RecyclerView.ViewHolder viewHolder = getSliceViewHolder();
|
||||
final ContextualCard card = buildContextualCard();
|
||||
|
||||
mHelper.bindView(viewHolder, card, buildSlice(), Collections.emptySet());
|
||||
|
||||
assertThat(((SliceViewHolder) viewHolder).sliceView.getTag()).isEqualTo(card.getSliceUri());
|
||||
}
|
||||
|
||||
@Test
|
||||
public void bindView_shouldSetModeToLarge() {
|
||||
final RecyclerView.ViewHolder viewHolder = getSliceViewHolder();
|
||||
|
||||
mHelper.bindView(viewHolder, buildContextualCard(), buildSlice(), Collections.emptySet());
|
||||
|
||||
assertThat(((SliceViewHolder) viewHolder).sliceView.getMode()).isEqualTo(
|
||||
SliceView.MODE_LARGE);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void bindView_shouldSetSlice() {
|
||||
final RecyclerView.ViewHolder viewHolder = getSliceViewHolder();
|
||||
|
||||
mHelper.bindView(viewHolder, buildContextualCard(), buildSlice(), Collections.emptySet());
|
||||
|
||||
assertThat(((SliceViewHolder) viewHolder).sliceView.getSlice().getUri()).isEqualTo(
|
||||
TEST_SLICE_URI);
|
||||
}
|
||||
|
||||
private RecyclerView.ViewHolder getSliceViewHolder() {
|
||||
final RecyclerView recyclerView = new RecyclerView(mActivity);
|
||||
recyclerView.setLayoutManager(new LinearLayoutManager(mActivity));
|
||||
final View view = LayoutInflater.from(mActivity).inflate(VIEW_TYPE_FULL_WIDTH, recyclerView,
|
||||
false);
|
||||
return mHelper.createViewHolder(view);
|
||||
}
|
||||
|
||||
private ContextualCard buildContextualCard() {
|
||||
return new ContextualCard.Builder()
|
||||
.setName("test_name")
|
||||
.setCategory(ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE)
|
||||
.setCardType(ContextualCard.CardType.SLICE)
|
||||
.setSliceUri(TEST_SLICE_URI)
|
||||
.setIsHalfWidth(false /* isHalfWidth */)
|
||||
.build();
|
||||
}
|
||||
|
||||
private Slice buildSlice() {
|
||||
final String title = "test_title";
|
||||
final IconCompat icon = IconCompat.createWithResource(mActivity, R.drawable.empty_icon);
|
||||
final PendingIntent pendingIntent = PendingIntent.getActivity(
|
||||
mActivity,
|
||||
title.hashCode() /* requestCode */,
|
||||
new Intent("test action"),
|
||||
0 /* flags */);
|
||||
final SliceAction action
|
||||
= SliceAction.createDeeplink(pendingIntent, icon, ListBuilder.SMALL_IMAGE, title);
|
||||
return new ListBuilder(mActivity, TEST_SLICE_URI, ListBuilder.INFINITY)
|
||||
.addRow(new ListBuilder.RowBuilder()
|
||||
.addEndItem(icon, ListBuilder.ICON_IMAGE)
|
||||
.setTitle(title)
|
||||
.setPrimaryAction(action))
|
||||
.build();
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,119 @@
|
||||
/*
|
||||
* Copyright (C) 2019 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.homepage.contextualcards.slices;
|
||||
|
||||
import static com.android.settings.homepage.contextualcards.slices.SliceContextualCardRenderer.VIEW_TYPE_HALF_WIDTH;
|
||||
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
|
||||
import android.app.Activity;
|
||||
import android.app.PendingIntent;
|
||||
import android.content.Intent;
|
||||
import android.net.Uri;
|
||||
import android.view.LayoutInflater;
|
||||
import android.view.View;
|
||||
|
||||
import androidx.core.graphics.drawable.IconCompat;
|
||||
import androidx.recyclerview.widget.LinearLayoutManager;
|
||||
import androidx.recyclerview.widget.RecyclerView;
|
||||
import androidx.slice.Slice;
|
||||
import androidx.slice.SliceProvider;
|
||||
import androidx.slice.builders.ListBuilder;
|
||||
import androidx.slice.builders.SliceAction;
|
||||
import androidx.slice.widget.SliceLiveData;
|
||||
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.homepage.contextualcards.ContextualCard;
|
||||
import com.android.settings.homepage.contextualcards.slices.SliceHalfCardRendererHelper.HalfCardViewHolder;
|
||||
import com.android.settings.intelligence.ContextualCardProto;
|
||||
|
||||
import org.junit.Before;
|
||||
import org.junit.Test;
|
||||
import org.junit.runner.RunWith;
|
||||
import org.robolectric.Robolectric;
|
||||
import org.robolectric.RobolectricTestRunner;
|
||||
|
||||
@RunWith(RobolectricTestRunner.class)
|
||||
public class SliceHalfCardRendererHelperTest {
|
||||
|
||||
private static final Uri TEST_SLICE_URI = Uri.parse("content://test/test");
|
||||
|
||||
private Activity mActivity;
|
||||
private SliceHalfCardRendererHelper mHelper;
|
||||
|
||||
@Before
|
||||
public void setUp() {
|
||||
// Set-up specs for SliceMetadata.
|
||||
SliceProvider.setSpecs(SliceLiveData.SUPPORTED_SPECS);
|
||||
mActivity = Robolectric.buildActivity(Activity.class).create().get();
|
||||
mActivity.setTheme(R.style.Theme_Settings_Home);
|
||||
mHelper = new SliceHalfCardRendererHelper(mActivity);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void createViewHolder_shouldAlwaysReturnCustomViewHolder() {
|
||||
final RecyclerView.ViewHolder viewHolder = getHalfCardViewHolder();
|
||||
|
||||
assertThat(viewHolder).isInstanceOf(HalfCardViewHolder.class);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void bindView_shouldSetTitle() {
|
||||
final RecyclerView.ViewHolder viewHolder = getHalfCardViewHolder();
|
||||
|
||||
mHelper.bindView(viewHolder, buildContextualCard(), buildSlice());
|
||||
|
||||
assertThat(((HalfCardViewHolder) viewHolder).title.getText()).isEqualTo("test_title");
|
||||
}
|
||||
|
||||
private RecyclerView.ViewHolder getHalfCardViewHolder() {
|
||||
final RecyclerView recyclerView = new RecyclerView(mActivity);
|
||||
recyclerView.setLayoutManager(new LinearLayoutManager(mActivity));
|
||||
final View view = LayoutInflater.from(mActivity).inflate(VIEW_TYPE_HALF_WIDTH, recyclerView,
|
||||
false);
|
||||
|
||||
return mHelper.createViewHolder(view);
|
||||
}
|
||||
|
||||
private ContextualCard buildContextualCard() {
|
||||
return new ContextualCard.Builder()
|
||||
.setName("test_name")
|
||||
.setCategory(ContextualCardProto.ContextualCard.Category.SUGGESTION_VALUE)
|
||||
.setCardType(ContextualCard.CardType.SLICE)
|
||||
.setSliceUri(TEST_SLICE_URI)
|
||||
.setIsHalfWidth(false /* isHalfWidth */)
|
||||
.build();
|
||||
}
|
||||
|
||||
private Slice buildSlice() {
|
||||
final String title = "test_title";
|
||||
final IconCompat icon = IconCompat.createWithResource(mActivity, R.drawable.empty_icon);
|
||||
final PendingIntent pendingIntent = PendingIntent.getActivity(
|
||||
mActivity,
|
||||
title.hashCode() /* requestCode */,
|
||||
new Intent("test action"),
|
||||
0 /* flags */);
|
||||
final SliceAction action
|
||||
= SliceAction.createDeeplink(pendingIntent, icon, ListBuilder.SMALL_IMAGE, title);
|
||||
return new ListBuilder(mActivity, TEST_SLICE_URI, ListBuilder.INFINITY)
|
||||
.addRow(new ListBuilder.RowBuilder()
|
||||
.addEndItem(icon, ListBuilder.ICON_IMAGE)
|
||||
.setTitle(title)
|
||||
.setPrimaryAction(action))
|
||||
.build();
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user